Closed
Bug 1692945
Opened 4 years ago
Closed 4 years ago
Clean up some configure parts related to yasm/nasm
Categories
(Firefox Build System :: General, task)
Firefox Build System
General
Tracking
(firefox87 fixed)
RESOLVED
FIXED
87 Branch
| Tracking | Status | |
|---|---|---|
| firefox87 | --- | fixed |
People
(Reporter: glandium, Assigned: glandium)
References
Details
Attachments
(2 files)
No description provided.
|
Assignee | |
Comment 1•4 years ago
|
||
Building the ICU data file hasn't required yasm since bug 1650299, and
all the compilers we support now have a GAS-like assembler that we use
unconditionally.
|
|
Assignee | |
Comment 2•4 years ago
|
||
None of HAVE_NASM, HAVE_YASM, NASM_MAJOR_VERSION and NASM_MINOR_VERSION are
used. Also, the YASM variable is not necessary for old-configure anymore.
Pushed by mh@glandium.org:
https://hg.mozilla.org/integration/autoland/rev/36fd576c711b
Remove check for yasm/gas in js. r=firefox-build-system-reviewers,andi,dmajor
https://hg.mozilla.org/integration/autoland/rev/44e6140070a0
Remove unused [YN]ASM variables. r=firefox-build-system-reviewers,andi,dmajor
|
||
Comment 4•4 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/36fd576c711b
https://hg.mozilla.org/mozilla-central/rev/44e6140070a0
Status: NEW → RESOLVED
Closed: 4 years ago
status-firefox87:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 87 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•