Closed Bug 1666108 Opened 1 year ago Closed 1 year ago

GCC9 cc1plus: warning: ‘-Werror=’ argument ‘-Werror=implicit-function-declaration’ is not valid for C++

Categories

(Firefox Build System :: General, defect)

x86_64
Linux
defect

Tracking

(firefox84 fixed)

RESOLVED FIXED
84 Branch
Tracking Status
firefox84 --- fixed

People

(Reporter: smurfd, Assigned: sfink)

References

(Blocks 1 open bug, Regression)

Details

(Keywords: regression)

Attachments

(1 file)

Hey

So this below is my mozconfig file

export CC="/usr/bin/gcc-9"
export CXX="/usr/bin/g++-9"

mk_add_options PYTHON=/usr/bin/python3.6

ac_add_options --disable-elf-hack

# Debug builds
ac_add_options --enable-debug
ac_add_options --enable-debug-symbols
# Stricter builds
ac_add_options --enable-warnings-as-errors

# To fix _FORTIFY_SOURCE issues
ac_add_options --disable-hardening
#ac_add_options --enable-optimize

Steps to reproduce : Build with GCC9 (atleast on linux)
Actual results :
I receive ALOT(about 1600 warnings of a fresh m-c build) of these warnings :
cc1plus: warning: ‘-Werror=’ argument ‘-Werror=implicit-function-declaration’ is not valid for C++

Expected results : No warnings
Workaround : Seems that if i change to GCC8 i will not receive these warnings.

note: it breaks the build with --enable-warnings-as-errors
To make it build, I am using this ugly workaround:

sed -i -e "s|-Wno-unreachable-code-return||g"  security/sandbox/linux/moz.build 

As it is a bug in gcc, it doesn't break firefox (it is still possible to build it), I will just close this bug.

Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → WONTFIX

The hazard build logs are getting spammed with this, so I would prefer to fix it.

Status: RESOLVED → REOPENED
Resolution: WONTFIX → ---
Assignee: nobody → sphink
Status: REOPENED → ASSIGNED
Pushed by sfink@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/6ed74fab9add
Eliminate -Wimplicit-function-declaration warnings r=firefox-build-system-reviewers,glandium
Status: ASSIGNED → RESOLVED
Closed: 1 year ago1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 84 Branch
You need to log in before you can comment on or make changes to this bug.