Closed
Bug 582955
Opened 14 years ago
Closed 14 years ago
Require yasm for arches where we need it
Categories
(Firefox Build System :: General, defect)
Tracking
(blocking2.0 betaN+)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
blocking2.0 | --- | betaN+ |
People
(Reporter: benjamin, Assigned: benjamin)
Details
Attachments
(1 file)
3.05 KB,
patch
|
ted
:
review+
|
Details | Diff | Splinter Review |
Builds should be deterministic, not based on whatever tools are installed on the build machine. We should require yasm for arches where we have assembly for it. I happened to discover this accidentally via http://www.listshow.net/201007/ubuntu-mozillateam/30042-xulrunner-should-depend-on-yasm.html
Assignee | ||
Comment 1•14 years ago
|
||
Attachment #461210 -
Flags: review?(ted.mielczarek)
Comment 2•14 years ago
|
||
I agree, we should make this change.
Comment 3•14 years ago
|
||
Comment on attachment 461210 [details] [diff] [review] Require YASM, rev. 1 Looks like it's just the "yasm" package here on Ubuntu, so that's easy enough. Please fix the error message to have more details (including the exact --disable-webm switch) like the one about MASM above.
Attachment #461210 -
Flags: review?(ted.mielczarek) → review+
Assignee | ||
Comment 4•14 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/f9bfe618ccec
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Updated•6 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•