Bug #112017 for XML-XPath: Wide character in print at /usr/bin/xpath line 93.

Sun Feb 14 10:32:07 2016 gregoa [...] cpan.org - Ticket created

From:	gregoa [...] cpan.org
Subject:	Wide character in print at /usr/bin/xpath line 93.

We have the following bug reported to the Debian package of XML-XPath (https://bugs.debian.org/747268): It doesn't seem to be a bug in the packaging, so you may want to take a look. Thanks! Note that I can still reproduce the issue with 1.30. ------8<-----------8<-----------8<-----------8<-----------8<----- Package: libxml-xpath-perl Version: 1.13-7 File: /usr/bin/xpath I don't think there is anything the user can do to fix this. $ echo '<X>å</X>'|xpath -e /X Found 1 nodes in stdin: -- NODE -- Wide character in print at /usr/bin/xpath line 93. <X>å</X> ------8<-----------8<-----------8<-----------8<-----------8<----- Thanks for considering, gregor herrmann, Debian Perl Group

Sun Feb 21 15:54:23 2016 MANWAR [...] cpan.org - Correspondence added

Hi, Thanks for reporting the issue. I will get back to you with a fix asap. Best Regards, Mohammad S Anwar

Sun Feb 21 15:54:23 2016 The RT System itself - Status changed from 'new' to 'open'

Sun Feb 21 15:54:24 2016 MANWAR [...] cpan.org - Taken

Sun Feb 21 15:54:45 2016 MANWAR [...] cpan.org - Severity Normal added

Sun Feb 21 15:54:46 2016 MANWAR [...] cpan.org - Broken in 1.30 added

Mon Feb 22 08:15:52 2016 MANWAR [...] cpan.org - Correspondence added

Hi, I have patched the distribution, please give it a try. https://metacpan.org/release/XML-XPath https://github.com/Manwar/XML-XPath Many Thanks. Best Regards, Mohammad S Anwar

Mon Feb 22 15:12:11 2016 MANWAR [...] cpan.org - Fixed in 1.31 added

Mon Feb 22 16:28:15 2016 gregoa [...] debian.org - Correspondence added

Subject:	Re: [rt.cpan.org #112017] Wide character in print at /usr/bin/xpath line 93.
Date:	Mon, 22 Feb 2016 22:27:41 +0100
To:	Mohammad Sajid Anwar via RT <bug-XML-XPath [...] rt.cpan.org>
From:	gregor herrmann <gregoa [...] debian.org>

On Mon, 22 Feb 2016 08:15:53 -0500, Mohammad Sajid Anwar via RT wrote: Show quoted text

> I have patched the distribution, please give it a try. > > https://metacpan.org/release/XML-XPath > https://github.com/Manwar/XML-XPath

Thank you! This look indeed good with 1.31: % echo '<X>哇</X>'|xpath -e /X Found 1 nodes in stdin: -- NODE -- <X>哇</X> If I may ask you another favour, and without opening a couple of new tickets: We are currently carrying 4 patches in the Debian package; they are from times when we were not good at forwarding patches upstream :/ They are at https://anonscm.debian.org/cgit/pkg-perl/packages/libxml-xpath-perl.git/tree/debian/patches and apply in the order from the series file. Maybe it would make sense to apply them, so everyone profits and we could drop them? Thanks in advance, gregor -- .''`. Homepage https://info.comodo.priv.at/ - OpenPGP key 0xBB3A68018649AA06 : :' : Debian GNU/Linux user, admin, and developer - https://www.debian.org/ `. `' Member of VIBE!AT & SPI, fellow of the Free Software Foundation Europe `- NP: Little Richard: Tutti Frutti

Download signature.asc
application/pgp-signature 949b

Message body not shown because it is not plain text.

Tue Feb 23 08:49:08 2016 MANWAR [...] cpan.org - Correspondence added

Resolved.

Tue Feb 23 08:49:11 2016 MANWAR [...] cpan.org - Status changed from 'open' to 'resolved'

Tue Feb 23 10:05:20 2016 MANWAR [...] cpan.org - Correspondence added

Hi @GREGOA, Thanks for the patches, much appreciated. I have applied most of them except the patch for xpath script. It requires thorough review as it is big change. I will do my best to get it patched soon. https://github.com/Manwar/XML-XPath https://metacpan.org/release/MANWAR/XML-XPath-1.32 Best Regards, Mohammad S Anwar

Wed Feb 24 11:47:38 2016 gregoa [...] debian.org - Correspondence added

Subject:	Re: [rt.cpan.org #112017] Wide character in print at /usr/bin/xpath line 93.
Date:	Wed, 24 Feb 2016 17:47:14 +0100
To:	Mohammad Sajid Anwar via RT <bug-XML-XPath [...] rt.cpan.org>
From:	gregor herrmann <gregoa [...] debian.org>

On Tue, 23 Feb 2016 10:05:26 -0500, Mohammad Sajid Anwar via RT wrote: Show quoted text

> <URL: https://rt.cpan.org/Ticket/Display.html?id=112017 > > > Thanks for the patches, much appreciated. > I have applied most of them

That's excellent news, thank you very much. Show quoted text

> except the patch for xpath script. It requires thorough review as > it is big change. I will do my best to get it patched soon.

Indeed, this patch changes the script quite fundamentally (and the patch is not easy to read :/) Show quoted text

> https://github.com/Manwar/XML-XPath > https://metacpan.org/release/MANWAR/XML-XPath-1.32

I've now uploaded 1.32 to unstable. In our git repo I removed the three patches you applied, and moved the remainder of xpath-option-to-work-without-internet-connection.patch (the POD changes for examples/xpath) into the remaining example.patch: https://anonscm.debian.org/cgit/pkg-perl/packages/libxml-xpath-perl.git/plain/debian/patches/example.patch Thanks again, gregor -- .''`. Homepage https://info.comodo.priv.at/ - OpenPGP key 0xBB3A68018649AA06 : :' : Debian GNU/Linux user, admin, and developer - https://www.debian.org/ `. `' Member of VIBE!AT & SPI, fellow of the Free Software Foundation Europe `- NP: Tracy Chapman: In The Dark

Download signature.asc
application/pgp-signature 949b

Message body not shown because it is not plain text.