Bug#430118: libxml-sax-perl: patch for choosing the default parser better

Niko Tyni ntyni at iki.fi
Thu Nov 8 21:23:34 UTC 2007


tag 430118 patch
thanks

Hi,

I'm attaching a proposed patch that fixes these RC issues mentioned in
this bug report. 

1) update-perl-sax-parsers makes the default parser be the last one
   returned by readdir(), resulting in the worst possible choice
   (XML::SAX::PurePerl) on ext3 filesystems with the dir_index option.

   The proposed fix is to assign a priority to each parser and set
   the XML::SAX::PurePerl priority low enough that all others will
   override it.

   The priorities are encoded into the ParserDetails.d entry filenames.
   The default priority is 50, XML::SAX::PurePerl gets 10, and parsers
   with the same priority are sorted in an alphabetic order.

   The priorities should be discussed between the maintainers
   of the parser modules (at least libxml-libxml-perl and
   libxml-sax-expat-perl). Currently XML::SAX::Expat ends up last and
   thus is the default parser when installed. This is probably the
   desired end result as well, so the transition should be smooth.

2) local changes to /etc/perl/XML/SAX/ParserDetails.ini aren't honored
   
   The patch handles the configuration file with ucf. The preinst script
   calculates and stores the md5sum for an unmodified config file to
   make sure that ucf dialogs don't pop up without reason.

3) local changes to the parser entries in /etc/perl/XML/SAX/ParserDetails.d/
   aren't honored

   The patch moves the ParserDetails.d directory under
   /var/lib/libxml-sax-perl. This means that the priorities introduced 
   in 1) can't be overridden by local configuration. I considered this
   but decided that it would be overengineering, as local configuration
   can go into the resulting ParserDetails.ini as well.

The issues should possibly cloned into three separate bugs, but the
patches can't really be made independent since they touch the same parts
of the code in several places.

The patch is formatted as an NMU, but the changes inside aren't quite as
minimal as would perhaps be proper for that. In particular, I decided
to remove the preinst pre-sarge cleanup of different iterations of the
/etc/perl/XML/SAX directory rather than patch the logic to handle yet
one move.

As the changes are rather extensive, I'd prefer something like this to
be included in a maintainer upload rather than an NMU. Jay, I haven't
seen anything from you on this since July. Would it be OK with you if
we adopted this for the pkg-perl group?

Any comments on the patch are naturally welcome.

Cheers,
-- 
Niko Tyni   ntyni at iki.fi
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 430118.patch
Type: text/x-diff
Size: 21733 bytes
Desc: not available
Url : http://lists.alioth.debian.org/pipermail/pkg-perl-maintainers/attachments/20071108/1cd91e63/attachment.patch 


More information about the pkg-perl-maintainers mailing list