Re: os.name for windows return "Windows XP"?

[barry]

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]