[MacPorts] #36354: docbook-utils @ 0.6.14 regex bug
#36354: docbook-utils @ 0.6.14 regex bug ---------------------------------+-------------------------------- Reporter: lockhart@… | Owner: macports-tickets@… Type: defect | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.1.2 Keywords: haspatch maintainer | Port: docbook-utils ---------------------------------+-------------------------------- The jw script wrapper contains two regex patterns fed to both grep and sed. GNU grep 2.5.1 on Lion did not complain, but Mountain Lion apparently ships BSD grep 2.5.1-FreeBSD and this one complains about it. But in any case the regex seems to be broken on all versions; it uses a character class [:space:] but does not surround it with an extra bracket (e.g. [[:space:]]) and does not call grep with the -E option to allow it to recognize extended patterns. Patch is tested on both Lion and Mountain Lion. -- Ticket URL: <https://trac.macports.org/ticket/36354> MacPorts <http://www.macports.org/> Ports system for Mac OS
#36354: docbook-utils @ 0.6.14 regex bug ----------------------------+--------------------------------- Reporter: lockhart@… | Owner: macports-tickets@… Type: defect | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.1.2 Resolution: | Keywords: haspatch maintainer Port: docbook-utils | ----------------------------+--------------------------------- Comment (by lockhart@…): Replying to [ticket:36354 lockhart@…]:
The jw script wrapper contains two regex patterns fed to both grep and sed. GNU grep 2.5.1 on Lion did not complain, but Mountain Lion apparently ships BSD grep 2.5.1-FreeBSD and this one complains about it. But in any case the regex seems to be broken on all versions; it uses a character class [:space:] but does not surround it with an extra bracket (e.g. {{{[[:space:]]}}}) and does not call grep with the -E option to allow it to recognize extended patterns. Patch is tested on both Lion and Mountain Lion.
Whoops, didn't escape that second pattern properly. Intended to have: {{{ [[:space:]] }}} -- Ticket URL: <https://trac.macports.org/ticket/36354#comment:1> MacPorts <http://www.macports.org/> Ports system for Mac OS
#36354: docbook-utils @ 0.6.14 regex bug ----------------------------+--------------------------------- Reporter: lockhart@… | Owner: macports-tickets@… Type: defect | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.1.2 Resolution: | Keywords: haspatch maintainer Port: docbook-utils | ----------------------------+--------------------------------- Description changed by ryandesign@…: Old description:
The jw script wrapper contains two regex patterns fed to both grep and sed. GNU grep 2.5.1 on Lion did not complain, but Mountain Lion apparently ships BSD grep 2.5.1-FreeBSD and this one complains about it. But in any case the regex seems to be broken on all versions; it uses a character class [:space:] but does not surround it with an extra bracket (e.g. [[:space:]]) and does not call grep with the -E option to allow it to recognize extended patterns. Patch is tested on both Lion and Mountain Lion.
New description: The jw script wrapper contains two regex patterns fed to both grep and sed. GNU grep 2.5.1 on Lion did not complain, but Mountain Lion apparently ships BSD grep 2.5.1-FreeBSD and this one complains about it. But in any case the regex seems to be broken on all versions; it uses a character class [:space:] but does not surround it with an extra bracket (e.g. ![[:space:]]) and does not call grep with the -E option to allow it to recognize extended patterns. Patch is tested on both Lion and Mountain Lion. -- -- Ticket URL: <https://trac.macports.org/ticket/36354#comment:2> MacPorts <http://www.macports.org/> Ports system for Mac OS
#36354: docbook-utils @ 0.6.14 regex bug ----------------------------+--------------------------------- Reporter: lockhart@… | Owner: macports-tickets@… Type: defect | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.1.2 Resolution: | Keywords: haspatch maintainer Port: docbook-utils | ----------------------------+--------------------------------- Comment (by lockhart@…): Hmm. I see that the latest RPM has patches for the same issue. Please wait until I can integrate those patches in directly to try to keep as compatible as possible with the upstream sources and packaging. -- Ticket URL: <https://trac.macports.org/ticket/36354#comment:3> MacPorts <http://www.macports.org/> Ports system for Mac OS
#36354: docbook-utils @ 0.6.14 regex bug ----------------------------+--------------------------------- Reporter: lockhart@… | Owner: macports-tickets@… Type: defect | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.1.2 Resolution: | Keywords: haspatch maintainer Port: docbook-utils | ----------------------------+--------------------------------- Comment (by lockhart@…): Added all patch files and an extra "helper program" from the original RPM used as source for this port. Note that there is an extraneous copy of docbook-utils-newgrep.patch (*-2.patch) which I accidentally posted. Just use the original file docbook-utils-newgrep.patch. I am able to generate docbook documentation using this version of the port with all of these patches applied. -- Ticket URL: <https://trac.macports.org/ticket/36354#comment:4> MacPorts <http://www.macports.org/> Ports system for Mac OS
#36354: docbook-utils @ 0.6.14 regex bug ----------------------------+--------------------------------- Reporter: lockhart@… | Owner: macports-tickets@… Type: defect | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.1.2 Resolution: | Keywords: haspatch maintainer Port: docbook-utils | ----------------------------+--------------------------------- Comment (by piergiuseppe.fogli@…): Does not work on Snow Leopard (10.6.8), MacPorts 2.1.2 . Install log attached. -- Ticket URL: <https://trac.macports.org/ticket/36354#comment:5> MacPorts <http://www.macports.org/> Ports system for Mac OS
#36354: docbook-utils @ 0.6.14 regex bug ----------------------------+--------------------------------- Reporter: lockhart@… | Owner: macports-tickets@… Type: defect | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.1.2 Resolution: | Keywords: haspatch maintainer Port: docbook-utils | ----------------------------+--------------------------------- Comment (by lockhart@…): Replying to [comment:5 piergiuseppe.fogli@…]:
Does not work on Snow Leopard (10.6.8), MacPorts 2.1.2 . Install log attached.
I don't have Snow Leopard to test. Do you know if the previous version of docbook-utils worked on Snow Leopard? If so I can just make some of these patches conditional on later releases of Mac OS X which apparently have newer (or different) grep and sed. -- Ticket URL: <https://trac.macports.org/ticket/36354#comment:7> MacPorts <http://www.macports.org/> Ports system for Mac OS
#36354: docbook-utils @ 0.6.14 regex bug ----------------------------+--------------------------------- Reporter: lockhart@… | Owner: macports-tickets@… Type: defect | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.1.2 Resolution: | Keywords: haspatch maintainer Port: docbook-utils | ----------------------------+--------------------------------- Comment (by pixilla@…): Replying to [comment:5 piergiuseppe.fogli@…]:
Does not work on Snow Leopard (10.6.8), MacPorts 2.1.2 . Install log attached.
Works here on Snow Leopard: {{{ $ sw_vers -productVersion; xcodebuild -version | head -n1;uname -p; port -q installed docbook-utils 10.6.8 Xcode 3.2.6 i386 docbook-utils @0.6.14_0 docbook-utils @0.6.14_1 (active) }}} -- Ticket URL: <https://trac.macports.org/ticket/36354#comment:8> MacPorts <http://www.macports.org/> Ports system for Mac OS
#36354: docbook-utils @ 0.6.14 regex bug ----------------------------+--------------------------------- Reporter: lockhart@… | Owner: macports-tickets@… Type: defect | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.1.2 Resolution: | Keywords: haspatch maintainer Port: docbook-utils | ----------------------------+--------------------------------- Comment (by piergiuseppe.fogli@…): Thank you for your reply! [[BR]] Finally I succeeded in installing it.[[BR]] I don't know if previous versions worked, it's a new install not an upgrade.[[BR]] On a different Mac with identical software environment the install process went OK:[[BR]] {{{ $ sw_vers -productVersion; xcodebuild -version | head -n1;uname -p; port version; port -q installed docbook-utils 10.6.8 Xcode 3.2.6 i386 Version: 2.1.2 docbook-utils @0.6.14_1 (active) }}} I discovered a difference between the two Mac in the file /opt/local/etc/sgml/catalog: On the "working" Mac: {{{ -- Created by XML Catalog Manager 2.2 -- CATALOG "/opt/local/share/dsssl/docbook-dsssl/catalog" CATALOG "/opt/local/share/sgml/iso8879/catalog" CATALOG "/opt/local/share/sgml/docbook/4.2/docbook.cat" CATALOG "/opt/local/share/sgml/openjade/catalog" }}} On the "failing" Mac: {{{ -- Created by XML Catalog Manager 2.2 -- CATALOG "/opt/local/share/xml/docbook/4.2/docbook.cat" CATALOG "/opt/local/share/xml/docbook/4.1.2/docbook.cat" CATALOG "/opt/local/share/xml/docbook/4.3/docbook.cat" CATALOG "/opt/local/share/xml/docbook/4.4/docbook.cat" CATALOG "/opt/local/share/xml/docbook/4.5/docbook.cat" CATALOG "/opt/local/share/dsssl/docbook-dsssl/catalog" CATALOG "/opt/local/share/sgml/iso8879/catalog" CATALOG "/opt/local/share/sgml/docbook/4.2/docbook.cat" CATALOG "/opt/local/share/sgml/openjade/catalog" }}} When I changed the file on the "failing" Mac to: {{{ -- Created by XML Catalog Manager 2.2 -- CATALOG "/opt/local/share/dsssl/docbook-dsssl/catalog" CATALOG "/opt/local/share/sgml/iso8879/catalog" CATALOG "/opt/local/share/sgml/docbook/4.2/docbook.cat" CATALOG "/opt/local/share/sgml/openjade/catalog" CATALOG "/opt/local/share/xml/docbook/4.2/docbook.cat" CATALOG "/opt/local/share/xml/docbook/4.1.2/docbook.cat" CATALOG "/opt/local/share/xml/docbook/4.3/docbook.cat" CATALOG "/opt/local/share/xml/docbook/4.4/docbook.cat" CATALOG "/opt/local/share/xml/docbook/4.5/docbook.cat" }}} the install process was successful. -- Ticket URL: <https://trac.macports.org/ticket/36354#comment:9> MacPorts <http://www.macports.org/> Ports system for Mac OS
#36354: docbook-utils @ 0.6.14 regex bug ----------------------------+--------------------------------- Reporter: lockhart@… | Owner: macports-tickets@… Type: defect | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.1.2 Resolution: | Keywords: haspatch maintainer Port: docbook-utils | ----------------------------+--------------------------------- Comment (by lockhart@…): Replying to [comment:9 piergiuseppe.fogli@…]:
Thank you for your reply! [[BR]] Finally I succeeded in installing it.[[BR]] ... the install process was successful.
Thanks for the detail on what seemed to have failed. The catalog file has basic content from sgml-common then is incrementally modified by the various other docbook-xml versions. Not sure why it was not complete. Since it worked on the next Snow Leopard machine I'm thinking that whatever the problem was it may not be specific to Snow Leopard and may have been a hiccup on the first one. -- Ticket URL: <https://trac.macports.org/ticket/36354#comment:10> MacPorts <http://www.macports.org/> Ports system for Mac OS
#36354: docbook-utils @ 0.6.14 regex bug ----------------------------+--------------------------------- Reporter: lockhart@… | Owner: macports-tickets@… Type: defect | Status: new Priority: Normal | Milestone: Component: ports | Version: 2.1.2 Resolution: | Keywords: haspatch maintainer Port: docbook-utils | ----------------------------+--------------------------------- Comment (by lockhart@…): The report on Snow Leopard machines indicates that in at least some cases the installation succeeds and is usable. btw the failed installation is due to a failure to have docbook-utils build its own documentation. Maybe there was an ordering problem in which packages were installed first? But for now the problem seems to be not reproducible. Can we push the updates through? -- Ticket URL: <https://trac.macports.org/ticket/36354#comment:11> MacPorts <http://www.macports.org/> Ports system for Mac OS
#36354: docbook-utils @ 0.6.14 regex bug ----------------------------+--------------------------------- Reporter: lockhart@… | Owner: macports-tickets@… Type: defect | Status: closed Priority: Normal | Milestone: Component: ports | Version: 2.1.2 Resolution: fixed | Keywords: haspatch maintainer Port: docbook-utils | ----------------------------+--------------------------------- Changes (by pixilla@…): * status: new => closed * resolution: => fixed Comment: Replying to [comment:11 lockhart@…]:
Can we push the updates through?
See r98259 on 2012-09-30. -- Ticket URL: <https://trac.macports.org/ticket/36354#comment:12> MacPorts <http://www.macports.org/> Ports system for Mac OS
participants (1)
-
MacPorts