Closed Bug 647388 Opened 9 years ago Closed 9 years ago

Remove makefile.ref

Categories

(Core :: JavaScript Engine, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla6

People

(Reporter: paul.biggar, Assigned: philor)

References

Details

(Whiteboard: fixed-in-tracemonkey)

Attachments

(1 file)

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.
On the same "we don't seem to use this file" tangent, what about killing js/landbranch.pl?
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.
Attached patch rm, take 1Splinter Review
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.)
Assignee: general → philringnalda
Status: NEW → ASSIGNED
Attachment #524994 - Flags: review?(jimb)
No objections.
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 on attachment 524994 [details] [diff] [review]
rm, take 1

Take it away, Phil!
Attachment #524994 - Flags: review?(jimb) → review+
Duplicate of this bug: 462542
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
Awe-some.
Thank you once again, philor.

With this I think I owe you 93 beers. Save up for the yacht is my advice.
http://hg.mozilla.org/mozilla-central/rev/dc6ce44bedd4
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.