<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Nov 5, 2013 at 2:46 PM, Mark Tompsett <span dir="ltr"><<a href="mailto:mtompset@hotmail.com" target="_blank">mtompset@hotmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><br>
In the beginning, I read something like:<br>
<a href="http://wiki.koha-community.org/wiki/Koha_on_ubuntu_-_tarball#Test_to_make_sure_the_SAX_Parser_is_setup_correctly" target="_blank">http://wiki.koha-community.<u></u>org/wiki/Koha_on_ubuntu_-_<u></u>tarball#Test_to_make_sure_the_<u></u>SAX_Parser_is_setup_correctly</a><br>


It was less clear and flushed out, but that was it.<br></blockquote><div><br></div><div>This all became a subject for discussion when I run <a href="http://sax_parser_test.pl">sax_parser_test.pl</a> on our packages-driven production server. I was impressed that it would saying PurePerl was the current SAX parser. I remembered this being the cause of several nightmares discussed on IRC and the official lists (even koha-es).</div>

<div><br></div><div>So I became aware of this and changed the wiki so we ask people to do for packages the exact thing we do for tarball installs. i.e. I proposed running this three lines</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

   sudo update-perl-sax-parsers --remove XML::LibXML::SAX::Parser<br>
   sudo update-perl-sax-parsers --priority 100 --add XML::LibXML::SAX::Parser<br>
   sudo update-perl-sax-parsers --update<br>
</blockquote><div><br></div><div>I was even about to post that as a patch for the debian install scripts. And then Galen told mtompset and I that the new MARC::File::XML version didn't rely on SAX anymore, so we wouldn't need to do that.</div>

<div><br></div><div>If that solves the whole SAX parser problem I'd ask Robin to add the latest version to our repos and remove any reference to this from the (3.12.x  onwards) stable releases docs.</div><div><br></div>

<div>The question would be: is this (handling MARCXML) the only piece of code that depends on properly set the SAX parser?</div><div><br></div><div>BTW, I have to insist that IMHO all the install-related documentation should refer to the stable branches and avoid confusing the users. So, until we make a desicion on this we should not make references to things we do for the current master.</div>

<div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">So, there are few questions:<br>
1) Is this still an issue?<br></blockquote><div><br></div><div>I think it is, and went unnoticed a while ago, for packages.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">


2) If so, how do we solve it best?<br></blockquote><div><br></div><div>+1 for pushing MARC::File::XML into our repo, if there are no others pieces of code that rely on having the expected SAX parser set as default; in which case I'd just add the referred lines to the docs so packages users set the Debian/Ubuntu SAX parser properly.</div>

<div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
3) If not, why is it no longer an issue?<br></blockquote><div><br></div><div>I hope it isn't.</div><div><br></div><div>Regards</div><div>To+</div></div><div><br></div>-- <br><div dir="ltr"><div>Tomás Cohen Arazi</div>

<div>Prosecretaría de Informática</div><div>Universidad Nacional de Córdoba</div><div>✆ +54 351 4333190 ext 13168</div><div>GPG: B76C 6E7C 2D80 551A C765  E225 0A27 2EA1 B2F3 C15F</div></div>
</div></div>