I opened bug 28423 for the problem described above. The simple fix would be to copy the touch code from ftp_retrieve_list() and place it after the call to ftp_loop_internal() in the non-glob case. Bugs like this tend to raise questions about the overall equivalence of the two code paths. Perhaps it would be better to collapse them down to one more general execution path. This is a little more along the lines of architecture work for wget - does anyone have any insight on this? Would a patch in that vein from me be likely to be accepted or is it better to leave that kind of work to official maintainers?
Thanks, David On Sat, Dec 26, 2009 at 9:52 PM, David Holman <[email protected]> wrote: > The changes in configure.ac were just to make it compile correctly on > my machine, they weren't intended as part of the fix for the bug. > Looking at this a little more closely, it appears that wget actually > does not even update the timestamps correctly on a single file > download via ftp. This execution path goes through > ftp_loop_internal(), but the only place touch() is used (for ftp) is > in ftp_retrieve_list() which is not called for a single-file download. > Perhaps this bug should be fixed first? How does one open new bugs on > wget? > > Thanks, > David > > On Sat, Dec 26, 2009 at 6:19 AM, Keisial <[email protected]> wrote: >> David Holman wrote: >>> Hi all, >>> I'm interested in contributing to wget, so I thought 25322 would be >>> a good place to start because it seems like a simple thing to fix. >>> I've attached a diff of code I've written to address this (taken >>> against mainline hg). >>> >> >> I admit being a complete ignorant regarding automake, but why are you >> doing this? >>> --configure.ac >>> dnl >>> dnl Automake setup >>> dnl >>> -AM_INIT_AUTOMAKE(dist-bzip2 dist-lzma 1.9) >>> +AM_INIT_AUTOMAKE(1.9) >> >>> In general, how are new options like this named? >>> Is my name acceptable (--match-server-timestamps)? >> Given that the default is to match the server timestamp, I'd call the >> new option >> more like --ignore-server-timestamp >> >> I also miss the new option at print_help(). >> >>> Is sending patches >>> to this list the best way for me to proceed with bug fixing? >>> >>> Thanks, >>> David >> So says http://wget.addictivecode.org/HelpingWithWget :) >> >> >
