Remove binutils from mingw-clang build script
Categories
(Firefox Build System :: General: Unsupported Platforms, enhancement, P5)
Tracking
(Not tracked)
People
(Reporter: tjr, Assigned: tjr)
References
(Blocks 1 open bug)
Details
(Whiteboard: [no-nag])
Attachments
(1 file)
3.14 KB,
patch
|
jacek
:
review+
|
Details | Diff | Splinter Review |
Georg noticed that binutils doesn't seem to be used anymore. A try run confirms that nothing seems to break. I believe we can remove it.
Assignee | ||
Comment 1•5 years ago
•
|
||
Sorry; the observation was that we can use llvm-nm instead of binutil's nm.
The original need for binutils' nm was we needed it to perform the NSModule check; but we switched that over to use readobj when we started generating pdbs.
Assignee | ||
Comment 2•5 years ago
|
||
Comment 3•5 years ago
|
||
I think the patch is fine for nm itsel, but we should probably resolve bug 1471698 before going forward with this patch. It has a pending patch patch adding binutils objcopy and strip to the toolchain. While it should be possible to do the build without them (clang-cl toolchain doesn't have them), I think you mentioned at some point that it's needed to run tests for some reason (bug 1475994). llvm-objcopy exists, but doesn't support Windows targets yet, so if we want to use it, we will need binutils for that.
Comment 4•5 years ago
|
||
Comment on attachment 9036715 [details] [diff] [review] Bug 1520311 - Remove binutils from the mingw-clang toolchain build r?jacek,Build llvm-objcopy supports COFF files, to it looks like we won't need binutils for that. The patch looks fine then.
Comment 5•5 years ago
|
||
There's a r+ patch which didn't land and no activity in this bug for 2 weeks.
:tjr, could you have a look please?
Comment 7•5 years ago
|
||
There's a r+ patch which didn't land and no activity in this bug for 2 weeks.
:tjr, could you have a look please?
Assignee | ||
Comment 8•5 years ago
|
||
No change. Can we disable the bot for this bug?
Updated•5 years ago
|
Comment 10•5 years ago
|
||
I'll fix that, sorry for the noise.
Updated•5 years ago
|
Updated•5 years ago
|
Assignee | ||
Comment 11•5 years ago
|
||
Done as part of Bug 1471698
Description
•