Copying ant code requires many classes to be duplicated ( Execute, Os, CommandLauncher and it's implementations, Process, ExecuteStreamHandler, ExecuteWatchdog ...)
I've attached a simplier implementation to http://issues.apache.org/jira/browse/CONFIGURATION-284 that delegates to ant, addidn an optional dependency. I also notice dependencies on commons-jxpath and commons-codec should be optional in the POM. Nico. 2007/9/20, nicolas de loof <[EMAIL PROTECTED]>: > I'll look at this and attach a patch to Jira. > thanks. > > 2007/9/20, Oliver Heger <[EMAIL PROTECTED]>: > > nicolas de loof wrote: > > > Could you please tell me what would be better : > > > > > > - copy/paste code from ANT to avoid a new dependency just for some methods > > > - use the ant code as a new (optional) dependency > > > > > > Nico. > > > > I would prefer not to add an additional dependency. If we talk about one > > or two classes, simply copying them seems to be alright. Or are there > > more classes involved? > > > > Oliver > > > > > > > > 2007/9/20, nicolas de loof <[EMAIL PROTECTED]>: > > >> Your right, I was reading javadoc to fast > > >> > > >> 2007/9/19, sebb <[EMAIL PROTECTED]>: > > >>> On 19/09/2007, nicolas de loof <[EMAIL PROTECTED]> wrote: > > >>>> Looking at > > >>>> http://svn.apache.org/viewvc/ant/core/trunk/src/main/org/apache/tools/ant/taskdefs/Execute.java?view=markup > > >>>> procEnvironment() > > >>>> > > >>>> Netware, os/2, z/os, unixes and windows are supported. OpenVMS and > > >>>> MacOS <= 9 seems to be unsupported platforms. MacOS <=9 has no > > >>>> environment variable at all ! > > >>> Huh? > > >>> OpenVMS does seem to be supported - by parsing the output of "Show > > >>> logical". > > >>> > > >>>> This method also has been "un deprecated" on Java 5 and can be used on > > >>>> such a runtime. We can invoke it if System.getProprty("java.version") > > >>>> returns > 1.5. > > >>>> > > >>>> Nico. > > >>>> > > >>>> > > >>>> 2007/9/19, nicolas de loof <[EMAIL PROTECTED]>: > > >>>>> My idea was simply to copy/paste code from the ant Property class. > > >>>>> AFAIK this one has been largely tested and works on many (if not all > > >>>>> ?) platforms. > > >>>>> > > >>>>> Nico. > > >>>>> > > >>>>> 2007/9/19, Oliver Heger <[EMAIL PROTECTED]>: > > >>>>>> nicolas de loof wrote: > > >>>>>>> Hello, > > >>>>>>> > > >>>>>>> Did anyone allready ask for support in commons-configuration to > > >>>>>>> acces > > >>>>>>> environement variables, the same way it is available with ant > > >>>>>>> <property> ? > > >>>>>>> > > >>>>>>> I'll need it and can contribute for this. > > >>>>>>> > > >>>>>>> Nico. > > >>>>>>> > > >>>>>> There is an enhancement ticket about this topic [1] with a longer > > >>>>>> discussion. There was also a thread on the mailing list at this time. > > >>>>>> > > >>>>>> How would you implement access to OS environment variables in a > > >>>>>> portable > > >>>>>> way? > > >>>>>> > > >>>>>> Oliver > > >>>>>> > > >>>>>> [1] http://issues.apache.org/jira/browse/CONFIGURATION-284 > > >>>>>> > > >>>>>> --------------------------------------------------------------------- > > >>>>>> To unsubscribe, e-mail: [EMAIL PROTECTED] > > >>>>>> For additional commands, e-mail: [EMAIL PROTECTED] > > >>>>>> > > >>>>>> > > >>>> --------------------------------------------------------------------- > > >>>> To unsubscribe, e-mail: [EMAIL PROTECTED] > > >>>> For additional commands, e-mail: [EMAIL PROTECTED] > > >>>> > > >>>> > > >>> --------------------------------------------------------------------- > > >>> To unsubscribe, e-mail: [EMAIL PROTECTED] > > >>> For additional commands, e-mail: [EMAIL PROTECTED] > > >>> > > >>> > > > > > > --------------------------------------------------------------------- > > > To unsubscribe, e-mail: [EMAIL PROTECTED] > > > For additional commands, e-mail: [EMAIL PROTECTED] > > > > > > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: [EMAIL PROTECTED] > > For additional commands, e-mail: [EMAIL PROTECTED] > > > > > --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
