Closed
Bug 1317504
Opened 8 years ago
Closed 8 years ago
Move LD to python configure
Categories
(Firefox Build System :: General, defect)
Firefox Build System
General
Tracking
(firefox53 fixed)
RESOLVED
FIXED
mozilla53
Tracking | Status | |
---|---|---|
firefox53 | --- | fixed |
People
(Reporter: glandium, Assigned: glandium)
References
(Blocks 1 open bug)
Details
Attachments
(5 files)
58 bytes,
text/x-review-board-request
|
chmanchester
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
chmanchester
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
chmanchester
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
chmanchester
:
review+
|
Details |
58 bytes,
text/x-review-board-request
|
chmanchester
:
review+
|
Details |
No description provided.
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment 6•8 years ago
|
||
mozreview-review |
Comment on attachment 8814016 [details]
Bug 1317504 - Warn that LD is not used by the build system.
https://reviewboard.mozilla.org/r/95310/#review96518
Attachment #8814016 -
Flags: review?(cmanchester) → review+
Comment 7•8 years ago
|
||
mozreview-review |
Comment on attachment 8814017 [details]
Bug 1317504 - Remove unused non-GCC/clang flags settings.
https://reviewboard.mozilla.org/r/95312/#review96522
Attachment #8814017 -
Flags: review?(cmanchester) → review+
Comment 8•8 years ago
|
||
mozreview-review |
Comment on attachment 8814018 [details]
Bug 1317504 - Remove the GNU_LD variable, nothing uses it.
https://reviewboard.mozilla.org/r/95314/#review96532
Attachment #8814018 -
Flags: review?(cmanchester) → review+
Comment 9•8 years ago
|
||
mozreview-review |
Comment on attachment 8814019 [details]
Bug 1317504 - Make MSVC builds use the LINK variable instead of LD.
https://reviewboard.mozilla.org/r/95316/#review96534
Attachment #8814019 -
Flags: review?(cmanchester) → review+
Comment 10•8 years ago
|
||
mozreview-review |
Comment on attachment 8814020 [details]
Bug 1317504 - Don't try to set LD from old-configure.
https://reviewboard.mozilla.org/r/95318/#review96536
Attachment #8814020 -
Flags: review?(cmanchester) → review+
Comment 11•8 years ago
|
||
Pushed by mh@glandium.org:
https://hg.mozilla.org/integration/autoland/rev/cffe1cd111f6
Warn that LD is not used by the build system. r=chmanchester
https://hg.mozilla.org/integration/autoland/rev/677f47fec9e7
Remove unused non-GCC/clang flags settings. r=chmanchester
https://hg.mozilla.org/integration/autoland/rev/fd007caf51df
Remove the GNU_LD variable, nothing uses it. r=chmanchester
https://hg.mozilla.org/integration/autoland/rev/6353e704bf2c
Make MSVC builds use the LINK variable instead of LD. r=chmanchester
https://hg.mozilla.org/integration/autoland/rev/8c1409a443ee
Don't try to set LD from old-configure. r=chmanchester
Comment 12•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/cffe1cd111f6
https://hg.mozilla.org/mozilla-central/rev/677f47fec9e7
https://hg.mozilla.org/mozilla-central/rev/fd007caf51df
https://hg.mozilla.org/mozilla-central/rev/6353e704bf2c
https://hg.mozilla.org/mozilla-central/rev/8c1409a443ee
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox53:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
Updated•7 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•