Return-Path: 
 <user-return-37999-apmail-ant-user-archive=ant.apache.org@ant.apache.org>
Delivered-To: apmail-ant-user-archive@www.apache.org
Received: (qmail 90469 invoked from network); 21 Oct 2003 10:48:22 -0000
Received: from daedalus.apache.org (HELO mail.apache.org) (208.185.179.12)
  by minotaur-2.apache.org with SMTP; 21 Oct 2003 10:48:22 -0000
Received: (qmail 21794 invoked by uid 500); 21 Oct 2003 10:48:10 -0000
Delivered-To: apmail-ant-user-archive@ant.apache.org
Received: (qmail 21759 invoked by uid 500); 21 Oct 2003 10:48:10 -0000
Mailing-List: contact user-help@ant.apache.org; run by ezmlm
Precedence: bulk
List-Unsubscribe: <mailto:user-unsubscribe@ant.apache.org>
List-Subscribe: <mailto:user-subscribe@ant.apache.org>
List-Help: <mailto:user-help@ant.apache.org>
List-Post: <mailto:user@ant.apache.org>
List-Id: "Ant Users List" <user.ant.apache.org>
Reply-To: "Ant Users List" <user@ant.apache.org>
Delivered-To: mailing list user@ant.apache.org
Received: (qmail 21724 invoked from network); 21 Oct 2003 10:48:09 -0000
Received: from unknown (HELO corvil.com) (213.94.219.177)
  by daedalus.apache.org with SMTP; 21 Oct 2003 10:48:09 -0000
Received: from preilly.local.corvil.com (preilly.local.corvil.com
 [172.18.1.173])
	by corvil.com (8.12.9/8.12.5) with ESMTP id h9LAm8d8018840
	for <user@ant.apache.org>; Tue, 21 Oct 2003 11:48:08 +0100 (IST)
	(envelope-from peter.reilly@corvil.com)
From: peter reilly <peter.reilly@corvil.com>
Organization: corvil
To: Ant Users List <user@ant.apache.org>
Subject: Re: MacroDef: use attribute inside another attributes default cla
  use
Date: Tue, 21 Oct 2003 11:48:30 +0100
User-Agent: KMail/1.5
References: <879A5AD5DD0ED511891F0003473A9B5608FF71EC@Z011004>
In-Reply-To: <879A5AD5DD0ED511891F0003473A9B5608FF71EC@Z011004>
MIME-Version: 1.0
Content-Type: text/plain;
  charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
Content-Disposition: inline
Message-Id: <200310211148.30319.peter.reilly@corvil.com>
X-Spam-Rating: daedalus.apache.org 1.6.2 0/1000/N
X-Spam-Rating: minotaur-2.apache.org 1.6.2 0/1000/N

On Tuesday 21 October 2003 11:35, Jan.Materne@rzf.fin-nrw.de wrote:
> I havent found the patch.
> Why you haven=B4t committed that to cvs? Just in test phase?

It makes changes to the behaviour of macrodef, so it cannot
be done in ant 1.7 without being done in ant 1.6.
The changes are a bit late in the 1.6 cycle, so I wanted some
feedback before committing them.

The patch is with the bugzilla report:
http://issues.apache.org/bugzilla/show_bug.cgi?id=3D23942

>
> Your second suggestion sounds good. But I would do it more comfortable:
> <macrodef>
>     <attribute name=3D"foo" default=3D"bar"/>
> should create a
>     <property name=3D"foo" default=3D"bar"/>
>
> So the user doesn=B4t have to do it. It=B4s available as attribute, has d=
efault
> behaviour and supports property replacement.
>     <attribute name=3D"foo"/>
>     <attribute name=3D"bar" default=3D"${foo}"/>

This won't work as the default attribute gets processed by ant before
macrodef sees it.
for example:
<property name=3D"foo" value=3D"global foo"/>
<macrodef name=3D"macro">
   <attribute name=3D"foo"/>
   <attribute name=3D"bar" default=3D"${foo}"/>
   <sequential>
     <echo>bar is ${bar}</echo>
   </sequential>
</macrodef>
<macro foo=3D"macro's foo"/>

generates
foo:
bar is global foo

Peter
> would create the
>     <property name=3D"bar" value=3D"${foo}"/>
> and that will work.
>
>
> Jan
>
> > -----Original Message-----
> > From: peter reilly [mailto:peter.reilly@corvil.com]
> > Sent: Tuesday, October 21, 2003 12:19 PM
> > To: Ant Users List
> > Subject: Re: MacroDef: use attribute inside another attributes default
> > cla use
> >
> > On Tuesday 21 October 2003 11:09, Jan.Materne@rzf.fin-nrw.de wrote:
> > > Ok, but I want to have two attributes. One of them optional. So a
> > >     <dep root=3D"dist-lite" file=3D"my-dependencies.txt"/>
> > > should work.
> >
> > Ah.
> >
> > > Does <local> introduce a new attribute?
> >
> > No.
> >
> > But the following should work:
> >   <target name=3D"script">
> >     <macrodef name=3D"dep">
> >       <attribute name=3D"root"/>
> >       <attribute name=3D"file" default=3D""/>
> >       <sequential>
> >           <script language=3D"javascript">
> >             root =3D project.getProperty("root");
> >             filename =3D project.getProperty("file");
> >             if (filename.equals("")) {
> >                 filename =3D root + ".dep"
> >             }
> >             importClass(java.lang.System);
> >             System.out.println("root is " + root
> >                                + " and filename is " + filename);
> >           </script>
> >         </sequential>
> >     </macrodef>
> >     <dep root=3D"build.xml"/>
> >   </target>
> >
> > I could make an optional attribute to <attribute> allowing:
> >   <target name=3D"script">
> >     <macrodef name=3D"dep">
> >       <attribute name=3D"root"/>
> >       <attribute name=3D"file" optional=3D"yes"/>
> >       <sequential>
> >           <!-- set file if file not set -->
> >           <property name=3D"file" value=3D"${file}.dep"/>
> >           <script language=3D"javascript">
> >             root =3D project.getProperty("root");
> >             importClass(java.lang.System);
> >             System.out.println("root is " + root
> >                                + " and filename is " + filename);
> >           </script>
> >         </sequential>
> >     </macrodef>
> >     <dep root=3D"build.xml"/>
> >   </target>
> >
> > > Jan
> > >
> > > > -----Original Message-----
> > > > From: peter reilly [mailto:peter.reilly@corvil.com]
> > > > Sent: Tuesday, October 21, 2003 12:07 PM
> > > > To: Ant Users List
> > > > Subject: Re: MacroDef: use attribute inside another
> >
> > attributes default
> >
> > > > clause
> > > >
> > > >
> > > > The default in the attribute element may not expand other
> >
> > attributes
> >
> > > > as this is expanded by ant before macrodef sees it.
> > > >
> > > > However the local patch I posted yesterday will fix this problem
> > > > in a different way.
> > > > With the patch, the macrodef attributes are real
> > > > properties - with a local scope, so
> > > > the following will work;
> > > >
> > > >   <target name=3D"script">
> > > >     <macrodef name=3D"dep">
> > > >         <attribute name=3D"root"/>
> > > >         <sequential>
> > > >           <local name=3D"file" value=3D"${root}.dep"/>
> > > >           <script language=3D"javascript">
> > > >             root =3D project.getProperty("root");
> > > >             filename =3D project.getProperty("file");
> > > >             importClass(java.lang.System);
> > > >             System.out.println("root is " + root
> > > >                                + " and filename is " + filename);
> > > >           </script>
> > > >         </sequential>
> > > >     </macrodef>
> > > >     <dep root=3D"build.xml"/>
> > > >   </target>
> > > >
> > > > With the output:
> > > > script:
> > > > root is build.xml and filename is build.xml.dep
> > > >
> > > > Peter
> > > >
> > > > On Tuesday 21 October 2003 10:43,
> >
> > Jan.Materne@rzf.fin-nrw.de wrote:
> > > > > I want to reuse one attribute for setting the default value
> > > >
> > > > of another one.
> > > >
> > > > > But that is not possible because the name is not resolved
> > > >
> > > > on that place.
> > > >
> > > > >     <macrodef name=3D"dep">
> > > > >         <attribute name=3D"root"/>
> > > > >         <attribute name=3D"file" default=3D"${root}.dep"/>
> > > > >         <sequential>
> > > > >         <script language=3D"javascript"> <![CDATA[
> > > > >             // attribute expansion from macrodef (script
> > > >
> > > > can=B4t reach the
> > > >
> > > > > values)
> > > > >             root     =3D "${root}";
> > > > >             filename =3D "${file}";
> > > > >
> > > > > Example:  <dep root=3D"foo"/>
> > > > > The "filename" is not set to "foo.dep" but to "${root}.dep".
> > > > >
> > > > > Workaround:
> > > > > Test that behaviour and reset filename:
> > > > >
> > > > >         <script language=3D"javascript"> <![CDATA[
> > > > >             // attribute expansion from macrodef (script
> > > >
> > > > can=B4t reach the
> > > >
> > > > > values)
> > > > >             root     =3D "${root}";
> > > > >             filename =3D "${file}";
> > > > >
> > > > >             // handle the problem that the ${root} is not
> > > >
> > > > replaced in the
> > > >
> > > > >             // attribute tag. Mask the ${root} string -
> > > >
> > > > should not be
> > > >
> > > > > replaced
> > > > >             // in the if-clause.
> > > > >             if (filename.equals("${ro"+"ot}.dep")) {
> > > > >                 filename =3D "${root}.dep";
> > > > >             }
> > > > >
> > > > >
> > > > > Is that a bug (which will be fixed - maybe until 1.6 final)
> > > >
> > > > or is it a
> > > >
> > > > > bigger problem?
> > > > >
> > > > >
> > > > > Jan
> >
> > ---------------------------------------------------------------------
> >
> > > > To unsubscribe, e-mail: user-unsubscribe@ant.apache.org
> > > > For additional commands, e-mail: user-help@ant.apache.org
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: user-unsubscribe@ant.apache.org
> > For additional commands, e-mail: user-help@ant.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: user-unsubscribe@ant.apache.org
For additional commands, e-mail: user-help@ant.apache.org