DO NOT REPLY [Bug 32281] - fileset and dirset do not work with date selector

2004-12-02 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=32281





--- Additional Comments From [EMAIL PROTECTED]  2004-12-03 00:58 ---
Also merged to the 1.6 branch.

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



cvs commit: ant/docs/manual/CoreTypes selectors.html

2004-12-02 Thread mbenson
mbenson 2004/12/02 15:57:56

  Modified:docs/manual/CoreTypes Tag: ANT_16_BRANCH selectors.html
  Log:
  Merge
  PR: 32281
  Submitted by: Robert Anderson
  
  Revision  ChangesPath
  No   revision
  No   revision
  1.15.2.10 +7 -0  ant/docs/manual/CoreTypes/selectors.html
  
  Index: selectors.html
  ===
  RCS file: /home/cvs/ant/docs/manual/CoreTypes/selectors.html,v
  retrieving revision 1.15.2.9
  retrieving revision 1.15.2.10
  diff -u -r1.15.2.9 -r1.15.2.10
  --- selectors.html19 Nov 2004 09:10:03 -  1.15.2.9
  +++ selectors.html2 Dec 2004 23:57:56 -   1.15.2.10
  @@ -179,6 +179,13 @@
   
   No
 
  +  
  +checkdirs
  +
  +  Indicates whether or not to check dates on directories.
  +
  +No, defaults to false
  +  
   
   
   Here is an example of how to use the Date Selector:
  
  
  

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 32281] - fileset and dirset do not work with date selector

2004-12-02 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=32281


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED




--- Additional Comments From [EMAIL PROTECTED]  2004-12-03 00:50 ---
Patch committed to CVS HEAD; thanks.

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



cvs commit: ant/docs/manual/CoreTypes selectors.html

2004-12-02 Thread mbenson
mbenson 2004/12/02 15:50:03

  Modified:docs/manual/CoreTypes selectors.html
  Log:
  Document the checkdirs attribute of the date selector.
  PR: 32281
  Submitted by: Robert Anderson
  
  Revision  ChangesPath
  1.29  +7 -0  ant/docs/manual/CoreTypes/selectors.html
  
  Index: selectors.html
  ===
  RCS file: /home/cvs/ant/docs/manual/CoreTypes/selectors.html,v
  retrieving revision 1.28
  retrieving revision 1.29
  diff -u -r1.28 -r1.29
  --- selectors.html26 Nov 2004 14:39:26 -  1.28
  +++ selectors.html2 Dec 2004 23:50:03 -   1.29
  @@ -182,6 +182,13 @@
   
   No
 
  +  
  +checkdirs
  +
  +  Indicates whether or not to check dates on directories.
  +
  +No, defaults to false
  +  
   
   
   Here is an example of how to use the Date Selector:
  
  
  

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 32506] New: - ChecksumTest fails on Windows

2004-12-02 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=32506

   Summary: ChecksumTest fails on Windows
   Product: Ant
   Version: 1.7Alpha (nightly)
  Platform: PC
OS/Version: Windows 2000
Status: NEW
  Severity: normal
  Priority: P2
 Component: Core tasks
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]


I don't know if this relates directly to
http://marc.theaimsgroup.com/?l=ant-dev&m=109540457213078&w=2 although I suspect
it may.  Maybe I shouldn't give away how long it's been since I tried to run all
the tests, but this is failing on Windows, apparently due to the fact that the
generated checksum file is in Windows format and the test file is in Un*x
format.  Should we add a  to the xml file so that the
assertions pass?

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



RE: fileset and dirset do not work with date selector

2004-12-02 Thread Anderson, Rob (Global Trade)
Too late. I just attached a patch to the bug report.

-Rob A

> -Original Message-
> From: Matt Benson [mailto:[EMAIL PROTECTED]
> Sent: Thursday, December 02, 2004 3:07 PM
> To: Ant Developers List
> Subject: RE: fileset and dirset do not work with date selector
> 
> 
> Don't worry about the doc.  I'll hit it now that you
> have verified this works for you.
> 
> -Matt
> 
> --- "Anderson, Rob (Global Trade)"
> <[EMAIL PROTECTED]> wrote:
> 
> > Yes. It works as expected when setting the checkdirs
> > attribute to true. I will attempt to update the
> > documentation and submit a patch to the bug report I
> > mentioned earlier. Thanks for your help.
> > 
> > -Rob Anderson
> > 
> > > -Original Message-
> > > From: Matt Benson [mailto:[EMAIL PROTECTED]
> > > Sent: Thursday, December 02, 2004 2:28 PM
> > > To: Ant Developers List
> > > Subject: Re: fileset and dirset do not work with
> > date selector
> > > 
> > > 
> > > Okay, I take that back.  I was using 
> > to
> > > try and see what was selected.  But 
> > > relies on  which does not see directories
> > > selected from s (it will SOON).  Is
> > checkdirs
> > > working for you?
> > > 
> > > -Matt
> > > 
> > > --- Matt Benson <[EMAIL PROTECTED]> wrote:
> > > 
> > > > --- Matt Benson <[EMAIL PROTECTED]> wrote:
> > > > > Answer:  set the (undocumented) checkdirs
> > > > attribute
> > > > > to
> > > > > "true|on|yes".  ;)
> > > > 
> > > > Well, hang on, that doesn't want to work right
> > > > either...
> > > > 
> > > > > 
> > > > > -Matt
> > > > > 
> > > > > 
> > > > >   
> > > > > __ 
> > > > > Do you Yahoo!? 
> > > > > Take Yahoo! Mail with you! Get it on your
> > mobile
> > > > > phone. 
> > > > > http://mobile.yahoo.com/maildemo 
> > > > > 
> > > > >
> > > >
> > >
> >
> -
> > > > > To unsubscribe, e-mail:
> > > > > [EMAIL PROTECTED]
> > > > > For additional commands, e-mail:
> > > > > [EMAIL PROTECTED]
> > > > > 
> > > > > 
> > > > 
> > > > 
> > > > 
> > > > 
> > > > __ 
> > > > Do you Yahoo!? 
> > > > The all-new My Yahoo! - What will yours do?
> > > > http://my.yahoo.com 
> > > > 
> > > >
> > >
> >
> -
> > > > To unsubscribe, e-mail:
> > > > [EMAIL PROTECTED]
> > > > For additional commands, e-mail:
> > > > [EMAIL PROTECTED]
> > > > 
> > > > 
> > > 
> > > 
> > > 
> > >   
> > > __ 
> > > Do you Yahoo!? 
> > > The all-new My Yahoo! - Get yours free! 
> > > http://my.yahoo.com 
> > >  
> > > 
> > > 
> > >
> >
> -
> > > 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]
> > 
> > 
> 
> 
> 
>   
> __ 
> Do you Yahoo!? 
> Yahoo! Mail - now with 250MB free storage. Learn more.
> http://info.mail.yahoo.com/mail_250
> 
> -
> 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]



DO NOT REPLY [Bug 32281] - fileset and dirset do not work with date selector

2004-12-02 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=32281





--- Additional Comments From [EMAIL PROTECTED]  2004-12-03 00:15 ---
Created an attachment (id=13634)
 --> (http://issues.apache.org/bugzilla/attachment.cgi?id=13634&action=view)
update to the documentation

Update to the documentation.

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 29391] - javac task uses user.dir for temporary file instead of java.io.tmpdir

2004-12-02 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=29391





--- Additional Comments From [EMAIL PROTECTED]  2004-12-03 00:14 ---
I realize the patch is against HEAD and thus presupposes the
FileUtils.createTempFile from my original change... it doesn't matter to me, I
was just trying to get out of some work.  ;)

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 29391] - javac task uses user.dir for temporary file instead of java.io.tmpdir

2004-12-02 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=29391





--- Additional Comments From [EMAIL PROTECTED]  2004-12-03 00:11 ---
The tempfile changes (bug #32501) were intended for HEAD only, to be on the safe
side, unless someone sees a need for them in ANT_16_BRANCH. (Is there some
specific published policy about what gets merged??) Anyway the patch in #32501
does not include this patch - it presupposes it. So this #29391 should be
considered independently.

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: path, dirs, filesets WAS fileset and dirset do not work with date selector

2004-12-02 Thread Matt Benson
--- Matt Benson <[EMAIL PROTECTED]> wrote:

> Okay, I take that back.  I was using 
> to
> try and see what was selected.  But 
> relies on  which does not see directories
> selected from s (it will SOON).  Is

[SNIP]

Followup:  I had an aneurysm and forgot that DirSet
and FileSet are nearly the same thing.  I therefore do
not see any need at this time for  to include
directories from s so I am going to drop
this.

-Matt



__ 
Do you Yahoo!? 
Read only the mail you want - Yahoo! Mail SpamGuard. 
http://promotions.yahoo.com/new_mail 

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



RE: fileset and dirset do not work with date selector

2004-12-02 Thread Matt Benson
Don't worry about the doc.  I'll hit it now that you
have verified this works for you.

-Matt

--- "Anderson, Rob (Global Trade)"
<[EMAIL PROTECTED]> wrote:

> Yes. It works as expected when setting the checkdirs
> attribute to true. I will attempt to update the
> documentation and submit a patch to the bug report I
> mentioned earlier. Thanks for your help.
> 
> -Rob Anderson
> 
> > -Original Message-
> > From: Matt Benson [mailto:[EMAIL PROTECTED]
> > Sent: Thursday, December 02, 2004 2:28 PM
> > To: Ant Developers List
> > Subject: Re: fileset and dirset do not work with
> date selector
> > 
> > 
> > Okay, I take that back.  I was using 
> to
> > try and see what was selected.  But 
> > relies on  which does not see directories
> > selected from s (it will SOON).  Is
> checkdirs
> > working for you?
> > 
> > -Matt
> > 
> > --- Matt Benson <[EMAIL PROTECTED]> wrote:
> > 
> > > --- Matt Benson <[EMAIL PROTECTED]> wrote:
> > > > Answer:  set the (undocumented) checkdirs
> > > attribute
> > > > to
> > > > "true|on|yes".  ;)
> > > 
> > > Well, hang on, that doesn't want to work right
> > > either...
> > > 
> > > > 
> > > > -Matt
> > > > 
> > > > 
> > > > 
> > > > __ 
> > > > Do you Yahoo!? 
> > > > Take Yahoo! Mail with you! Get it on your
> mobile
> > > > phone. 
> > > > http://mobile.yahoo.com/maildemo 
> > > > 
> > > >
> > >
> >
>
-
> > > > To unsubscribe, e-mail:
> > > > [EMAIL PROTECTED]
> > > > For additional commands, e-mail:
> > > > [EMAIL PROTECTED]
> > > > 
> > > > 
> > > 
> > > 
> > > 
> > >   
> > > __ 
> > > Do you Yahoo!? 
> > > The all-new My Yahoo! - What will yours do?
> > > http://my.yahoo.com 
> > > 
> > >
> >
>
-
> > > To unsubscribe, e-mail:
> > > [EMAIL PROTECTED]
> > > For additional commands, e-mail:
> > > [EMAIL PROTECTED]
> > > 
> > > 
> > 
> > 
> > 
> > 
> > __ 
> > Do you Yahoo!? 
> > The all-new My Yahoo! - Get yours free! 
> > http://my.yahoo.com 
> >  
> > 
> > 
> >
>
-
> > 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]
> 
> 




__ 
Do you Yahoo!? 
Yahoo! Mail - now with 250MB free storage. Learn more.
http://info.mail.yahoo.com/mail_250

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 32281] - fileset and dirset do not work with date selector

2004-12-02 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=32281





--- Additional Comments From [EMAIL PROTECTED]  2004-12-02 23:58 ---
Setting the checkdirs attribute to true fixes the issue (thanks Matt Benson). 
This attribute is undocumented. I will update the documentation and attach a 
patch to this bug report.

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



RE: fileset and dirset do not work with date selector

2004-12-02 Thread Anderson, Rob (Global Trade)
Yes. It works as expected when setting the checkdirs attribute to true. I will 
attempt to update the documentation and submit a patch to the bug report I 
mentioned earlier. Thanks for your help.

-Rob Anderson

> -Original Message-
> From: Matt Benson [mailto:[EMAIL PROTECTED]
> Sent: Thursday, December 02, 2004 2:28 PM
> To: Ant Developers List
> Subject: Re: fileset and dirset do not work with date selector
> 
> 
> Okay, I take that back.  I was using  to
> try and see what was selected.  But 
> relies on  which does not see directories
> selected from s (it will SOON).  Is checkdirs
> working for you?
> 
> -Matt
> 
> --- Matt Benson <[EMAIL PROTECTED]> wrote:
> 
> > --- Matt Benson <[EMAIL PROTECTED]> wrote:
> > > Answer:  set the (undocumented) checkdirs
> > attribute
> > > to
> > > "true|on|yes".  ;)
> > 
> > Well, hang on, that doesn't want to work right
> > either...
> > 
> > > 
> > > -Matt
> > > 
> > > 
> > >   
> > > __ 
> > > Do you Yahoo!? 
> > > Take Yahoo! Mail with you! Get it on your mobile
> > > phone. 
> > > http://mobile.yahoo.com/maildemo 
> > > 
> > >
> >
> -
> > > To unsubscribe, e-mail:
> > > [EMAIL PROTECTED]
> > > For additional commands, e-mail:
> > > [EMAIL PROTECTED]
> > > 
> > > 
> > 
> > 
> > 
> > 
> > __ 
> > Do you Yahoo!? 
> > The all-new My Yahoo! - What will yours do?
> > http://my.yahoo.com 
> > 
> >
> -
> > To unsubscribe, e-mail:
> > [EMAIL PROTECTED]
> > For additional commands, e-mail:
> > [EMAIL PROTECTED]
> > 
> > 
> 
> 
> 
>   
> __ 
> Do you Yahoo!? 
> The all-new My Yahoo! - Get yours free! 
> http://my.yahoo.com 
>  
> 
> 
> -
> 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]



DO NOT REPLY [Bug 29391] - javac task uses user.dir for temporary file instead of java.io.tmpdir

2004-12-02 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=29391





--- Additional Comments From [EMAIL PROTECTED]  2004-12-02 23:30 ---
Jesse:  I can merge this from HEAD for 1.6.3 as I have heard of no issues but if
you make your tempfile changes from your other bugrep the point will be moot. 
Incidentally I am +0 to those changes...

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: fileset and dirset do not work with date selector

2004-12-02 Thread Matt Benson
Okay, I take that back.  I was using  to
try and see what was selected.  But 
relies on  which does not see directories
selected from s (it will SOON).  Is checkdirs
working for you?

-Matt

--- Matt Benson <[EMAIL PROTECTED]> wrote:

> --- Matt Benson <[EMAIL PROTECTED]> wrote:
> > Answer:  set the (undocumented) checkdirs
> attribute
> > to
> > "true|on|yes".  ;)
> 
> Well, hang on, that doesn't want to work right
> either...
> 
> > 
> > -Matt
> > 
> > 
> > 
> > __ 
> > Do you Yahoo!? 
> > Take Yahoo! Mail with you! Get it on your mobile
> > phone. 
> > http://mobile.yahoo.com/maildemo 
> > 
> >
>
-
> > To unsubscribe, e-mail:
> > [EMAIL PROTECTED]
> > For additional commands, e-mail:
> > [EMAIL PROTECTED]
> > 
> > 
> 
> 
> 
>   
> __ 
> Do you Yahoo!? 
> The all-new My Yahoo! - What will yours do?
> http://my.yahoo.com 
> 
>
-
> To unsubscribe, e-mail:
> [EMAIL PROTECTED]
> For additional commands, e-mail:
> [EMAIL PROTECTED]
> 
> 




__ 
Do you Yahoo!? 
The all-new My Yahoo! - Get yours free! 
http://my.yahoo.com 
 


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 32503] - [PATCH] Include RCS versioning files in default excludes

2004-12-02 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=32503





--- Additional Comments From [EMAIL PROTECTED]  2004-12-02 23:15 ---
Created an attachment (id=13631)
 --> (http://issues.apache.org/bugzilla/attachment.cgi?id=13631&action=view)
Possible patch


-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 32503] New: - [PATCH] Include RCS versioning files in default excludes

2004-12-02 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=32503

   Summary: [PATCH] Include RCS versioning files in default excludes
   Product: Ant
   Version: 1.7Alpha (nightly)
  Platform: All
OS/Version: All
Status: NEW
  Severity: minor
  Priority: P4
 Component: Core
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]


I don't know if DirectoryScanner.DEFAULTEXCLUDES is even open for changes at
this point, but if so, here's a suggested addition for RCS. Of course users who
are using RCS can add this for themselves using ; this patch
would just save them the trouble. Unlikely, but possible, that someone is using
one of these file name patterns for a different reason. +0 from me.

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



cvs commit: ant/src/main/org/apache/tools/ant/taskdefs ImportTask.java

2004-12-02 Thread jglick
jglick  2004/12/02 14:06:16

  Modified:src/main/org/apache/tools/ant/taskdefs ImportTask.java
  Log:
  Trivial Javadoc formatting edits.
  
  Revision  ChangesPath
  1.29  +8 -9  
ant/src/main/org/apache/tools/ant/taskdefs/ImportTask.java
  
  Index: ImportTask.java
  ===
  RCS file: 
/home/cvs/ant/src/main/org/apache/tools/ant/taskdefs/ImportTask.java,v
  retrieving revision 1.28
  retrieving revision 1.29
  diff -u -r1.28 -r1.29
  --- ImportTask.java   28 Apr 2004 09:01:08 -  1.28
  +++ ImportTask.java   2 Dec 2004 22:06:15 -   1.29
  @@ -29,27 +29,26 @@
   import java.util.Vector;
   
   /**
  - * 
* Task to import another build file into the current project.
* 
* It must be 'top level'. On execution it will read another Ant file
* into the same Project.
  + * 
* 
* Important: we have not finalized how relative file references
  - * will be resolved in deep/complex build hierarchies -such as what happens
  + * will be resolved in deep/complex build hierarchies - such as what happens
* when an imported file imports another file. Use absolute references for
* enhanced build file stability, especially in the imported files.
  - *
  - * Examples
  + * 
  + * Examples:
* 
  - * 
  + * 
* 
  - * Import targets from a file in a parent directory.
  - *
  + * Import targets from a file in a parent directory.
* 
  - * 
  + * 
* 
  - * Import the project defined by the property deploy-platform
  + * Import the project defined by the property 
deploy-platform.
*
* @since Ant1.6
* @ant.task category="control"
  
  
  

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



cvs commit: ant/src/main/org/apache/tools/ant/taskdefs TempFile.java

2004-12-02 Thread jglick
jglick  2004/12/02 13:59:13

  Modified:src/main/org/apache/tools/ant/taskdefs TempFile.java
  Log:
  Small Javadoc change.
  CVS: --
  CVS: PR:
  CVS:   If this change addresses a PR in the problem report tracking
  CVS:   database, then enter the PR number(s) here.
  CVS: Obtained from:
  CVS:   If this change has been taken from another system, such as NCSA,
  CVS:   then name the system in this line, otherwise delete it.
  CVS: Submitted by:
  CVS:   If this code has been contributed to Apache by someone else; i.e.,
  CVS:   they sent us a patch or a new module, then include their name/email
  CVS:   address here. If this is your work then delete this line.
  CVS: Reviewed by:
  CVS:   If we are doing pre-commit code reviews and someone else has
  CVS:   reviewed your changes, include their name(s) here.
  CVS:   If you have not had it reviewed then delete this line.
  
  Revision  ChangesPath
  1.13  +1 -2  ant/src/main/org/apache/tools/ant/taskdefs/TempFile.java
  
  Index: TempFile.java
  ===
  RCS file: /home/cvs/ant/src/main/org/apache/tools/ant/taskdefs/TempFile.java,v
  retrieving revision 1.12
  retrieving revision 1.13
  diff -u -r1.12 -r1.13
  --- TempFile.java 9 Mar 2004 16:48:06 -   1.12
  +++ TempFile.java 2 Dec 2004 21:59:12 -   1.13
  @@ -23,8 +23,7 @@
   
   /**
*  This task sets a property to  the name of a temporary file.
  - *  Unlike the Java1.2 method to create a temporary file, this task
  - *  does work on Java1.1. Also, it does not actually create the
  + *  Unlike [EMAIL PROTECTED] File#createTempFile}, this task does not 
actually create the
*  temporary file, but it does guarantee that the file did not
*  exist when the task was executed.
* 
  
  
  

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 32501] - [PATCH] Use File.createTempFile wherever appropriate

2004-12-02 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=32501





--- Additional Comments From [EMAIL PROTECTED]  2004-12-02 22:56 ---
Created an attachment (id=13629)
 --> (http://issues.apache.org/bugzilla/attachment.cgi?id=13629&action=view)
Proposed patch


-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 32501] New: - [PATCH] Use File.createTempFile wherever appropriate

2004-12-02 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=32501

   Summary: [PATCH] Use File.createTempFile wherever appropriate
   Product: Ant
   Version: 1.7Alpha (nightly)
  Platform: All
OS/Version: All
Status: NEW
  Severity: minor
  Priority: P3
 Component: Core
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
CC: [EMAIL PROTECTED]


Partial deprecation of FileUtils.createTempFile, in the spirit of using standard
Java APIs whenever they suffice, for both readability and maintainability.

Also using the default ${java.io.tmpdir} rather than getProject().getBaseDir()
in AbstractMetamataTask (no documented change) and JUnitTask (change of
documented default), both to avoid leaving crud in a project directory in case
the Ant process is killed, and to avoid problems in case the project directory
is read-only.

Also a change to use a particular dir (generally dir of target file) rather than
${java.io.tmpdir} in ReplaceRegExp and FixCRLF, so FileUtils.rename need not
cross drive/mount boundaries (may be more efficient). Zip and Replace were
already doing this.

I have tried to leave untouched any usages of FileUtils.createTempFile which
actually depended on the file *not* existing after the call, at least according
to my understanding of the surrounding code. This seems to be relatively 
uncommon.

Advance code reviews welcome.

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 29391] - javac task uses user.dir for temporary file instead of java.io.tmpdir

2004-12-02 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=29391


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]
   Target Milestone|--- |1.7




--- Additional Comments From [EMAIL PROTECTED]  2004-12-02 22:25 ---
Was responsible for a major bug affecting NetBeans users:

http://www.netbeans.org/issues/show_bug.cgi?id=51482

Workaround was same as Matt's, tempdir="${java.io.tmpdir}". Would recommend for
a merge into 1.6.3, if no unpleasant side effects have surfaced.

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: fileset and dirset do not work with date selector

2004-12-02 Thread Matt Benson
--- Matt Benson <[EMAIL PROTECTED]> wrote:
> Answer:  set the (undocumented) checkdirs attribute
> to
> "true|on|yes".  ;)

Well, hang on, that doesn't want to work right
either...

> 
> -Matt
> 
> 
>   
> __ 
> Do you Yahoo!? 
> Take Yahoo! Mail with you! Get it on your mobile
> phone. 
> http://mobile.yahoo.com/maildemo 
> 
>
-
> To unsubscribe, e-mail:
> [EMAIL PROTECTED]
> For additional commands, e-mail:
> [EMAIL PROTECTED]
> 
> 




__ 
Do you Yahoo!? 
The all-new My Yahoo! - What will yours do?
http://my.yahoo.com 

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: fileset and dirset do not work with date selector

2004-12-02 Thread Matt Benson
--- "Anderson, Rob (Global Trade)"
<[EMAIL PROTECTED]> wrote:

> I have a question about using a date selector with
> fileset and dirset. My problem is documented in bug
> # 32281...
[SNIP]
> By default includeDirs is set to false (on line 40).
[SNIP]
> My question is: What must I change in my build.xml
> to override the default that is set on line 40. I am
> using the latest attachement to the bug mentioned
> above as my build file.

Answer:  set the (undocumented) checkdirs attribute to
"true|on|yes".  ;)

-Matt



__ 
Do you Yahoo!? 
Take Yahoo! Mail with you! Get it on your mobile phone. 
http://mobile.yahoo.com/maildemo 

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 5003] - exec task does not return after executed command finished on Windows only

2004-12-02 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=5003





--- Additional Comments From [EMAIL PROTECTED]  2004-12-02 21:17 ---
Notepad is special; it is a GUI app. If you look at how windows execs guis, it
does *odd* things, things that only make sense from a historical perspective.


try on a command line app instead of notepad.

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



fileset and dirset do not work with date selector

2004-12-02 Thread Anderson, Rob (Global Trade)
I have a question about using a date selector with fileset and dirset. My 
problem is documented in bug # 32281...

http://issues.apache.org/bugzilla/show_bug.cgi?id=32281

I have been using and interactive debugger to try to find the problem in the 
code and I have come across what appears to be my problem. In the isSelected() 
method of DateSelector.java I see the following starting on line 246...

if (file.isDirectory() && (!includeDirs)) {
return true;
}

By default includeDirs is set to false (on line 40). There is a method to 
override the default beginning on line 126...

public void setCheckdirs(boolean includeDirs) {
this.includeDirs = includeDirs;
}

My question is: What must I change in my build.xml to override the default that 
is set on line 40. I am using the latest attachement to the bug mentioned above 
as my build file.

Thanks,


_
Robert Anderson  Sr. System Engineer  Nike - Global Trade IT  (503) 532-6803  d


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 28505] - [PATCH] File paths to imported build files should not be canonicalized

2004-12-02 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=28505





--- Additional Comments From [EMAIL PROTECTED]  2004-12-02 20:46 ---
Re. : cf. bug #30438, fixed for 1.6.3.

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 30631] - Need a way to get process-id for task

2004-12-02 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=30631


[EMAIL PROTECTED] changed:

   What|Removed |Added

 OS/Version||All




--- Additional Comments From [EMAIL PROTECTED]  2004-12-02 20:01 ---
By the way: the 'jps' command included in JDK 5.0 can be valuable for finding
the pid of a running Java process (or whatever the unique ID is on your OS).
Take a look at some of the JDK 5.0 monitoring & management stuff. Still no way
to get a PID back from a Process created by Runtime.exec however (unless you are
willing to use reflection and bypass the Java APIs).

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 5003] - exec task does not return after executed command finished on Windows only

2004-12-02 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=5003





--- Additional Comments From [EMAIL PROTECTED]  2004-12-02 19:52 ---
Hi Peter,
I do follow, the exec method starts the process and returns. Did you tried that 
?

BTW: part of the Runtime.exec javadoc:
 "Executes the specified string command in a separate process."

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



cvs commit: ant/src/main/org/apache/tools/ant/taskdefs WhichResource.java

2004-12-02 Thread peterreilly
peterreilly2004/12/02 10:12:40

  Modified:src/main/org/apache/tools/ant/taskdefs Tag: ANT_16_BRANCH
WhichResource.java
  Log:
  checkstyle
  
  Revision  ChangesPath
  No   revision
  No   revision
  1.5.2.7   +1 -1  
ant/src/main/org/apache/tools/ant/taskdefs/WhichResource.java
  
  Index: WhichResource.java
  ===
  RCS file: 
/home/cvs/ant/src/main/org/apache/tools/ant/taskdefs/WhichResource.java,v
  retrieving revision 1.5.2.6
  retrieving revision 1.5.2.7
  diff -u -r1.5.2.6 -r1.5.2.7
  --- WhichResource.java9 Mar 2004 17:01:34 -   1.5.2.6
  +++ WhichResource.java2 Dec 2004 18:12:40 -   1.5.2.7
  @@ -110,7 +110,7 @@
   
   /**
* execute it
  - * @throws BuildException
  + * @throws BuildException on error
*/
   public void execute() throws BuildException {
   validate();
  
  
  

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



cvs commit: ant/src/main/org/apache/tools/ant/taskdefs WhichResource.java

2004-12-02 Thread peterreilly
peterreilly2004/12/02 10:12:23

  Modified:src/main/org/apache/tools/ant/taskdefs WhichResource.java
  Log:
  checkstyle
  
  Revision  ChangesPath
  1.13  +1 -1  
ant/src/main/org/apache/tools/ant/taskdefs/WhichResource.java
  
  Index: WhichResource.java
  ===
  RCS file: 
/home/cvs/ant/src/main/org/apache/tools/ant/taskdefs/WhichResource.java,v
  retrieving revision 1.12
  retrieving revision 1.13
  diff -u -r1.12 -r1.13
  --- WhichResource.java12 Nov 2004 15:14:59 -  1.12
  +++ WhichResource.java2 Dec 2004 18:12:23 -   1.13
  @@ -110,7 +110,7 @@
   
   /**
* execute it
  - * @throws BuildException
  + * @throws BuildException on error
*/
   public void execute() throws BuildException {
   validate();
  
  
  

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



cvs commit: ant/src/main/org/apache/tools/ant/types defaults.properties

2004-12-02 Thread peterreilly
peterreilly2004/12/02 10:07:44

  Modified:src/main/org/apache/tools/ant/types defaults.properties
  Log:
  opps
  
  Revision  ChangesPath
  1.32  +1 -0  
ant/src/main/org/apache/tools/ant/types/defaults.properties
  
  Index: defaults.properties
  ===
  RCS file: 
/home/cvs/ant/src/main/org/apache/tools/ant/types/defaults.properties,v
  retrieving revision 1.31
  retrieving revision 1.32
  diff -u -r1.31 -r1.32
  --- defaults.properties   22 Nov 2004 11:52:39 -  1.31
  +++ defaults.properties   2 Dec 2004 18:07:44 -   1.32
  @@ -35,5 +35,6 @@
   assertions=org.apache.tools.ant.types.Assertions
   concatfilter=org.apache.tools.ant.filters.ConcatFilter
   issigned=org.apache.tools.ant.taskdefs.condition.IsSigned
  +isfileselected=org.apache.tools.ant.taskdefs.condition.IsFileSelected
   ispingable=org.apache.tools.ant.taskdefs.optional.condition.IsPingable
   mavenrepository=org.apache.tools.ant.taskdefs.repository.MavenRepository
  
  
  

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



RE: final keyword WAS [Patch] style of UpToDate

2004-12-02 Thread Jose Alberto Fernandez
> From: Matt Benson [mailto:[EMAIL PROTECTED] 
> 
> --- Phil Weighill-Smith
> <[EMAIL PROTECTED]> wrote:
> 
> > I personally think the use of "final" for method
> > parameters is "a good
> > thing" - it reminds me of that great C++ feature
> > "const" (which was
> > actually far more sophisticated and useful, and
> > strangely is a Java
> > reserved keyword).
> 
> I read the sample chapter Kevin pointed out, and I
> understood the points made.  But if you look at the
> issue from a different perspective, the basic
> take-away message is that "final" should be used as a
> crutch.  The obvious response to that statement is
> then "what's wrong with that?"  Nothing, I suppose,
> but there is unit testing to help avoid the kinds of
> mistakes warned against in the chapter.  I suppose
> it's all a matter of taste.  The
> compiler-optimization-based conditional compilation
> was interesting, but personally I would rather be able
> to throw switches without touching java code.
> 

For more good and fashionable writing test cases can be.
There is no better way to remove the posibility of bug, than having
the compiler semantically verify what was written.

If people think otherwise, then maybe they should try writing all
your parameters as type Object and make the testcases check that you
are doing the right thing on every case. ;-)

The more the compiler can check for you the better.

Jose Alberto


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 32461] - The "Xslt/Style" task should allow inlined xslt code

2004-12-02 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=32461





--- Additional Comments From [EMAIL PROTECTED]  2004-12-02 18:54 ---
Since this is about the stylesheet to use, i thought i might pop in. Can this
task use the stylsheet referenced by the XML file to transform (in it's
xml-stylesheet PI)? This would be real usefull when dealing with XML files that
use different XSLT stylsheets. I have implemented such a task so if you'd like i
could work out a patch to integrate this functionality where it belongs IMHO.


-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 28261] - java.lang.OutOfMemoryError

2004-12-02 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=28261





--- Additional Comments From [EMAIL PROTECTED]  2004-12-02 18:49 ---
ping, is this an issue with ant?

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 30710] - taskdef: com.sun.tools.ant.Task not found.

2004-12-02 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=30710





--- Additional Comments From [EMAIL PROTECTED]  2004-12-02 18:47 ---
Please provide the urls of the jars needed to reproduce
this problem and the xsd's files to complete the test

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 30557] - testRedirect1(org.apache.tools.ant.taskdefs.JavaTest) fails

2004-12-02 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=30557


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED




--- Additional Comments From [EMAIL PROTECTED]  2004-12-02 18:45 ---
Closing this, reopen if there is still a problem

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 27614] - Ant hangs when running build with junit.

2004-12-02 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=27614





--- Additional Comments From [EMAIL PROTECTED]  2004-12-02 18:43 ---
Can you provide an example build file that shows the problem?

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 5003] - exec task does not return after executed command finished on Windows only

2004-12-02 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=5003


[EMAIL PROTECTED] changed:

   What|Removed |Added

 OS/Version||All




--- Additional Comments From [EMAIL PROTECTED]  2004-12-02 18:09 ---
Hi Adam,
I do not follow your example with notepad.
The Runtime.getRuntime().exec("notepad") will not
return until the notepad process stops.



-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 22433] - Checking out files from cvs with a paticular version hangs thru ant

2004-12-02 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=22433





--- Additional Comments From [EMAIL PROTECTED]  2004-12-02 17:33 ---
We have the same issue here
When using the very command from the command line, the update is performed
normally without any prompt nor user interaction.
Dumping the stack trace at this moment makes me think that ant is kindly waiting
for the spawned process to finish. I'd tend to think it isn't ant fault ...

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: final keyword WAS [Patch] style of UpToDate

2004-12-02 Thread Matt Benson
--- Phil Weighill-Smith
<[EMAIL PROTECTED]> wrote:

> I personally think the use of "final" for method
> parameters is "a good
> thing" - it reminds me of that great C++ feature
> "const" (which was
> actually far more sophisticated and useful, and
> strangely is a Java
> reserved keyword).

I read the sample chapter Kevin pointed out, and I
understood the points made.  But if you look at the
issue from a different perspective, the basic
take-away message is that "final" should be used as a
crutch.  The obvious response to that statement is
then "what's wrong with that?"  Nothing, I suppose,
but there is unit testing to help avoid the kinds of
mistakes warned against in the chapter.  I suppose
it's all a matter of taste.  The
compiler-optimization-based conditional compilation
was interesting, but personally I would rather be able
to throw switches without touching java code.

$0.02
-Matt

> 
> Phil :n.




__ 
Do you Yahoo!? 
Meet the all-new My Yahoo! - Try it today! 
http://my.yahoo.com 
 


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 30631] - Need a way to get process-id for task

2004-12-02 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=30631


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2004-12-02 17:22 ---
*** Bug 32488 has been marked as a duplicate of this bug. ***

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 32488] - Improvement to 'spawn' attribute

2004-12-02 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=32488


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||DUPLICATE




--- Additional Comments From [EMAIL PROTECTED]  2004-12-02 17:22 ---
This cannot be done in java - see 30631

*** This bug has been marked as a duplicate of 30631 ***

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: [Patch] style of UpToDate

2004-12-02 Thread Phil Weighill-Smith
I personally think the use of "final" for method parameters is "a good
thing" - it reminds me of that great C++ feature "const" (which was
actually far more sophisticated and useful, and strangely is a Java
reserved keyword).

Just because people haven't previously adopted a practice doesn't mean
that it can't become best practice; this is a definite candidate for the
latter.

Phil :n.

On Thu, 2004-12-02 at 15:54, Peter Reilly wrote:

> Kevin Jackson wrote:
> 
> > Peter Reilly wrote:
> >
> >> Kevin Jackson wrote:
> >>
> >>> Just removed the C++ style private variables
> >>
> >>
> >>
> >> cool, but...
> >>  1) why place a "final" in the arguments
> >>  2) you should only use the "this." in the setter
> >
> >
> > 1) I was reading hardcore java - www.oreilly.com/catalog/*hardcore*jv/
> >
> > The arguments made in favour of declaring variables to methods as 
> > final far outweighed (in my mind), the hassle of typing a few extra 
> > characters.  The chapter on final being used in this (and a variety of 
> > other manners) is available as a pdf and I thoroughly recommend 
> > reading it, just to see what you think, but I couldn't disagree with 
> > his reasoning.
> 
> Mmmm.
> declaring method parameters as final may reduce a number of silly bugs, 
> however I cannot think of any
> bugs in the ant code that has been caused by setting method parameters. 
> It is not normal java style to use
> final method parameters and it
> is not normal ant coding style (although some of Magesh's code does do 
> this).**
> 
> >
> > 2) yeah, sorry, I got carried away I think - in other methods there 
> > should be no need to use this - doh!
> >
> > The main reason I did this was that I've been trying to contribute for 
> > the last few days and I haven't found anything that I felt I had the 
> > time to start on and this looked like a quick contribution,
> 
> An ongoing project is making the ant source code pass a checkstyle 
> audit. There is check.xml at the top level in the ant source code.
> The top offending files are the moment are:
> 
> taskdefs/Rmic.java
> taskdefs/Zip.java
> taskdefs/optional/vss/MSVSS.java
> taskdefs/optional/javacc/JavaCC.java
> taskdefs/optional/NetRexxC.java
> taskdefs/optional/junit/JUnitTestRunner.java
> types/Path.java
> taskdefs/repository/Library.java
> taskdefs/optional/jsp/JspC.java
> 
> Peter
> 
> 
> >
> > Kev
> >
> > -
> > 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]

-- 
Phil Weighill-Smith <[EMAIL PROTECTED]>
Volantis Systems


Re: [Patch] style of UpToDate

2004-12-02 Thread Peter Reilly
Kevin Jackson wrote:
Peter Reilly wrote:
Kevin Jackson wrote:
Just removed the C++ style private variables

cool, but...
 1) why place a "final" in the arguments
 2) you should only use the "this." in the setter

1) I was reading hardcore java - www.oreilly.com/catalog/*hardcore*jv/
The arguments made in favour of declaring variables to methods as 
final far outweighed (in my mind), the hassle of typing a few extra 
characters.  The chapter on final being used in this (and a variety of 
other manners) is available as a pdf and I thoroughly recommend 
reading it, just to see what you think, but I couldn't disagree with 
his reasoning.
Mmmm.
declaring method parameters as final may reduce a number of silly bugs, 
however I cannot think of any
bugs in the ant code that has been caused by setting method parameters. 
It is not normal java style to use
final method parameters and it
is not normal ant coding style (although some of Magesh's code does do 
this).**

2) yeah, sorry, I got carried away I think - in other methods there 
should be no need to use this - doh!

The main reason I did this was that I've been trying to contribute for 
the last few days and I haven't found anything that I felt I had the 
time to start on and this looked like a quick contribution,
An ongoing project is making the ant source code pass a checkstyle 
audit. There is check.xml at the top level in the ant source code.
The top offending files are the moment are:

taskdefs/Rmic.java
taskdefs/Zip.java
taskdefs/optional/vss/MSVSS.java
taskdefs/optional/javacc/JavaCC.java
taskdefs/optional/NetRexxC.java
taskdefs/optional/junit/JUnitTestRunner.java
types/Path.java
taskdefs/repository/Library.java
taskdefs/optional/jsp/JspC.java
Peter

Kev
-
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]


DO NOT REPLY [Bug 11242] - When will ANT support Viewpathing (VPATH)?

2004-12-02 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=11242





--- Additional Comments From [EMAIL PROTECTED]  2004-12-02 15:28 ---
(In reply to comment #1)
> If you've got some suggestions about how "viewpathing" could work, I'd be
> interested to hear them.

If filesets, etc, were to allow a "pathref=" or "path=", in stead of "dir=",
that would be a pretty good start.

The expected behavior would be similar to the SOURCEPATH behavior -- find the
first unique filename on the path.

To make this applicable in the maximum number of places, it were probably best
to have a special tag  a la . The difference being that 
never yields the same basename twice:


  
  


When you use a vpath instead of a path, of "SomeName.ext" is located in source/,
then the copy of SomeName.ext in othersource/ is not reported.

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: [GUMP@brutus]: Project ant-xdocs-proposal (in module ant) failed

2004-12-02 Thread Erik Hatcher
You should consider using something besides XDoclet when starting 
fresh.  XDoclet2 has been gaining momentum.  And QDox and Doxygen may 
be worth looking into as well.  XDoclet is a bear and requires of 
resources to run.  XDoclet2 is supposedly much more streamlined.

Sorry I've been absent from xdocs for a long while - lately I've been 
blissfully simple - staying away from EJB/Struts projects that got me 
into XDoclet in the first place.  I'm quite happy with just , 
, and  most of the time :)

Erik
On Dec 2, 2004, at 6:41 AM, Steve Loughran wrote:

On Thu, 02 Dec 2004 09:32:22 +0100, Stefan Bodewig <[EMAIL PROTECTED]>
wrote:
this is a bug in Gump which currently ignores the  setting
telling it to add more memory to the build.
I'm in the process of brushing up my minimal Python knowledge to
a-bit-more-than-minimal-but-still-nothing-to-talk-of so I can fix it.
Don't hold your breath 8-)
Stefan
I actually want to sit down and get this whole xdocs things working
properly. Wonderful that it is, it was written two+ years ago, and 
hasnt
been touched since. I use it for generating docs for external project
(Axis, smartfrog) and that really hurts.

What I'd like is
- fetch of needed components
-generate docbook content for feeding into db2pdf or FOP
-coverage of datatypes and nested elements
-better support for example code
-generate a quickref as well as the pages and a big PDF version
-usable by other projects
I'd probably start this in a new proposal subdir, as it would be fairly
different.
No time to start this this year, as I am off on a 3 week holiday, and
have no laptop. 3 weeks with no laptop!

-steve
-
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]


Re: [Patch] style of UpToDate

2004-12-02 Thread Kevin Jackson
Peter Reilly wrote:
Kevin Jackson wrote:
Just removed the C++ style private variables

cool, but...
 1) why place a "final" in the arguments
 2) you should only use the "this." in the setter
1) I was reading hardcore java - www.oreilly.com/catalog/*hardcore*jv/
The arguments made in favour of declaring variables to methods as final 
far outweighed (in my mind), the hassle of typing a few extra 
characters.  The chapter on final being used in this (and a variety of 
other manners) is available as a pdf and I thoroughly recommend reading 
it, just to see what you think, but I couldn't disagree with his reasoning.

2) yeah, sorry, I got carried away I think - in other methods there 
should be no need to use this - doh!

The main reason I did this was that I've been trying to contribute for 
the last few days and I haven't found anything that I felt I had the 
time to start on and this looked like a quick contribution,

Kev
-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


DO NOT REPLY [Bug 27814] - javadoc task does not encode baskslashes when generating file list by useexternalfile

2004-12-02 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=27814


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2004-12-02 13:10 ---
I found this issue while fixing NetBeans issue
. I can reproduce the
problem on WinXP too and playing directly with the javadoc.exe from JDK 1.4.2 or
from 1.5 I believe that it is JDK bug. I filed it as


-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: [Patch] style of UpToDate

2004-12-02 Thread Peter Reilly
Kevin Jackson wrote:
Just removed the C++ style private variables
cool, but...
 1) why place a "final" in the arguments
 2) you should only use the "this." in the setter
Why set dir to null here?  Is there a problem with the size of the 
object and the GC not collecting it in time?  if not it doesn't seem 
to be relevant
I do not know the code, but the "dir" is use in the sfs.restrict() call, 
so this may be the reason that it is set to null.

protected boolean scanDir(File srcDir, String[] files) {
   SourceFileScanner sfs = new SourceFileScanner(this);
   FileNameMapper mapper = null;
   File dir = srcDir;
   if (mapperElement == null) {
   MergingMapper mm = new MergingMapper();
   mm.setTo(this.targetFile.getAbsolutePath());
   mapper = mm;
   dir = null;
   } else {
   mapper = mapperElement.getImplementation();
   }
   return sfs.restrict(files, srcDir, dir, mapper).length == 0;
   }
Kev

Index: UpToDate.java
===
RCS file: 
/home/cvspublic/ant/src/main/org/apache/tools/ant/taskdefs/UpToDate.java,v
retrieving revision 1.36
diff -u -r1.36 UpToDate.java
--- UpToDate.java   9 Mar 2004 16:48:06 -   1.36
+++ UpToDate.java   2 Dec 2004 09:44:05 -
@@ -42,10 +42,10 @@
public class UpToDate extends Task implements Condition {
-private String _property;
-private String _value;
-private File _sourceFile;
-private File _targetFile;
+private String property;
+private String value;
+private File sourceFile;
+private File targetFile;
private Vector sourceFileSets = new Vector();
protected Mapper mapperElement = null;
@@ -56,8 +56,8 @@
 *
 * @param property the name of the property to set if Target is up-to-date.
 */
-public void setProperty(String property) {
-_property = property;
+public void setProperty(final String property) {
+this.property = property;
}
/**
@@ -66,15 +66,15 @@
 *
 * @param value the value to set the property to if Target is up-to-date
 */
-public void setValue(String value) {
-_value = value;
+public void setValue(final String value) {
+this.value = value;
}
/**
 * Returns the value, or "true" if a specific value wasn't provided.
 */
private String getValue() {
-return (_value != null) ? _value : "true";
+return (this.value != null) ? this.value : "true";
}
/**
@@ -83,8 +83,8 @@
 *
 * @param file the file we are checking against.
 */
-public void setTargetFile(File file) {
-_targetFile = file;
+public void setTargetFile(final File file) {
+this.targetFile = file;
}
/**
@@ -93,14 +93,14 @@
 *
 * @param file the file we are checking against the target file.
 */
-public void setSrcfile(File file) {
-_sourceFile = file;
+public void setSrcfile(final File file) {
+this.sourceFile = file;
}
/**
 * Nested  element.
 */
-public void addSrcfiles(FileSet fs) {
+public void addSrcfiles(final FileSet fs) {
sourceFileSets.addElement(fs);
}
@@ -121,32 +121,32 @@
 * see if the target(s) is/are up-to-date.
 */
public boolean eval() {
-if (sourceFileSets.size() == 0 && _sourceFile == null) {
+if (sourceFileSets.size() == 0 && this.sourceFile == null) {
throw new BuildException("At least one srcfile or a nested "
 + " element must be set.");
}
-if (sourceFileSets.size() > 0 && _sourceFile != null) {
+if (sourceFileSets.size() > 0 && this.sourceFile != null) {
throw new BuildException("Cannot specify both the srcfile "
 + "attribute and a nested  "
 + "element.");
}
-if (_targetFile == null && mapperElement == null) {
+if (this.targetFile == null && mapperElement == null) {
throw new BuildException("The targetfile attribute or a nested "
 + "mapper element must be set.");
}
// if the target file is not there, then it can't be up-to-date
-if (_targetFile != null && !_targetFile.exists()) {
-log("The targetfile \"" + _targetFile.getAbsolutePath()
+if (this.targetFile != null && !this.targetFile.exists()) {
+log("The targetfile \"" + this.targetFile.getAbsolutePath()
+ "\" does not exist.", Project.MSG_VERBOSE);
return false;
}
// if the source file isn't there, throw an exception
-if (_sourceFile != null && !_sourceFile.exists()) {
-throw new BuildException(_sourceFile.getAbsolutePath()
+if (this.sourceFile != null && !this.sourc

Re: [GUMP@brutus]: Project ant-xdocs-proposal (in module ant) failed

2004-12-02 Thread Steve Loughran


On Thu, 02 Dec 2004 09:32:22 +0100, Stefan Bodewig <[EMAIL PROTECTED]>
wrote:
> this is a bug in Gump which currently ignores the  setting
> telling it to add more memory to the build.
> 
> I'm in the process of brushing up my minimal Python knowledge to
> a-bit-more-than-minimal-but-still-nothing-to-talk-of so I can fix it.
> Don't hold your breath 8-)
> 
> Stefan
> 

I actually want to sit down and get this whole xdocs things working
properly. Wonderful that it is, it was written two+ years ago, and hasnt
been touched since. I use it for generating docs for external project
(Axis, smartfrog) and that really hurts.

What I'd like is

- fetch of needed components
-generate docbook content for feeding into db2pdf or FOP
-coverage of datatypes and nested elements
-better support for example code
-generate a quickref as well as the pages and a big PDF version
-usable by other projects

I'd probably start this in a new proposal subdir, as it would be fairly
different.

No time to start this this year, as I am off on a 3 week holiday, and
have no laptop. 3 weeks with no laptop!



-steve


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 32488] - Improvement to 'spawn' attribute

2004-12-02 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=32488





--- Additional Comments From [EMAIL PROTECTED]  2004-12-02 12:40 ---
How do we kill processes in a cross platform manner? What process ID were you
thinking of for Windows Apps?

All we have in the runtime is the Process object, and I think the way spawning
works is we forget about that after starting the program. There is no process 
ID. 

For your problem, deploy the server and tests in . 

Alternatively, and this is RMI here, have some RMI object that shuts down the
server, that you can talk to to tell it to terminate itself. This is what I do.

 

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[Patch] style of UpToDate

2004-12-02 Thread Kevin Jackson
Just removed the C++ style private variables
Why set dir to null here?  Is there a problem with the size of the 
object and the GC not collecting it in time?  if not it doesn't seem to 
be relevant

protected boolean scanDir(File srcDir, String[] files) {
   SourceFileScanner sfs = new SourceFileScanner(this);
   FileNameMapper mapper = null;
   File dir = srcDir;
   if (mapperElement == null) {
   MergingMapper mm = new MergingMapper();
   mm.setTo(this.targetFile.getAbsolutePath());
   mapper = mm;
   dir = null;
   } else {
   mapper = mapperElement.getImplementation();
   }
   return sfs.restrict(files, srcDir, dir, mapper).length == 0;
   }
Kev
Index: UpToDate.java
===
RCS file: 
/home/cvspublic/ant/src/main/org/apache/tools/ant/taskdefs/UpToDate.java,v
retrieving revision 1.36
diff -u -r1.36 UpToDate.java
--- UpToDate.java   9 Mar 2004 16:48:06 -   1.36
+++ UpToDate.java   2 Dec 2004 09:44:05 -
@@ -42,10 +42,10 @@
 
 public class UpToDate extends Task implements Condition {
 
-private String _property;
-private String _value;
-private File _sourceFile;
-private File _targetFile;
+private String property;
+private String value;
+private File sourceFile;
+private File targetFile;
 private Vector sourceFileSets = new Vector();
 
 protected Mapper mapperElement = null;
@@ -56,8 +56,8 @@
  *
  * @param property the name of the property to set if Target is up-to-date.
  */
-public void setProperty(String property) {
-_property = property;
+public void setProperty(final String property) {
+this.property = property;
 }
 
 /**
@@ -66,15 +66,15 @@
  *
  * @param value the value to set the property to if Target is up-to-date
  */
-public void setValue(String value) {
-_value = value;
+public void setValue(final String value) {
+this.value = value;
 }
 
 /**
  * Returns the value, or "true" if a specific value wasn't provided.
  */
 private String getValue() {
-return (_value != null) ? _value : "true";
+return (this.value != null) ? this.value : "true";
 }
 
 /**
@@ -83,8 +83,8 @@
  *
  * @param file the file we are checking against.
  */
-public void setTargetFile(File file) {
-_targetFile = file;
+public void setTargetFile(final File file) {
+this.targetFile = file;
 }
 
 /**
@@ -93,14 +93,14 @@
  *
  * @param file the file we are checking against the target file.
  */
-public void setSrcfile(File file) {
-_sourceFile = file;
+public void setSrcfile(final File file) {
+this.sourceFile = file;
 }
 
 /**
  * Nested  element.
  */
-public void addSrcfiles(FileSet fs) {
+public void addSrcfiles(final FileSet fs) {
 sourceFileSets.addElement(fs);
 }
 
@@ -121,32 +121,32 @@
  * see if the target(s) is/are up-to-date.
  */
 public boolean eval() {
-if (sourceFileSets.size() == 0 && _sourceFile == null) {
+if (sourceFileSets.size() == 0 && this.sourceFile == null) {
 throw new BuildException("At least one srcfile or a nested "
  + " element must be set.");
 }
 
-if (sourceFileSets.size() > 0 && _sourceFile != null) {
+if (sourceFileSets.size() > 0 && this.sourceFile != null) {
 throw new BuildException("Cannot specify both the srcfile "
  + "attribute and a nested  "
  + "element.");
 }
 
-if (_targetFile == null && mapperElement == null) {
+if (this.targetFile == null && mapperElement == null) {
 throw new BuildException("The targetfile attribute or a nested "
  + "mapper element must be set.");
 }
 
 // if the target file is not there, then it can't be up-to-date
-if (_targetFile != null && !_targetFile.exists()) {
-log("The targetfile \"" + _targetFile.getAbsolutePath()
+if (this.targetFile != null && !this.targetFile.exists()) {
+log("The targetfile \"" + this.targetFile.getAbsolutePath()
 + "\" does not exist.", Project.MSG_VERBOSE);
 return false;
 }
 
 // if the source file isn't there, throw an exception
-if (_sourceFile != null && !_sourceFile.exists()) {
-throw new BuildException(_sourceFile.getAbsolutePath()
+if (this.sourceFile != null && !this.sourceFile.exists()) {
+throw new BuildException(this.sourceFile.getAbsolutePath()
  + " not found.");
 }
 
@@ -159,14 +159,14 @@
ds.getIncludedFiles()

DO NOT REPLY [Bug 32488] New: - Improvement to 'spawn' attribute

2004-12-02 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=32488

   Summary: Improvement to 'spawn' attribute
   Product: Ant
   Version: 1.6.2
  Platform: PC
OS/Version: Linux
Status: NEW
  Severity: enhancement
  Priority: P2
 Component: Core tasks
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]


If this is the right place to ask this, I will, if not please ignore me. If this
is already in place I'd be grateful for a hint. 

I would like to be able to store the pid of a spawned process in an ant property
to be able to easily kill the spawned process after a build. If this is
impossible to do please ignore this request, I really have no idea how
complicated this might be.

Example: my test target starts an rmi server, I spawn this because I want to
continue with my build after starting the server. Then I run a test client,
afterwards I want to stop the server and the rmiregistry. This may not be the
best example because this specific problem might possibly be solved
programmatically within my code but I would need to modifiy my code for testing
which is not quite my favorite option.

Otherwise: thanks for an absolutely cool tool which makes every day as a
developer more enjoyable, keep up the great work.

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: [GUMP@brutus]: Project ant-xdocs-proposal (in module ant) failed

2004-12-02 Thread Stefan Bodewig
this is a bug in Gump which currently ignores the  setting
telling it to add more memory to the build.

I'm in the process of brushing up my minimal Python knowledge to
a-bit-more-than-minimal-but-still-nothing-to-talk-of so I can fix it.
Don't hold your breath 8-)

Stefan

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 32487] New: - requires OS-specific slashes

2004-12-02 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=32487

   Summary:   requires OS-specific slashes
   Product: Ant
   Version: 1.6.2
  Platform: PC
OS/Version: Windows XP
Status: NEW
  Severity: normal
  Priority: P2
 Component: Core tasks
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]


I've got the following task:


  
  


And what I notice is that if I run "ant -v" under win32, the paths returned by
 contain \\ style directory slashes and the mapper specified fails to
match the resulting path because it uses a different slash.

Expected behavior:  should be smart enough to convert slashes to the
appropriate format returned by  or vice-versa.

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[GUMP@brutus]: Project test-ant (in module ant) failed

2004-12-02 Thread Gump Integration Build
To whom it may engage...

This is an automated request, but not an unsolicited one. For 
more information please visit http://gump.apache.org/nagged.html, 
and/or contact the folk at [EMAIL PROTECTED]

Project test-ant has an issue affecting its community integration.
This issue affects 1 projects.
The current state of this project is 'Failed', with reason 'Build Failed'.
For reference only, the following projects are affected by this:
- test-ant :  Java based build tool


Full details are available at:
http://brutus.apache.org/gump/public/ant/test-ant/index.html

That said, some information snippets are provided here.

The following annotations (debug/informational/warning/error messages) were 
provided:
 -INFO- Failed with reason build failed



The following work was performed:
http://brutus.apache.org/gump/public/ant/test-ant/gump_work/build_ant_test-ant.html
Work Name: build_ant_test-ant (Type: Build)
Work ended in a state of : Failed
Elapsed: 6 mins 41 secs
Command Line: java -Djava.awt.headless=true 
-Xbootclasspath/p:/usr/local/gump/public/workspace/xml-xerces2/java/build/xercesImpl.jar:/usr/local/gump/public/workspace/xml-xerces2/java/build/xml-apis.jar:/usr/local/gump/public/workspace/xml-commons/java/external/build/xml-apis.jar:/usr/local/gump/public/workspace/xml-xalan/java/build/xalan-unbundled.jar:/usr/local/gump/public/workspace/xml-xalan/java/build/serializer.jar
 org.apache.tools.ant.Main 
-Dgump.merge=/home/gump/workspaces2/public/gump/work/merge.xml 
-Dbuild.sysclasspath=only -Dtest.haltonfailure=false 
-Dant.home=/usr/local/gump/public/workspace/ant/dist run-tests 
[Working Directory: /usr/local/gump/public/workspace/ant]
CLASSPATH: 
/opt/jdk1.4/lib/tools.jar:/usr/local/gump/public/workspace/ant/build/testcases:/usr/local/gump/public/workspace/ant/src/testcases:/usr/local/gump/public/workspace/ant/src/etc/testcases:/usr/local/gump/public/workspace/ant/build/lib/ant-stylebook.jar:/usr/local/gump/public/workspace/ant/build/lib/ant-jmf.jar:/usr/local/gump/public/workspace/ant/build/lib/ant-swing.jar:/usr/local/gump/public/workspace/ant/build/lib/ant-junit.jar:/usr/local/gump/public/workspace/ant/build/lib/ant-apache-resolver.jar:/usr/local/gump/public/workspace/ant/build/lib/ant-javamail.jar:/usr/local/gump/public/workspace/ant/build/lib/ant-apache-bcel.jar:/usr/local/gump/public/workspace/ant/build/lib/ant-apache-regexp.jar:/usr/local/gump/public/workspace/ant/build/lib/ant-trax.jar:/usr/local/gump/public/workspace/ant/build/lib/ant-commons-net.jar:/usr/local/gump/public/workspace/ant/build/lib/ant-jsch.jar:/usr/local/gump/public/workspace/ant/build/lib/ant-apache-log4j.jar:/usr/local/gump/public/workspace/ant/build/lib/ant-antlr.jar:/usr/local/gump/public/workspace/ant/build/lib/ant-commons-logging.jar:/usr/local/gump/public/workspace/ant/build/lib/ant-jdepend.jar:/usr/local/gump/public/workspace/ant/build/lib/ant-apache-bsf.jar:/usr/local/gump/public/workspace/ant/build/lib/ant-launcher.jar:/usr/local/gump/public/workspace/ant/build/lib/ant-nodeps.jar:/usr/local/gump/public/workspace/ant/build/lib/ant-apache-oro.jar:/usr/local/gump/public/workspace/ant/build/lib/ant.jar:/usr/local/gump/public/workspace/ant/build/lib/ant-jai.jar:/usr/local/gump/public/workspace/ant/dist/lib/ant-stylebook.jar:/usr/local/gump/public/workspace/ant/dist/lib/ant-jmf.jar:/usr/local/gump/public/workspace/ant/dist/lib/ant-swing.jar:/usr/local/gump/public/workspace/ant/dist/lib/ant-trax.jar:/usr/local/gump/public/workspace/ant/dist/lib/ant-junit.jar:/usr/local/gump/public/workspace/ant/dist/lib/ant-launcher.jar:/usr/local/gump/public/workspace/ant/dist/lib/ant-nodeps.jar:/usr/local/gump/public/workspace/ant/dist/lib/ant.jar:/usr/local/gump/public/workspace/ant/bootstrap/lib/ant-launcher.jar:/usr/local/gump/public/workspace/ant/bootstrap/lib/ant.jar:/usr/local/gump/public/workspace/xml-stylebook/bin/stylebook-1.0-b3_xalan-2.jar:/usr/local/gump/public/workspace/dist/junit/junit.jar:/usr/local/gump/packages/antlr-2.7.3/antlr.jar:/usr/local/gump/public/workspace/jakarta-commons/logging/dist/commons-logging.jar:/usr/local/gump/public/workspace/jakarta-commons/logging/dist/commons-logging-api.jar:/usr/local/gump/public/workspace/jakarta-commons/net/dist/commons-net-01122004.jar:/usr/local/gump/packages/jaf-1.0.1/activation.jar:/usr/local/gump/public/workspace/jakarta-bcel/bin/bcel.jar:/usr/local/gump/public/workspace/jakarta-bsf/build/lib/bsf.jar:/usr/local/gump/public/workspace/logging-log4j/log4j-01122004.jar:/usr/local/gump/public/workspace/logging-log4j/log4j-chainsaw-01122004.jar:/usr/local/gump/public/workspace/jakarta-oro/jakarta-oro-01122004.jar:/usr/local/gump/public/workspace/jakarta-regexp/build/jakarta-regexp-01122004.jar:/usr/local/gump/public/workspace/jakarta-servletapi-4/lib/servlet.jar:/usr/local/gump/packages/javamail-1.3.2/mail.jar:/usr/local/gump/packages/javamail-1.3.2/lib/mailapi.jar:/usr/local/gump/packages/jdepend-2.6/lib/jdepend.jar:/usr/local/gump/packages/jsch-0.1.17/dist/lib/