From: Inger, M. <inger@Synygy.com> - 2003-03-24 15:58:53
|
OK. I think Pete Reilly is taking care of this. Thanks for the input. ---------------- Matthew Inger [in...@sy...] Software Developer Synygy, Inc 610-664-7433 x 7770 "Self Respect - The secure feeling that no one, as yet, is suspicious" - HL Mencken -----Original Message----- From: Dominique Devienne [mailto:DDe...@lg...] Sent: Monday, March 24, 2003 10:50 AM To: 'Inger, Matthew' Cc: ant...@li... Subject: RE: [Ant-contrib-developers] 1.5.1 Compatible version Thanks, Matthew. All I need really is an additional attribute called 'force' taking a boolean (you can call it however you want). When true, <delete>s all target files and (necessarily unconditionally) runs the <sequential>/<parallel> block. Out-of-date status of the source files can be skipped, since doesn't matter (but doing the delete of the target files implicitly triggers the tasks block...). When false (the default value), task behaves as it does now. The purpose of this new attribute is two fold: 1) Allow debugging of the tasks within the <sequential>/<parallel> block 2) Force execution the tasks within the <sequential>/<parallel> block in case timestamps cannot be trusted, when doing cvs updates back in time for example. Thanks, --DD <!-- ========================== --> <target name="generate" depends="init" description="Compile custom scanners/parsers (*.l/*.y)"> <property name="force" value="false" /> <outofdate force="${force}"> <sourcefiles> <pathelement location="pmodel/pmodel_gram.l" /> </sourcefiles> <targetfiles> <pathelement location="pmodel/pmodel_gram.l.cpp" /> </targetfiles> <sequential> <echo>Flexing pmodel/pmodel_gram.l...</echo> <exec executable="${tools.dir}/bin/flex.exe">... </exec> </sequential> </outofdate> <outofdate force="${force}"> <sourcefiles> <pathelement location="pmodel/pmodel_gram.y" /> </sourcefiles> <targetfiles> <pathelement location="pmodel/pmodel_gram.y.cpp" /> <pathelement location="pmodel/pmodel_gram.y.h" /> </targetfiles> <sequential> <echo>Bisoning pmodel/pmodel_gram.y...</echo> <exec executable="${tools.dir}/bin/bison.exe">... </exec> </sequential> </outofdate> <target> -----Original Message----- From: Inger, Matthew [mailto:inger@Synygy.com] Sent: Monday, March 24, 2003 7:24 AM To: 'Dominique Devienne' Subject: RE: [Ant-contrib-developers] 1.5.1 Compatible version Ok. Given me a concreete example of the syntax you're expecting, and the expected results, and i'll try to get it implemented when i have a chance. ---------------- Matthew Inger [in...@sy...] Software Developer Synygy, Inc 610-664-7433 x 7770 "Self Respect - The secure feeling that no one, as yet, is suspicious" - HL Mencken -----Original Message----- From: Dominique Devienne [mailto:DDe...@lg...] Sent: Friday, March 21, 2003 6:12 PM To: 'Inger, Matthew'; Dominique Devienne; 'peter reilly' Cc: 'ant...@li...' Subject: RE: [Ant-contrib-developers] 1.5.1 Compatible version I'm not sure you understood me correctly... I want to force the regeneration of the target files, even though they are currently up-to-date (timestamp-wise). Putting the delete inside <outofdate <sequential>> is of no value to me in this case, since will never get there. Right now, as I said earlier, to force this re-generation, I have to use a <delete> that duplicate the target file information into another target. I do understand your point about intermediate files being generated as well (and I have some too), but I don't really care about these. Actually, I believe <outofdate> should have an optional attribute to delete targetfiles before running the process when something out of date, as a convenience (avoids having to pass a outputtargetfiles property to use inside sequential with a delete!!!). Could even have a third <intermediatefiles> Path to take care of this situation too. And BTW, using an external fileset doesn't work. The delete would work all right, since if the file is not there in the file-system, no need to delete it, *but* on the other hand you want to no if a target file in there in <outodate>!!! You thus can't use a <fileset>, since the latter doesn't know about files not there (and it's slower since has to scan the directory...). Since delete doesn't take a path or filelist (whose syntax I hate!!!), one's stuck with duplicating the target file list information in two places... NOT GOOD! So I persist in wanting a force attribute ;-) --DD -----Original Message----- From: Inger, Matthew [mailto:inger@Synygy.com] Sent: Friday, March 21, 2003 4:59 PM To: 'Dominique Devienne'; 'peter reilly' Cc: 'ant...@li...' Subject: RE: [Ant-contrib-developers] 1.5.1 Compatible version This is probably more appropriate: <outofdate .... > <sequential> <!-- do your clean here --> </sequential> </outofdate> The problem is that we don't necessarily want to assume what needs to be done in order to clean up your targets. A lot of times, there are intermediate files (such as when transforming IDL into Java, or some other code generators). So deleting the target files is not necessarily an option. I'd rather see people clean their own files, so it's done correctly according to their application. And you don't need to duplicate the target files, if you are defining your filesets with id's outside of the outofdate task. <fileset id="abc" ...> ... </fileset> <outofdate ...> <targetfiles> <fileset refid="abc" /> </targetfiles> ... <sequential> <delete> <fileset refid="abc" /> </delete> </sequential> </outofdate> ---------------- Matthew Inger [in...@sy...] Software Developer Synygy, Inc 610-664-7433 x 7770 "Self Respect - The secure feeling that no one, as yet, is suspicious" - HL Mencken -----Original Message----- From: Dominique Devienne [ mailto:DDe...@lg... <mailto:DDe...@lg...> ] Sent: Friday, March 21, 2003 5:19 PM To: Dominique Devienne; 'peter reilly' Cc: 'ant...@li...' Subject: RE: [Ant-contrib-developers] 1.5.1 Compatible version Sorry, meant to send my previous post to the list (I'm so used to posting to Jakarta lists ;-) Another feature I'd like <outofdate> to have is a 'force' attribute that deletes all targetfiles, and runs the <sequential>/<parallel> block. I keep having to write custom 'clean' targets that duplicate the target files to feed to <delete>... And I might even do it myself if no one does it in the next year or so ;-) Thanks for <outofdate>!!! I love it. --DD -----Original Message----- From: Dominique Devienne Sent: Friday, March 21, 2003 2:06 PM To: 'peter reilly' Subject: RE: [Ant-contrib-developers] 1.5.1 Compatible version Peter, <outofdate> works great for me, but seems to systematically outputs the source files it checks apparently, whether they're out of date or not. I like my targets to be silent when they don't do anything, and Just to get the tasks output themselves (within the nested <sequential>/<parallel>)... Most Ant task behave that way, and some have an additional extra attribute for more verbose output (without resorting to -verbose). Would it be possible to have this incorporated? Thanks, --DD -----Original Message----- From: peter reilly [ mailto:pet...@co... <mailto:pet...@co...> ] Sent: Friday, March 21, 2003 12:38 PM To: ant...@li... Subject: Re: [Ant-contrib-developers] 1.5.1 Compatible version It is (seems to be) compatible with Ant 1.5.2. Peter On Friday 21 March 2003 17:42, Dominique Devienne wrote: > I see ant-contrib's HEAD is not compatible with Ant 1.5.1... > > Any chance of creating a branch that would be 1.5.x compatible? > If not, what would be the date to use in the update to get back > to a 1.5.x compatible code base? I just wanted to get <outofdate> :( > > Thanks, --DD > > P:\net_sf\ant-contrib>C:\pro\ant1.5.1\bin\ant jar > Buildfile: build.xml > > build_dirs: > > build_contrib_jar: > [javac] Compiling 8 source files to P:\net_sf\ant-contrib\classes > [javac] > P:\net_sf\ant-contrib\src\net\sf\antcontrib\logic\ForEach.java:312: > handleOutput(java.la > ng.String) has protected access in org.apache.tools.ant.taskdefs.CallTarget > [javac] currentTarget.handleOutput(line); > [javac] ^ > [javac] > P:\net_sf\ant-contrib\src\net\sf\antcontrib\logic\ForEach.java:326: > handleErrorOutput(ja > va.lang.String) has protected access in > org.apache.tools.ant.taskdefs.CallTarget > [javac] currentTarget.handleErrorOutput(line); > [javac] ^ > [javac] 2 errors > > BUILD FAILED > file:P:/net_sf/ant-contrib/build.xml:133: Compile failed; see the compiler > error output for details. > > > Total time: 3 seconds > P:\net_sf\ant-contrib> > > > ------------------------------------------------------- > This SF.net email is sponsored by:Crypto Challenge is now open! > Get cracking and register here for some mind boggling fun and > the chance of winning an Apple iPod: > http://ads.sourceforge.net/cgi-bin/redirect.pl?thaw0031en <http://ads.sourceforge.net/cgi-bin/redirect.pl?thaw0031en> > _______________________________________________ > Ant-contrib-developers mailing list > Ant...@li... > https://lists.sourceforge.net/lists/listinfo/ant-contrib-developers <https://lists.sourceforge.net/lists/listinfo/ant-contrib-developers> ------------------------------------------------------- This SF.net email is sponsored by:Crypto Challenge is now open! Get cracking and register here for some mind boggling fun and the chance of winning an Apple iPod: http://ads.sourceforge.net/cgi-bin/redirect.pl?thaw0031en <http://ads.sourceforge.net/cgi-bin/redirect.pl?thaw0031en> _______________________________________________ Ant-contrib-developers mailing list Ant...@li... https://lists.sourceforge.net/lists/listinfo/ant-contrib-developers <https://lists.sourceforge.net/lists/listinfo/ant-contrib-developers> ------------------------------------------------------- This SF.net email is sponsored by:Crypto Challenge is now open! Get cracking and register here for some mind boggling fun and the chance of winning an Apple iPod: http://ads.sourceforge.net/cgi-bin/redirect.pl?thaw0031en <http://ads.sourceforge.net/cgi-bin/redirect.pl?thaw0031en> _______________________________________________ Ant-contrib-developers mailing list Ant...@li... https://lists.sourceforge.net/lists/listinfo/ant-contrib-developers <https://lists.sourceforge.net/lists/listinfo/ant-contrib-developers> |