media-video/smplayer: drop -Werror to allow gcc-8, bug #654724
authorSergei Trofimovich <slyfox@gentoo.org>
Mon, 7 May 2018 12:18:57 +0000 (13:18 +0100)
committerSergei Trofimovich <slyfox@gentoo.org>
Mon, 7 May 2018 12:19:08 +0000 (13:19 +0100)
gcc-8 covers even more potential buffer overflow conditions.
This causes new build failure due to blanket -Werror in smplayer.

For more details see Gentoo's take on -Werror handling in devmanual:
https://devmanual.gentoo.org/ebuild-writing/common-mistakes/index.html

Reported-by: Mike Lothian
Closes: https://bugs.gentoo.org/654724
Package-Manager: Portage-2.3.36, Repoman-2.3.9

media-video/smplayer/files/smplayer-18.3.0-disable-werror.patch [new file with mode: 0644]
media-video/smplayer/smplayer-18.3.0.ebuild
media-video/smplayer/smplayer-18.4.0.ebuild

diff --git a/media-video/smplayer/files/smplayer-18.3.0-disable-werror.patch b/media-video/smplayer/files/smplayer-18.3.0-disable-werror.patch
new file mode 100644 (file)
index 0000000..7905a44
--- /dev/null
@@ -0,0 +1,9 @@
+gcc-8 is smarter about possible overflow detection.
+
+Reported-by: Mike Lothian
+Bug: https://bugs.gentoo.org/654724
+--- a/webserver/Makefile
++++ b/webserver/Makefile
+@@ -5 +5 @@ SOURCES = main.c mongoose.c
+-CFLAGS = -O2 -W -Wall -Werror -Wno-unused-function $(CFLAGS_EXTRA) $(MODULE_CFLAGS)
++CFLAGS = -O2 -W -Wall -Wno-unused-function $(CFLAGS_EXTRA) $(MODULE_CFLAGS)
index d8ae8f1fe3c6b66de01883e9afbb4086fb6accdb..711f309f6bec5f3ca40443059dc4314184fbe82f 100644 (file)
@@ -48,6 +48,7 @@ PATCHES=(
        "${FILESDIR}/${PN}-14.9.0.6966-unbundle-qtsingleapplication.patch" # bug 487544
        "${FILESDIR}/${PN}-17.1.0-advertisement_crap.patch"
        "${FILESDIR}/${PN}-18.2.0-jobserver.patch"
+       "${FILESDIR}/${PN}-18.3.0-disable-werror.patch"
 )
 
 src_prepare() {
index 27c383e46b9ff98b03313128a35be2e479ff828d..8b62e36f20f221f926503c15c004ed7d03df241d 100644 (file)
@@ -48,6 +48,7 @@ PATCHES=(
        "${FILESDIR}/${PN}-14.9.0.6966-unbundle-qtsingleapplication.patch" # bug 487544
        "${FILESDIR}/${PN}-17.1.0-advertisement_crap.patch"
        "${FILESDIR}/${PN}-18.2.0-jobserver.patch"
+       "${FILESDIR}/${PN}-18.3.0-disable-werror.patch"
 )
 
 src_prepare() {