Stephen More <[EMAIL PROTECTED]> writes:

> 1. The prerequisites for HTML-Parser-3.14 should include HTML::Tagset

The Makefile.PL already had it in PREREQ_PM.  I have now fixed the
README (but it did not get into 3.15).  Did you notice any other
places?

> 2. I needed to edit Entities.pm in order to pass the 'make test'
>    I include my diff:
> 
> 311,312c311,312
> <    'and'      => chr(8743),
> <    'or'       => chr(8744),
> ---
> >    and      => chr(8743),
> >    or       => chr(8744),
> 315c315
> <    'int'      => chr(8747),
> ---
> >    int      => chr(8747),
> 320c320
> <    'ne'       => chr(8800),
> ---
> >    ne       => chr(8800),
> 322,324c322,324
> <    'le'       => chr(8804),
> <    'ge'       => chr(8805),
> <    'sub'      => chr(8834),
> ---
> >    le       => chr(8804),
> >    ge       => chr(8805),
> >    sub      => chr(8834),

I had that patched so it was included in 3.15.  This patch is only
needed to make the perl5.004 warnings about 'Ambiguous use of xxx =>
resolved to "xxx"' go away.  The tests should actually still pass.

> P.S. Isn't there a circular reference here:
> 
> In the README for HTML-Parser-3.14 one of the prerequisites is libwww-perl. 
> In the README for libwww-perl-5.48 one of the prerequisites is HTML-Parser.

Yes it is, but libwww-perl depends _more_ on HTML-Parser (via
HTML::HeadParser) than the other way around.  HTML::HeadParser can
actually be used without LWP; it just requires to be assosiated with
some object that implements the header() and push_header() methods.

Moving HTML::HeadParser to libwww-perl would fix this.  For now I just
rephrased the PREREQUISITES section of the HTML-Parser README to say:

| If you intend to use the HTML::HeadParser you probably want to install
| libwww-perl too.

Regards,
Gisle

Reply via email to