Closed
Bug 1787087
Opened 2 years ago
Closed 2 years ago
system-symbols-mac docker images are busted with src/common/dwarf/elf_reader.cc:200:15: error: 'strcmp' is not a member of 'std'; did you mean 'strcmp'?
Categories
(Firefox Build System :: Task Configuration, task)
Firefox Build System
Task Configuration
Tracking
(firefox106 fixed)
RESOLVED
FIXED
106 Branch
Tracking | Status | |
---|---|---|
firefox106 | --- | fixed |
People
(Reporter: glandium, Assigned: gsvelto)
References
Details
Attachments
(1 file)
src/common/dwarf/elf_reader.cc: In constructor 'google_breakpad::ElfSectionReader<ElfArch>::ElfSectionReader(const char*, const string&, int, const typename ElfArch::Shdr&)':
src/common/dwarf/elf_reader.cc:200:15: error: 'strcmp' is not a member of 'std'
if ((std::strcmp(name, ".strtab") == 0 ||
^~~~~~
src/common/dwarf/elf_reader.cc:200:15: note: suggested alternative:
In file included from src/common/dwarf/elf_reader.cc:35:
/usr/include/string.h:136:12: note: 'strcmp'
extern int strcmp (const char *__s1, const char *__s2)
^~~~~~
src/common/dwarf/elf_reader.cc:201:15: error: 'strcmp' is not a member of 'std'
std::strcmp(name, ".shstrtab") == 0) &&
^~~~~~
src/common/dwarf/elf_reader.cc:201:15: note: suggested alternative:
In file included from src/common/dwarf/elf_reader.cc:35:
/usr/include/string.h:136:12: note: 'strcmp'
extern int strcmp (const char *__s1, const char *__s2)
^~~~~~
The underlying problem is that building the docker image builds whatever version of breakpad is current at the time the docker image is built (and thus is not reproducible and a constant time bomb)
Reporter | ||
Comment 1•2 years ago
|
||
Comes from this upstream change: https://chromium.googlesource.com/breakpad/breakpad/+/f1f7b5272fdf105592fc262a9adaab55a3621122%5E%21/
Assignee | ||
Comment 2•2 years ago
|
||
Time to get breakpad out of the way, we can use the new dump_syms here.
Assignee: nobody → gsvelto
Status: NEW → ASSIGNED
Flags: needinfo?(gsvelto)
Assignee | ||
Comment 3•2 years ago
|
||
Pushed by gsvelto@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2cc1d718e530
Remove breakpad's dump_syms from the macOS system symbols scrapers and use the Rust one instead r=glandium
Comment 5•2 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
status-firefox106:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 106 Branch
Comment hidden (Intermittent Failures Robot) |
You need to log in
before you can comment on or make changes to this bug.
Description
•