Re: [Bug-wget] Bug #25322

[David Holman]

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 <address@hidden> 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 <address@hidden> 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 :)
>>
>>
>