bug-wget
[Top][All Lists]
Advanced

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

Re: Add --custom-html-attrs option to support custom HTML tags and attri


From: Darshit Shah
Subject: Re: Add --custom-html-attrs option to support custom HTML tags and attributes
Date: Thu, 23 Jan 2020 18:38:16 +0100

Hi Lyubomyr,

Thanks for your patch!

I haven't tested it yet, but I wonder why the existing switch
`--follow-tags=LIST` was not enough for your use case. IIRC, it was
exactly this case that it was implemented for.

* Lyubomyr Shaydariv <address@hidden> [200123 18:35]:
> Hi,
> 
> Some HTML documents use non-standard attributes that are essentially URLs 
> that might be walked through. The attached patch allows to specify new tags 
> and attributes to follow.
> 
> Example of use:
> 
> ./wget -nd -r -P . -A jpg \
>     --custom-html-attrs=div/big_img,div/med_img \
>     http://localhost/index.html
> 
> However:
> The patch does not include tests or external documentation updates of any 
> kind. It does not validate tag/attribute pairs except of simple "/" checking. 
> I'm not a C programmer, so I'm fine if the patch is considered poor and 
> consequently rejected.
> Thanks.


Attachment: signature.asc
Description: PGP signature


reply via email to

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