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 20205] - [PATCH] DifferentSelector: make ignoreFileTimes true by default
Date Sun, 25 May 2003 03:42:49 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=20205>.
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=20205

[PATCH] DifferentSelector: make ignoreFileTimes true by default

jefft@apache.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|CLOSED                      |REOPENED
         Resolution|FIXED                       |
            Summary|[PATCH] Small performance   |[PATCH] DifferentSelector:
                   |and doc fix for             |make ignoreFileTimes true by
                   |DifferentSelector           |default



------- Additional Comments From jefft@apache.org  2003-05-25 03:42 -------
Hi,

Yesterday, the logic of whether to ignore file timestamps was found to be
backwards, and fixed. I'd now like to suggest that the default value for
ignoreFileTimes be changed from true to false, because:

 - It's darn annoying ;) Most of the time, the file's timestamp is not
significant, and so IMHO the default should reflect this.
 - Current users of <different> are expecting file timestamps to be ignored: the
docs said they were ignored, and the code ignored them (doc and code bugs
cancelling out). The change yesterday probably broke scripts using <different>.

Small patch attached.

Thanks,

--Jeff

Mime
View raw message