if they all contain the same properties, how about having only one property file
per OS and load it ?
seb.
[EMAIL PROTECTED] wrote:
> All,
>
> How can I improve the following code in ant?
>
> Thanks
>
> B.
>
> <target name="all" depends="dist" />
> - <target name="set">
> - <condition property="isLinux">
> <os name="Linux" />
> </condition>
> - <!-- needs code for testing other windows platforms.
> -->
> - <condition property="isWindows">
> <os name="Windows XP" />
> </condition>
> </target>
> - <target name="checkwindows" depends="set" if="isWindows">
> <echo message="test1: ${isWindows} it is windows" />
> <loadproperties srcfile="Windows.properties" />
> </target>
> - <target name="checklinux" depends="set" if="isLinux">
> <echo message="test2: ${isLinux} it is linux" />
> <loadproperties srcfile="Linux.properties" />
> </target>
> <target name="os_check" depends="checklinux, checkwindows" />
> - <target name="clean" depends="os_check">
>
> ----- Original Message -----
> From: "Conor MacNeill" <[EMAIL PROTECTED]>
> To: "Ant Users List" <[EMAIL PROTECTED]>
> Sent: Thursday, January 15, 2004 6:31 PM
> Subject: Re: os.name for windows return "Windows XP"?
>
> > On Fri, 16 Jan 2004 12:24 pm, [EMAIL PROTECTED] wrote:
> > > Hi,
> > >
> > > There must be someone who is dealing with the same issue. I am trying to
> > > take multiplatform builds with different ${os.name}.properties file. Linux
> > > is not a problem, Linux.properties. What about Windows? It becomes Windows
> > > XP.properties files.
> > >
> > > Can someone enlight me the better way to do this with one word .properties
> > > name?
> > >
> >
> > I've used this approach in the past
> >
> > <target name="set-conditions">
> > <condition property="linux">
> > <os name="linux"/>
> > </condition>
> >
> > <condition property="solaris">
> > <os name="SunOS"/>
> > </condition>
> >
> > <condition property="windows">
> > <os family="windows"/>
> > </condition>
> > </target>
> >
> > You could probably use a single property name and different value attributes
> > and load the properties that way. Depends if you want to use if attributes
> > later on.
> >
> > Conor
> >
> >
> >
> > ---------------------------------------------------------------------
> > 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]
