I briefly looked into adding this... but looks like a lot of the bootstrap plumbing has changed. I'm a bit bewildered as to why we are booting up a temporary kernel before we get to command-line option parsing. I'm also a bit mystified as to why option parsing is now spread over several classes. IMO the bootstrap has become overly complicated... and probably needs some love to get it sorted. The task of adding commons-cli to the single Daemon class has become a wee bit more difficult due to all of this, so I'm gonna put this off for a little bit more while I soak up all of the recent changes and let it congeal in my head.

In the meantime... can anyone tell me why we are booting up a kernel before cli parsing? From what has congealed in my head so far... this is for access to the repository for classloading of bootstrap classes. Is there any other reason why we boot a kernel here before cli parsing?

--jason


On Feb 27, 2007, at 4:20 PM, Matt Hogstrom wrote:

Sounds good.

On Feb 26, 2007, at 9:02 PM, Jason Dillon wrote:

Anyone mind if I re-write org.apache.geronimo.system.main.Daemon to use commons-cli to handle option processing?

Use of commons-cli 1.0 add ~30k to the bootstrap classpath (uses commons-lang, which is already there), and will greatly simplify this code, and make it a little easier/nicer to add more command- line options in the future as well as providing.

--jason



Reply via email to