commit:     43a573c4850f15bd97b15db5b481eebf9c4d9f28
Author:     James Le Cuirot <chewi <AT> gentoo <DOT> org>
AuthorDate: Mon Oct 12 21:08:24 2020 +0000
Commit:     James Le Cuirot <chewi <AT> gentoo <DOT> org>
CommitDate: Mon Oct 12 21:08:24 2020 +0000
URL:        https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=43a573c4

media-video/makemkv: Patch around missing quotes in configure

I have reported this upstream and I believe they will use a proper
autoconf macro to set the -std flag in future.

Closes: https://bugs.gentoo.org/746368
Suggested-By: Matt Whitlock
Package-Manager: Portage-3.0.8, Repoman-3.0.1
Signed-off-by: James Le Cuirot <chewi <AT> gentoo.org>

 .../makemkv/files/1.15.3-configure-CXXFLAGS-quoting.patch    | 12 ++++++++++++
 media-video/makemkv/makemkv-1.15.3.ebuild                    |  6 +++++-
 2 files changed, 17 insertions(+), 1 deletion(-)

diff --git a/media-video/makemkv/files/1.15.3-configure-CXXFLAGS-quoting.patch 
b/media-video/makemkv/files/1.15.3-configure-CXXFLAGS-quoting.patch
new file mode 100644
index 00000000000..3dfe508a24a
--- /dev/null
+++ b/media-video/makemkv/files/1.15.3-configure-CXXFLAGS-quoting.patch
@@ -0,0 +1,12 @@
+diff -Naur makemkv-oss-1.15.3~/configure makemkv-oss-1.15.3/configure
+--- makemkv-oss-1.15.3~/configure      2020-10-01 13:59:27.000000000 +0000
++++ makemkv-oss-1.15.3/configure       2020-10-03 20:49:45.588257151 +0000
+@@ -11726,7 +11726,7 @@
+ ENABLE_DEBUG=$enable_debug
+ 
+ 
+-if test -z $CXXFLAGS; then
++if test -z "$CXXFLAGS"; then
+     CXXFLAGS='-std=c++11'
+ fi
+ 

diff --git a/media-video/makemkv/makemkv-1.15.3.ebuild 
b/media-video/makemkv/makemkv-1.15.3.ebuild
index 4d94806d84d..3c81c3d3988 100644
--- a/media-video/makemkv/makemkv-1.15.3.ebuild
+++ b/media-video/makemkv/makemkv-1.15.3.ebuild
@@ -47,7 +47,11 @@ BDEPEND="
 
 CONFIG_CHECK="~CHR_DEV_SG"
 S="${WORKDIR}/${MY_P}"
-PATCHES=( "${FILESDIR}"/${PN}-path.patch )
+
+PATCHES=(
+       "${FILESDIR}"/${PN}-path.patch
+       "${FILESDIR}"/${PV}-configure-CXXFLAGS-quoting.patch
+)
 
 src_configure() {
        # See bug #439380.

Reply via email to