ant-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bugzi...@apache.org
Subject DO NOT REPLY [Bug 4411] - FTP task scandir method bug
Date Mon, 15 Apr 2002 09:59:02 GMT
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://nagoya.apache.org/bugzilla/show_bug.cgi?id=4411>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://nagoya.apache.org/bugzilla/show_bug.cgi?id=4411

FTP task scandir method bug





------- Additional Comments From bodewig@apache.org  2002-04-15 09:59 -------
I am glad you haven't lost interest, it just looked that way as you didn't respond
to Stephane's question.

You can easily download the latest CVS version from the webcvs frontend, patch it
so it does what you want and create a plain "diff -u".  The problem with
submitting the whole file is, that the change itself is rather difficult to track.

I've just diffed your file (ignoring whitespace changes) against the latest CVS
version and have found some changes that don't seem to be related to what you want
at all - so please help me understanding your changes.

The format of the listFile method is the way (some) people want it, so if you
want a different format, you should make that a new option.

If I understand your patch correctly, FTP right now will not look at the
directory part of a file name when matching includes/excludes, which you now
enable.  Correct?

You also add a new conditional scandir call inside the part that handles
excluded directories.

I think you are correct and have committed a change to that effect - could you
please see whether nightly build 2002-04-16 works as expected?

--
To unsubscribe, e-mail:   <mailto:ant-dev-unsubscribe@jakarta.apache.org>
For additional commands, e-mail: <mailto:ant-dev-help@jakarta.apache.org>


Mime
View raw message