11396 - external dtd files that have xml version don't load

Status: VERIFIED INVALID

Description

[Kathleen :Brade]

in order to get my xul files to load with external dtd files I had to remove the
xml version line from each of the external dtd files.  See rev 1.17 of mozilla/
editor/ui/dialogs/content/EdHLineProps.xul and rev 1.1 of mozilla/editor/ui/
dialogs/locale/en-US/EditorHLineProps.dtd.  Revision 1.18 should work with rev
1.1 of its dtd file.

It should be ok to have <?xml version="1.0"?> in the external dtd file.

Comment 1

[Kathleen :Brade]

cc [e-mail]

Comment 2

[Nisheeth Ranjan]

Accepting bug and setting milestone to M12...

Comment 3

[Nisheeth Ranjan]

James, it seems like this is a bug in expat.  The attached .xml/.dtd file should
work, but we end up getting an "error in processing external entity reference"
error from expat.  Please take a look into this.  Thanks a lot.

Comment 4

[Nisheeth Ranjan]

Created attachment 1428 [details]
bug11396.xml

Comment 5

[Nisheeth Ranjan]

Created attachment 1429 [details]
bug11396.dtd

Comment 6

[James Clark]

This is not a bug.  A TextDecl has to include an EncodingDecl. See production 77
of the XML Recommendation.

Comment 7

[Nisheeth Ranjan]

James is right.  Kathy, according to the XML spec., you have to specify the
encoding in the text declaration as follows: <?xml version="1.0"
encoding="ISO-8859-1"?>.  Or, you can leave the text decl out.

Marking this bug invalid.

Comment 8

[tao]

Why "ISO-8859-1" instead of "UTF-8"?

Comment 9

[Nisheeth Ranjan]

Actually, I picked ISO-8859-1 out of a hat.  UTF-8 works equally well.  I was
just using that as an example.

Comment 10

[Christine Hoffman]

Verified invalid.