Skip Menu |

This queue is for tickets about the XML-TreeBuilder CPAN distribution.

Report information
The Basics
Id: 113781
Status: open
Priority: 0/
Queue: XML-TreeBuilder
People
Owner: Jeff.Fearn [...] gmail.com
Requestors: poki [...] redhat.com
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Thu Apr 14 03:03:32 2016 poki [...] redhat.com - Ticket created
Subject: XML::TreeBuilder bug report wrt. XML_CATALOG_FILES envvar handling
Date: Thu, 14 Apr 2016 09:03:15 +0200
To: bug-XML-TreeBuilder [...] rt.cpan.org
From: Jan Pokorný <poki [...] redhat.com>
Hello Jeff, this is to let you know that by actively touching XML_CATALOG_FILES environment variable (rather than a transparent pass-through to libxml2), more harm is actually done. The problem is that environment variable is intended to carry a space-separated list of URIs, but the Perl binding takes it as a single URI, hence escaping spaces as '%20'. This is worth fixing, IMHO. -- Jan (Poki)
Download (untitled)
application/pgp-signature 819b

Message body not shown because it is not plain text.

  Mon Apr 18 03:11:18 2016 Jeff.Fearn [...] gmail.com - Correspondence added
Hi Jan, can you tell me how this affecting you so I can test it? Cheers, Jeff.
  Mon Apr 18 03:11:18 2016 The RT System itself - Status changed from 'new' to 'open'
  Mon Apr 18 04:24:45 2016 poki [...] redhat.com - Correspondence added
Subject: Re: [rt.cpan.org #113781] XML::TreeBuilder bug report wrt. XML_CATALOG_FILES envvar handling
Date: Mon, 18 Apr 2016 10:24:32 +0200
To: Jeff Fearn via RT <bug-XML-TreeBuilder [...] rt.cpan.org>
From: Jan Pokorný <poki [...] redhat.com>
On 18/04/16 03:11 -0400, Jeff Fearn via RT wrote: Show quoted text
> <URL: https://rt.cpan.org/Ticket/Display.html?id=113781 > > > Hi Jan, can you tell me how this affecting you so I can test it?
See https://github.com/ClusterLabs/pacemaker/commit/2f6e74883b747087bc76ccdf74528681e3820c1b#diff-8d3e8f7411beefc4b162f0d8a5622353R77 Instead of generating a single catalog file (equivalent of publican-catalog-fallback in the referenced build arrangement) and passing XML_CATALOG_FILES="/etc/xml/catalog $(CURDIR)/publican-catalog-fallback" to publican script (and in turn, to TreeBuilder) I had to jump the extra hop and generate also a root catalog (publican-catalog, assumed equivalent to passing two files via variable) that is then passed to XML_CATALOG_FILES (and accepted by TreeBuilder this time around). It would be preferred if the compatibility with libxml2's inner handling of XML_CATALOG_FILES was retained in case of multiple URIs being specified, using space as a separator. Thanks for the responsiveness so far. -- Jan (Poki)
Download (untitled)
application/pgp-signature 819b

Message body not shown because it is not plain text.

  Mon Apr 18 18:48:38 2016 Jeff.Fearn [...] gmail.com - Queue changed from XML-TreeBuilder to XML-Catalog
  Mon Apr 18 18:53:37 2016 Jeff.Fearn [...] gmail.com - Correspondence added
Subject: XML_CATALOG_FILES can be a space separated list of local catalog
Moving this to XML::Catalog. "sub new" needs to split input url on spaces and loop over them.
  Mon Apr 18 18:53:38 2016 Jeff.Fearn [...] gmail.com - Taken
  Mon Apr 18 18:59:20 2016 Jeff.Fearn [...] gmail.com - Queue changed from XML-Catalog to XML-TreeBuilder
  Mon Apr 18 19:00:16 2016 Jeff.Fearn [...] gmail.com - Correspondence added
Moving back as the split needs to be in XML::TreeBuilder. Note to self: do not triage tickets before morning coffee.