The default bug view has changed. See this FAQ.

Remove makefile.ref

RESOLVED FIXED in mozilla6

Status

()

Core
JavaScript Engine
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: Paul Biggar, Assigned: philor)

Tracking

Trunk
mozilla6
Points:
---
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: fixed-in-tracemonkey)

Attachments

(1 attachment)

(Reporter)

Description

6 years ago
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

6 years ago
On the same "we don't seem to use this file" tangent, what about killing js/landbranch.pl?
(Reporter)

Comment 2

6 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

6 years ago
Created attachment 524994 [details] [diff] [review]
rm, take 1

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)

Comment 4

6 years ago
No objections.
(Assignee)

Comment 5

6 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

6 years ago
Comment on attachment 524994 [details] [diff] [review]
rm, take 1

Take it away, Phil!
Attachment #524994 - Flags: review?(jimb) → review+

Updated

6 years ago
Duplicate of this bug: 462542
(Assignee)

Comment 8

6 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

6 years ago
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
Last Resolved: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.