Closed
Bug 647388
Opened 13 years ago
Closed 13 years ago
Remove makefile.ref
Categories
(Core :: JavaScript Engine, defect)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
mozilla6
People
(Reporter: paul.biggar, Assigned: philor)
References
Details
(Whiteboard: fixed-in-tracemonkey)
Attachments
(1 file)
132.78 KB,
patch
|
jimb
:
review+
|
Details | Diff | Splinter Review |
We use Makefile.in for nearly everything. We probably don't need Makefile.ref anymore. This might involve porting the features over, if any exist which are useful, of which I'm not sure.
Comment 1•13 years ago
|
||
On the same "we don't seem to use this file" tangent, what about killing js/landbranch.pl?
Reporter | ||
Comment 2•13 years ago
|
||
There are a dozen files like this. I'm filing these because I'm not going to do them, and some will make [good first bug]s. Might be good to make a "delete old crap" meta.
Assignee | ||
Comment 3•13 years ago
|
||
Maybe this would have made a [good first bug], not sure, but I'm stealing it because I've wanted to do it several times, because there's ancient cruft in there that always gives me false-positives while searching for markers for ancient dead code in the rest of the tree, since it built like it was 1999. According to bug 462451, which put it back in, it's hanging on for two reasons, editline/readline and ICC PGO. Near as I can tell, bug 462004 took care of the former, and the latter is bug 511791, where Andreas went off to build ICC PGO on October 2, 2009, and hasn't been back since :) (I asked on irc, and he said "icc works ok with spidermonkey these days afaict," and if it was something we really needed to support and people actually used but it didn't work, I'd expect to see at least one metoo or a cc on that bug sometime since.)
Comment 4•13 years ago
|
||
No objections.
Assignee | ||
Comment 5•13 years ago
|
||
The only docs on devmo I could find that didn't already say "ick, that stuff's ancient, you don't want that" was in https://developer.mozilla.org/En/SpiderMonkey/Internals, so I just fixed that.
Comment 6•13 years ago
|
||
Comment on attachment 524994 [details] [diff] [review] rm, take 1 Take it away, Phil!
Attachment #524994 -
Flags: review?(jimb) → review+
Assignee | ||
Comment 8•13 years ago
|
||
http://hg.mozilla.org/tracemonkey/rev/dc6ce44bedd4
Flags: in-testsuite-
OS: Mac OS X → All
Hardware: x86 → All
Whiteboard: [good first bug] → fixed-in-tracemonkey
Target Milestone: --- → mozilla6
Reporter | ||
Comment 9•13 years ago
|
||
Awe-some.
Comment 10•13 years ago
|
||
Thank you once again, philor. With this I think I owe you 93 beers. Save up for the yacht is my advice.
Comment 11•13 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/dc6ce44bedd4
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•