After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 128620 - scrollkeeper complains on malformed XML
scrollkeeper complains on malformed XML
Status: RESOLVED FIXED
Product: rhythmbox
Classification: Other
Component: general
0.6.1
Other Linux
: Normal normal
: ---
Assigned To: RhythmBox Maintainers
RhythmBox Maintainers
Depends on:
Blocks:
 
 
Reported: 2003-12-05 21:11 UTC by Stephane Loeuillet
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: ---
GNOME version: 2.3/2.4


Attachments
Docs patch (2.49 KB, patch)
2003-12-10 11:47 UTC, Jordi Mallach
none Details | Review
Docs patch, take #2 (3.14 KB, patch)
2003-12-10 12:27 UTC, Jordi Mallach
none Details | Review

Description Stephane Loeuillet 2003-12-05 21:11:00 UTC
making a "scrollkeeper-rebuilddb", it complained :

/usr/share/gnome/help/rhythmbox/C/legal.xml:1: parser error : Missing
encoding in text declaration
<?xml version="1.0" ?>
                    ^
/usr/share/gnome/help/rhythmbox/C/rhythmbox.xml:72: error: Failure to
process entity legal
   &legal;
          ^
/usr/share/gnome/help/rhythmbox/C/rhythmbox.xml:72: parser error : Entity
'legal' not defined
   &legal;
          ^
/usr/share/gnome/help/rhythmbox/C/authors.xml:1: parser error : Missing
encoding in text declaration
<?xml version="1.0" ?>
                    ^
/usr/share/gnome/help/rhythmbox/C/rhythmbox.xml:213: error: Failure to
process entity authors
&authors;
         ^
/usr/share/gnome/help/rhythmbox/C/rhythmbox.xml:213: parser error : Entity
'authors' not defined
&authors;
         ^
/usr/share/gnome/help/rhythmbox/C/user-guide.xml:1: parser error : Missing
encoding in text declaration
<?xml version="1.0" ?>
                    ^
/usr/share/gnome/help/rhythmbox/C/rhythmbox.xml:215: error: Failure to
process entity user-guide
&user-guide;
            ^
/usr/share/gnome/help/rhythmbox/C/rhythmbox.xml:215: parser error : Entity
'user-guide' not defined
&user-guide;
            ^
/usr/share/gnome/help/rhythmbox/C/quick-reference.xml:1: parser error :
Missing encoding in text declaration
<?xml version="1.0" ?>
                    ^
/usr/share/gnome/help/rhythmbox/C/rhythmbox.xml:216: error: Failure to
process entity quick
&quick;
       ^
/usr/share/gnome/help/rhythmbox/C/rhythmbox.xml:216: parser error : Entity
'quick' not defined
&quick;
       ^
Document is not well-formed XML:
/usr/share/gnome/help/rhythmbox/C/rhythmbox.xml



hopefully, rhythmbox isn't the only bad boy for which scrollkeeper
complained. others were gnome-applets-2.4.1 and anjuta-1.1.98

gentoo x86 system, pretty up to date

i'm open to any question / testing

bye
Comment 1 Stephane Loeuillet 2003-12-10 09:39:33 UTC
change : anjuta is no longer a bad boy (1.2.0 fixed this). only
rhythmbox-0.6.1 and gnome-applets-2.4.1

anyway, it's not the most anoying problem i've encountered in my life.

better concentrate on gnome 2.5 / gstreamer 0.7
Comment 2 Jordi Mallach 2003-12-10 11:46:36 UTC
This patch should fix, please apply.
(Debian bugs #223538 and #220747).
Comment 3 Jordi Mallach 2003-12-10 11:47:28 UTC
Created attachment 22288 [details] [review]
Docs patch
Comment 4 Stephane Loeuillet 2003-12-10 12:14:05 UTC
the patch fixes everything but "authors.xml"

adding UTF-8 encoding to this file fixes the last bit
Comment 5 Jordi Mallach 2003-12-10 12:26:34 UTC
Woops. Silly autogenerated file. :)
This patch is better then.
Comment 6 Jordi Mallach 2003-12-10 12:27:35 UTC
Created attachment 22290 [details] [review]
Docs patch, take #2
Comment 7 Colin Walters 2003-12-13 01:46:47 UTC
I'm confused by this.  The XML standard explicity says that the
encoding= field is optional.  It then later very strongly implies that
if it is not supplied, implementations should assume UTF-8.

http://www.w3.org/TR/REC-xml#NT-XMLDecl
and:
http://www.w3.org/TR/REC-xml#charencoding

So it appears to me that this is a bug in Scrollkeeper.

I'll leave this as NEW pending your reply...
Comment 8 Colin Walters 2003-12-13 01:47:17 UTC
Should you disagree with my assessment, please re-add the PATCH keyword.
Comment 9 Jordi Mallach 2003-12-14 14:52:48 UTC
Hi walters,

Right, this appears to be a bug somwhere else (scrollkeeper, libxml2,
I don't know).

The patch does workaround it, and doesn't harm at all. I suggest it is
applied, the real bug will surely be fixed where it needs to be fixed.

If you don't agree, feel free to close the bug (I just wrote the patch
to stop the stream of Debian bugs coming in about the issue, and one
of the GNOME maintainers already applied it).
Comment 10 Colin Walters 2003-12-15 03:02:55 UTC
scrollkeeper-rebuilddb doesn't give this error on my system.  I'm
using 0.3.12.  Perhaps it's a bug introduced after that?

By the way, the proper name is "UTF-8", although probably most
implementations will accept "utf-8" too.

So I fixed that and applied your patch.  But could you forward this
issue on to the scrollkeeper upstream?  Thanks!