175394 - About page broken on MacOS (9 and X)

Status: VERIFIED DUPLICATE of bug 197520

(Core :: XML, defect, P2)

Description

[J.J. Enser]

Using recent Mozilla CFM trunk builds, the about page (no matter how it is
invoked) displays an XML parser error.
"XML Parsing error: undefined entity ... about.xhtml, line 90, column 15"
(screen shot will follow)

This was reproduced with trunk builds from various dates, using classic or
carbon builds, and on OSes from 8.6 to 10.2.

Heikki says: "about: is XML now. Either a DTD file is missing (xhtml11.dtd) or
the DOCTYPE line is missing/wrong."

Developers have also reported that debug builds don't expose this problem.

If this is indeed a packaging issue, whoever worked on this change may have
forgotten something for the mac :-(

Comment 1

[J.J. Enser]

Attachment: Screen shot: broken about page on Mac OS.

Comment 2

[J.J. Enser]

setting priority and tfv to 1.2 final cuz we'd look pretty stupid shipping any
"final" release like this.
-> asa

Comment 3

[Heikki Toivonen (remove -bugzilla when emailing directly)]

If debug builds work it seems it is almost certainly a packaging issue. I tried
to find out what is going on but I no longer understand what package lists we
are supposed to use, the GRE stuff has made things confusing.

I *think* this stuff is now handled in embedding/config/ (even for normal
browser). But only three of the files there talk about xhtml.dtd (and mathml.dtd
since we build that by default now I think): 

basebrowser-mac-cfm
basebrowser-mac-cfmDebug
gre-win-supp

I have no idea why there is gre-win* files but no gre-mac or gre-unix. I would
expect us to get rid of the gre-win* files and use the basebrowser-win files
instead. 

My guess would be that the entries concerning xhtml and mathml should be copied
into basebrowser-mac-macho and/or basebrowser-mac-unix.

Comment 4

[Heikki Toivonen (remove -bugzilla when emailing directly)]

Btw, I would expect that about and MathML don't work in any Unix-style system
installed with the installer.

Comment 5

[Chak Nanga]

I'm not sure the GRE stuff should be causing this issue.

Anyway, here some info/clarification regarding the GRE files:

1. There's only gre-win* files since Windows is the only platform on which we
currently support the GRE - we do not have resources to work on GRE on Mac/Linux
yet, hence there are no gre-mac or gre-unix currently. This will change once
there's someone to work on these platforms.

2. We should *not* get rid of the gre-win* files in favor of the basebrowser-*
files. Here's why(at least on Windows for now):

GRE package = Files in basebrowser-win + Files in gre-win

[The reason we did it this way is that there are embedding users out there who
want a minimal set of files - for those users sake we kept the basebrowser-win
so they can still continue to get via the basebrowser-win package list. The GRE
package simply adds on the files in the gre-win package to the files in the
basebrowser-win to get the final GRE package]

GRE application support package = Files in basebrowser-win-supp + Files in
gre-win-supp

[Same concept as above here too - the "app support" files are things such as
chrome, res, prefs etc. which are meant to be part of an application. We're
mainly doing this so embedding customers who use a GRE can simply copy files in
the "gre_app_support" dir into their app to get default chrome etc.]

Comment 6

[Jon Granrose]

cc asasaki since he was working on another about.xhtml issue recently.

Comment 7

[Red Wolf]

*** Bug 183398 has been marked as a duplicate of this bug. ***

Comment 8

[Red Wolf]

This bug still exists in 1.2.1 final on Mac OS 9.1.

Comment 9

[u60234]

*** Bug 186822 has been marked as a duplicate of this bug. ***

Comment 10

[Greg K.]

WorksForMe using FizzillaMach/2003021017.

Comment 11

[James Nord]

I've just finished building HEAD (debug) pulled from CVS (today 17:00 GMT) and I
see this exact same error on WinXP.  I have not seen it with CVS builds before
(occured sometime withing the last 2 weeks)

Comment 12

[timeless]

the windows problem is bug 197520, if packaging didn't fix this then the root
problem is probably the same although the fix may be different.

Comment 13

[Doug Turner (:dougt)]

i think this is a dup of 197520.

*** This bug has been marked as a duplicate of 197520 ***

Comment 14

[timeless]

based on the analysis, i agree, and there's even a patch :)

Comment 15

[benc]

->XML