commit:     b81fb8d368376faefc0d71a15c44d1cfd9d8dab1
Author:     Dustin Smith <smith.dustin2017 <AT> gmail <DOT> com>
AuthorDate: Fri Apr 22 02:22:08 2022 +0000
Commit:     Sam James <sam <AT> gentoo <DOT> org>
CommitDate: Sat Apr 23 01:20:05 2022 +0000
URL:        https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=b81fb8d3

media-libs/opus: fix word split issue in ebuild

shellbugbot points out an issue with shellcheck where $INTRINSIC_FLAGS
won't be split on space because of the double quoting.  If I am reading
the ebuild correctly, then "use" is receiving one big argument
"cpu_flags_x86_sse cpu_flags_arm_neon" in just one iteration.

This commit just removes the double quotes to fix this.

Bug: https://bugs.gentoo.org/839222
Closes: https://github.com/gentoo/gentoo/pull/25147
Signed-off-by: Dustin Smith <smith.dustin2017 <AT> gmail.com>
Signed-off-by: Sam James <sam <AT> gentoo.org>

 media-libs/opus/{opus-1.3.1-r2.ebuild => opus-1.3.1-r3.ebuild} | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/media-libs/opus/opus-1.3.1-r2.ebuild 
b/media-libs/opus/opus-1.3.1-r3.ebuild
similarity index 97%
rename from media-libs/opus/opus-1.3.1-r2.ebuild
rename to media-libs/opus/opus-1.3.1-r3.ebuild
index 10c7e4876985..e250256c7441 100644
--- a/media-libs/opus/opus-1.3.1-r2.ebuild
+++ b/media-libs/opus/opus-1.3.1-r3.ebuild
@@ -37,7 +37,7 @@ multilib_src_configure() {
        # We want to disable intrinsics if no flags are enabled
        # (This is a fun Bash construct to do that!)
        # bug #752069
-       for i in "${INTRINSIC_FLAGS}" ; do
+       for i in ${INTRINSIC_FLAGS} ; do
                use ${i} && myeconfargs+=( --enable-intrinsics ) && break
        done || myeconfargs+=( --disable-intrinsics )
 

Reply via email to