Re: [Bug-wget] Regular expression matching

[Giuseppe Scrivano]

Hi Gijs,

Gijs van Tulder <address@hidden> writes:

> Here is a patch that adds the --acceptregex and --rejectregex options.
>
> With these options it would be possible to do two things:
>
> 1. You can match complete urls, instead of just the directory prefix
> or the file name suffix (which you can do with --accept and
> --include-directories).
> 2. You can use regular expressions to do the matching, which is
> sometimes easier to than using a list of wildcard patterns.
>
> Now this isn't a new idea (there are long discussions in the archive,
> see [1]). But somehow the previous attempts didn't make it, so I
> thought I'd send my own version. It's a small patch, I've been using
> it for a while and found it really useful.
>
> I've made two versions of the patch: one uses PCRE, the other uses the
> gnulib regex library, which is probably easier to integrate.

could you please integrate these two versions into one patch?  The PCRE
library is not always available so configure should check for it (as
your patch does), while the gnulib regex version is always present.

I like the Micah's idea of having --regex-type=pcre|posix|...

I am planning to release wget in the next week(s) so I would like to
postpone this change after the release and have more time for tests.

Thanks to work on it.

Giuseppe