bug-wget
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Bug-wget] WARC output


From: Ángel González
Subject: Re: [Bug-wget] WARC output
Date: Sat, 08 Oct 2011 21:10:28 +0200
User-agent: Thunderbird

Giuseppe Scrivano wrote:
the patch is huge and I think we don't want to add some many files into
the wget tree.  Can't we assume the user will install the warc tools by
herself and let configure check if they are installed or not?  This will
require some more work but the result will be much less intrusive.  What
do you think?

Thanks,
Giuseppe
I don't think all those files are even remotely needed.
I am seeing for instance, python files for creating warc interacting with curl.
Why would that be useful in wget repository?
I -optimistically- think we could make warc files with a simpler implementation.
Also, the patch seems to duplicate code (compare lines 337731-337810 with
337944-338013 in the patch file). Surely that could be refactored?




reply via email to

[Prev in Thread] Current Thread [Next in Thread]