Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 273214 - =app-misc/lcdproc-0.5.3 has been released
Summary: =app-misc/lcdproc-0.5.3 has been released
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: High enhancement (vote)
Assignee: Robert Buchholz (RETIRED)
URL: https://sourceforge.net/tracker/?func...
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-06-08 15:36 UTC by Samuli Suominen (RETIRED)
Modified: 2010-12-29 01:02 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
--skip-validation (lcdproc-0.5.3_pre1-xmlto.patch,1.08 KB, patch)
2009-06-08 15:37 UTC, Samuli Suominen (RETIRED)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Samuli Suominen (RETIRED) gentoo-dev 2009-06-08 15:36:51 UTC
Attaching patch. This is because new xmlto adds -noent to xmllint options, so documentation validation is much more strict.
Comment 2 Robert Buchholz (RETIRED) gentoo-dev 2009-06-08 17:12:00 UTC
This would have to be fixed in the ebuild, not the Makefile as we call xmlto ourselves. Maybe we can get the xml fixed upstream? I filed a bug.
Comment 3 Samuli Suominen (RETIRED) gentoo-dev 2009-06-08 17:29:49 UTC
Upstream might want to take a look at,

http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=516253

For hints about howto fix their documentation.

Also, they might want to know the command to reproduce this by hand,

xmllint --noout -xinclude --postvalid --noent <filename>.docbook

That's same as new xmlto is running.
Comment 4 Samuli Suominen (RETIRED) gentoo-dev 2009-06-12 16:17:58 UTC
In upstream bug,

-- cut --

 Date: 2009-06-12 11:18
Sender: mmdolze

Fixed this in cvs HEAD. Please test!

-- cut --

Maybe do some cleanups of unused versions & add the skip validation hack to some old version(s) and drop it from next version?

Just let's not make this a blocker for the xmlto's stabling in next ~2 weeks..

Your turn :)
Comment 5 Robert Buchholz (RETIRED) gentoo-dev 2009-06-13 10:28:45 UTC
I'm still seeing this error on the nightly snapshot:

/usr/bin/xmlto: line 513: 22355 Segmentation fault      "$XMLLINT_PATH" --noout --nonet --xinclude --postvalid --noent "$INPUT_FILE" 2> "${VALIDATION}"
xmlto: lcdproc-CVS-current-20090613/docs/lcdproc-user/lcdproc-user.docbook does not validate (status 139)
xmlto: Fix document syntax or use --skip-validation option
I/O error : Attempt to load network entity http://www.oasis-open.org/docbook/xml/4.5/docbookx.dtd
lcdproc-CVS-current-20090613/docs/lcdproc-user/lcdproc-user.docbook:65: warning: failed to load external entity "http://www.oasis-open.org/docbook/xml/4.5/docbookx.dtd"
]>
  ^
lcdproc-CVS-current-20090613/docs/lcdproc-user/introduction.docbook:15: parser error : Entity 'smile' not defined
until the final release of LCDproc &smile;.
                                          ^
lcdproc-CVS-current-20090613/docs/lcdproc-user/introduction.docbook:148: parser error : chunk is not well balanced

^
lcdproc-CVS-current-20090613/docs/lcdproc-user/lcdproc-user.docbook:70: parser error : Failure to process entity introduction
&introduction;
              ^
lcdproc-CVS-current-20090613/docs/lcdproc-user/lcdproc-user.docbook:70: parser error : Entity 'introduction' not defined
&introduction;
              ^
lcdproc-CVS-current-20090613/docs/lcdproc-user/configuration.docbook:464: parser error : Entity 'hellip' not defined
The <command>&hellip;Key</command> lines define what the server does with keypre
                     ^
lcdproc-CVS-current-20090613/docs/lcdproc-user/configuration.docbook:740: parser error : chunk is not well balanced

^
lcdproc-CVS-current-20090613/docs/lcdproc-user/lcdproc-user.docbook:73: parser error : Failure to process entity configuration
&configuration;
               ^
lcdproc-CVS-current-20090613/docs/lcdproc-user/lcdproc-user.docbook:73: parser error : Entity 'configuration' not defined
&configuration;
               ^



There's some things wrong here: Why does it segfault? Why can't it load the DTD?
I wonder if the validity errors are valid or caused by the other (IO) errors.
Comment 6 Robert Buchholz (RETIRED) gentoo-dev 2009-06-13 10:38:43 UTC
+  13 Jun 2009; Robert Buchholz <rbu@gentoo.org> -lcdproc-0.5.1-r4.ebuild,
+  lcdproc-0.5.2-r1.ebuild, lcdproc-0.5.2-r2.ebuild,
+  lcdproc-0.5.3_pre1.ebuild:
+  Add --skip-validation to xmlto for bug 273214, will be fixed upstream in
+  0.5.3, clean up old ebuild.
Comment 7 Samuli Suominen (RETIRED) gentoo-dev 2009-07-05 14:24:56 UTC
(In reply to comment #5)
> I'm still seeing this error on the nightly snapshot:
> 
> /usr/bin/xmlto: line 513: 22355 Segmentation fault      "$XMLLINT_PATH" --noout
> --nonet --xinclude --postvalid --noent "$INPUT_FILE" 2> "${VALIDATION}"
> xmlto: lcdproc-CVS-current-20090613/docs/lcdproc-user/lcdproc-user.docbook does
> not validate (status 139)
> xmlto: Fix document syntax or use --skip-validation option
> I/O error : Attempt to load network entity
> http://www.oasis-open.org/docbook/xml/4.5/docbookx.dtd
> lcdproc-CVS-current-20090613/docs/lcdproc-user/lcdproc-user.docbook:65:
> warning: failed to load external entity
> "http://www.oasis-open.org/docbook/xml/4.5/docbookx.dtd"
> ]>
>   ^

It seems new release of lcdproc is out, 0.5.3 final.
I got the same error as you did, but when I installed:

=app-text/docbook-xml-dtd-4.5*

It started working. So I guess you need this as a DEPEND.
Comment 8 Samuli Suominen (RETIRED) gentoo-dev 2009-07-05 23:01:43 UTC
Changed Summary to reflect current situation
Comment 9 Robert Buchholz (RETIRED) gentoo-dev 2009-11-01 16:43:18 UTC
*lcdproc-0.5.3 (01 Nov 2009)

  01 Nov 2009; Robert Buchholz <rbu@gentoo.org> -lcdproc-0.5.3_pre1.ebuild,
  +lcdproc-0.5.3.ebuild:
  Add docbook-xml-dtd as direct dependency, version bump to final. Thanks to
  Samuli Suominen, bug #273214.
Comment 10 Leho Kraav (:macmaN @lkraav) 2010-12-29 01:02:28 UTC
side note: lcdproc-0.5.4_pre1 has just been released fresh off the press, compiled fine with just renaming 0.5.3 ebuild.