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)

task

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)

Blocks: 1785029
Flags: needinfo?(gsvelto)

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)
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
Blocks: 1709543
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 106 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: