[Bug-wget] Favicon is not downloaded (Suggestion for improvement)

2016-12-25 Thread Павел Серегов
Hi.

Often not exist code for favicon (in index.html), but site have.

My suggestion:
If use wget -m, need make download  http://example.com/favicon.ico

How do you like the idea?


Re: [Bug-wget] Favicon is not downloaded (Suggestion for improvement)

2017-01-04 Thread Ander Juaristi
Yeah, why not. I can't think of any reason why it would break current
wget deployments.

I know Gecko does that: it asks for /favicon.ico just after / if no
favicon was defined in the HTML.

On 25/12/16 15:42, Павел Серегов wrote:
> Hi.
> 
> Often not exist code for favicon (in index.html), but site have.
> 
> My suggestion:
> If use wget -m, need make download  http://example.com/favicon.ico
> 
> How do you like the idea?
> 



signature.asc
Description: OpenPGP digital signature


Re: [Bug-wget] Favicon is not downloaded (Suggestion for improvement)

2017-01-05 Thread Dale R. Worley
Павел Серегов  writes:
> Often not exist code for favicon (in index.html), but site have.
>
> My suggestion:
> If use wget -m, need make download  http://example.com/favicon.ico
>
> How do you like the idea?

The documentation for -m is:

   -m
   --mirror
   Turn on options suitable for mirroring.  This option turns on
   recursion and time-stamping, sets infinite recursion depth and
   keeps FTP directory listings.  It is currently equivalent to -r -N
   -l inf --no-remove-listing.

I suggest defining "--favicon" specifically to download
http(s):///favicon.ico, and then add --favicon to the specification
of --mirror.

Dale



Re: [Bug-wget] Favicon is not downloaded (Suggestion for improvement)

2017-01-05 Thread Tim Rühsen
On Donnerstag, 5. Januar 2017 10:40:48 CET Dale R. Worley wrote:
> Павел Серегов  writes:
> > Often not exist code for favicon (in index.html), but site have.
> > 
> > My suggestion:
> > If use wget -m, need make download  http://example.com/favicon.ico
> > 
> > How do you like the idea?
> 
> The documentation for -m is:
> 
>-m
>--mirror
>Turn on options suitable for mirroring.  This option turns on
>recursion and time-stamping, sets infinite recursion depth and
>keeps FTP directory listings.  It is currently equivalent to -r
> -N -l inf --no-remove-listing.
> 
> I suggest defining "--favicon" specifically to download
> http(s):///favicon.ico, and then add --favicon to the specification
> of --mirror.

With default 'on'. Dale, do you mind to open an issue for that at https://
github.com/rockdaboot/wget2 ?
IMO, it should go there first.

Tim


signature.asc
Description: This is a digitally signed message part.


Re: [Bug-wget] Favicon is not downloaded (Suggestion for improvement)

2017-01-06 Thread Dale R. Worley
Tim Rühsen  writes:
> Dale, do you mind to open an issue for that at https://
> github.com/rockdaboot/wget2 ?
> IMO, it should go there first.

Done.

I've also added it to Gnu Savannah's issue tracker.

Dale