Closed
Bug 5028
Opened 26 years ago
Closed 26 years ago
build error in RDF on HP-UX 10.20
Categories
(Core Graveyard :: RDF, defect, P3)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: bruce, Assigned: waterson)
Details
Attachments
(1 file)
(deleted),
patch
|
Details | Diff | Splinter Review |
The HP build is currently dead because of this error in
mozilla/rdf/base/public/rdf.h:
Error 104: "../../../dist/./include/rdf.h", line 47 # Token concatenation
did not create a single, valid new token.
static const char* kURI##prefix##_##name = ##namespace #name ;\
^^^^^^^^^^^^^
I've been watching it via tinderbox and the ports, so any prospective fix for it
will need to be tested there (I haven't got an HP machine).
Reporter | ||
Updated•26 years ago
|
Summary: build error in RDF on HP-UX 10.20
Reporter | ||
Comment 1•26 years ago
|
||
Btw, this happens from mozilla/rdf/base/src/nsContainerCursor.cpp (line 68):
DEFINE_RDF_VOCAB(RDF_NAMESPACE_URI, RDF, nextVal); // ad hoc way to
make containers fast
and RDF_NAMESPACE_URI is defined to be "http://www.w3.org/TR/WD-rdf-syntax#"
Comment 2•26 years ago
|
||
Comment 3•26 years ago
|
||
OK, I think DEFINE_RDF_VOCAB is just busted. I'm a little surprised that it
works on GCC, even. Patch attached that should fix things, so waterson et alii
can review and make sure that I'm not smoking crack. (It relies on the fact
that |namespace| will always be a string literal, which I believe is the case.)
If nobody objects, I'll check it in when the tree reopens.
Assignee | ||
Comment 4•26 years ago
|
||
Shaver, I verified on Mac and Win32: works fine. While you're at it, change
|namespace| to |ns| -- turns out that |namespace| is a C++ keyword, which is
surely just an accident waiting to happen.
Reporter | ||
Updated•26 years ago
|
Status: NEW → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 5•26 years ago
|
||
shaver fixed this one. (thanks) but the change from namespace to ns didn't
occur.
Assignee | ||
Comment 6•26 years ago
|
||
Okay, I'll do that next time I check in.
Reporter | ||
Updated•26 years ago
|
Status: RESOLVED → VERIFIED
Updated•6 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•