Closed
Bug 579137
Opened 14 years ago
Closed 14 years ago
Remove unneeded special extensions
Categories
(Other Applications Graveyard :: Narcissus, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: taustin, Assigned: taustin)
Details
(Whiteboard: fixed-in-tracemonkey)
Attachments
(1 file)
|
17.01 KB,
patch
|
gal
:
review+
|
Details | Diff | Splinter Review |
Since Narcissus no longer relies on a special build, these components can be removed from the C++ code (basically, everything in '#ifdef NARCISSUS' blocks).
|
Assignee | |
Comment 1•14 years ago
|
||
There is some reference to Narcissus in testing/sisyphus/bin/set-build-env.sh:
-narcissus)
export XCFLAGS="-DNARCISSUS=1"
export CFLAGS="-DNARCISSUS=1"
export CXXFLAGS="-DNARCISSUS=1"
;;
These lines should probably be removed, though I'm not sure how this script is used to test it out.|
|
Assignee | |
Comment 2•14 years ago
|
||
Attachment #457687 -
Flags: review?(gal)
|
||
Updated•14 years ago
|
Attachment #457687 -
Flags: review?(gal) → review+
|
|
Assignee | |
Comment 3•14 years ago
|
||
Changeset: http://hg.mozilla.org/tracemonkey/rev/2e96b91da603
Whiteboard: fixed-in-tracemonkey
|
||
Updated•14 years ago
|
Component: JavaScript Engine → Narcissus
Keywords: narcissus
Product: Core → Other Applications
QA Contact: general → narcissus
|
||
Comment 4•14 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/2e96b91da603
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
|
||
Updated•5 years ago
|
Product: Other Applications → Other Applications Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•