Closed
Bug 648876
Opened 13 years ago
Closed 13 years ago
Remove superfluous $(OS_ARCH) filter-outs from js/src/Makefile.in (bug 647389 follow-up)
Categories
(Core :: JavaScript Engine, defect)
Core
JavaScript Engine
Tracking
()
VERIFIED
FIXED
mozilla6
People
(Reporter: emorley, Assigned: emorley)
References
Details
Attachments
(1 file)
1.77 KB,
patch
|
jimb
:
review+
|
Details | Diff | Splinter Review |
Follow-up to bug 647389, which removed all instances of |WINCE| from js/src/Makefile.in This left a few of: > ifeq (,$(filter-out XXXXX,$(OS_ARCH))) and > ifneq (,$(filter-out XXXXX,$(OS_ARCH))) ...whereby XXXXX was just one platform; and as such is equivalent to: > ifeq ($(OS_ARCH),XXXXX) and > ifneq ($(OS_ARCH),XXXXX)
Assignee | ||
Comment 1•13 years ago
|
||
Updated•13 years ago
|
Attachment #524969 -
Flags: review?(jorendorff) → review?(jimb)
Comment 2•13 years ago
|
||
Shifting the review to jimb, who actually knows this language ;)
Assignee | ||
Comment 3•13 years ago
|
||
Sorry, just used your r+ in bug 647389 as inspiration for who to pick! :-) Is there a list somewhere of who is best to ask for reviews? (Other than just looking at hg log)
Comment 4•13 years ago
|
||
(In reply to comment #2) > Shifting the review to jimb, who actually knows this language ;) Oh, it's a very pleasant little stringy language! You'd *love* it!
Comment 5•13 years ago
|
||
Comment on attachment 524969 [details] [diff] [review] Patch v1 (based on TM rev 4ace629bb676) Thanks for the cleanup!
Attachment #524969 -
Flags: review?(jimb) → review+
Assignee | ||
Comment 6•13 years ago
|
||
(In reply to comment #5) > Thanks for the cleanup! No problem! Thanks for the speedy review :-)
Keywords: checkin-needed
Comment 7•13 years ago
|
||
(In reply to comment #3) > Sorry, just used your r+ in bug 647389 as inspiration for who to pick! :-) > > Is there a list somewhere of who is best to ask for reviews? (Other than just > looking at hg log) hg log and blame work best, but there is also this list: https://wiki.mozilla.org/JavaScript:Hackers
Assignee | ||
Comment 8•13 years ago
|
||
(In reply to comment #7) > hg log and blame work best, but there is also this list: > https://wiki.mozilla.org/JavaScript:Hackers Great, thanks!
Comment 9•13 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/58323bbaaa67
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla6
Comment 10•13 years ago
|
||
(In reply to comment #9) > http://hg.mozilla.org/mozilla-central/rev/58323bbaaa67 Just an FYI - changes in js/src/ should go into the tracemonkey tree. This should be fine this time though, no need for a backout or anything.
Comment 11•13 years ago
|
||
(In reply to comment #10) > (In reply to comment #9) > > http://hg.mozilla.org/mozilla-central/rev/58323bbaaa67 > > Just an FYI - changes in js/src/ should go into the tracemonkey tree. This > should be fine this time though, no need for a backout or anything. Sorry, I usually pay attention to not land js checkin-needed stuff, this one got through my list somehow...
Assignee | ||
Comment 12•13 years ago
|
||
http://mxr.mozilla.org/mozilla-central/search?string=%28%2C%24%28filter-out+WINNT&find=%2Fjs%2Fsrc%2FMakefile.in&findi=&filter=^[^\0]*%24&hitlimit=&tree=mozilla-central ...returns 0 results. -> Verified.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•