Summary: | [GTK][jhbuild] OpenWebRTC build fails with GCC 6.0 | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Joanmarie Diggs <jdiggs> | ||||||||
Component: | Tools / Tests | Assignee: | Joanmarie Diggs <jdiggs> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | alex, commit-queue, lforschler, mcatanzaro, pnormand | ||||||||
Priority: | P2 | ||||||||||
Version: | WebKit Nightly Build | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
Attachments: |
|
Description
Joanmarie Diggs
2016-02-29 15:47:44 PST
Created attachment 272526 [details]
Patch
Comment on attachment 272526 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=272526&action=review > Tools/gtk/jhbuild.modules:463 > <makeargs value="CFLAGS+=-Wno-error"/> Instead of adding two patches, let's just remove the condition here so that we use -Wno-error always. Otherwise we're just going to need to patch this again due to whatever new compiler warning GCC or Clang introduces next year. Actually, nowadays jhbuild passes -Wno-error to every module automatically, we can maybe just upgrade jhbuild. Joanie, since you have F24 already, want to test? In Tools/jhbuild/jhbuild-wrapper, just change the jhbuild_revision line to something new (here I pick the latest commit, it might work) and remove the obsolete comment: jhbuild_revision = '7a8490efbf497d04cfc244a2af0fc7361ad8dd17' Then you can get rid of the manual -Wno-error passing in jhbuild.modules. Created attachment 273200 [details]
Patch
Comment on attachment 273200 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=273200&action=review > Tools/gtk/jhbuild-webrtc.modules:57 > + <makeargs value="OBJCFLAGS+=-Wno-error"/> Ah, I didn't realize GStreamer depends on Objective C on Mac... in that case, you can get rid of these and set in in jhbuildrc like you did CFLAGS and CXXFLAGS. Probably with a comment to indicate the GStreamer dependency, else someone is liable to remove it cleaning up in the future. > Tools/gtk/jhbuildrc:39 > +# Motivated by build failures when building dependencies with GCC 6.0. I wouldn't leave this comment as it's not really related to any particular version of GCC; any new compiler will add new warnings, and if we use -Werror then that will cause the build to fail unnecessarily, it's clear enough we don't want that to happen. Created attachment 273220 [details]
Patch for landing
Comment on attachment 273220 [details] Patch for landing Clearing flags on attachment: 273220 Committed r197713: <http://trac.webkit.org/changeset/197713> All reviewed patches have been landed. Closing bug. |