commit:     ccc69bc760de2a0cba00f6123349dc4f380b4abb
Author:     Sergei Trofimovich <slyfox <AT> gentoo <DOT> org>
AuthorDate: Tue Jul  3 21:51:16 2018 +0000
Commit:     Sergei Trofimovich <slyfox <AT> gentoo <DOT> org>
CommitDate: Tue Jul  3 21:51:53 2018 +0000
URL:        https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=ccc69bc7

dev-lang/squirrel: declare used CMake variables, bug #659880

portage's QA check complains:
> One or more CMake variables were not used by the project:
>  DISABLE_STATIC
>  LONG_OUTPUT_NAMES

The warning is misleading. Variables are used but not declared
as 'options()'s. This change tweaks the declaration.

Reported-by: Toralf Förster
Closes: https://bugs.gentoo.org/659880
Package-Manager: Portage-2.3.41, Repoman-2.3.9

 dev-lang/squirrel/files/squirrel-3.1-declare-options.patch | 10 ++++++++++
 dev-lang/squirrel/squirrel-3.1.ebuild                      |  6 +++++-
 2 files changed, 15 insertions(+), 1 deletion(-)

diff --git a/dev-lang/squirrel/files/squirrel-3.1-declare-options.patch 
b/dev-lang/squirrel/files/squirrel-3.1-declare-options.patch
new file mode 100644
index 00000000000..11bdbf107ce
--- /dev/null
+++ b/dev-lang/squirrel/files/squirrel-3.1-declare-options.patch
@@ -0,0 +1,10 @@
+Explicitly declare used CMake options
+
+Bug: https://bugs.gentoo.org/659880
+--- a/CMakeLists.txt
++++ b/CMakeLists.txt
+@@ -1 +1,4 @@
++option(DISABLE_STATIC "Avoid building/installing static libraries.")
++option(LONG_OUTPUT_NAMES "Use longer names for binaries and libraries: 
squirrel3 (not sq).")
++
+ if(MSVC)

diff --git a/dev-lang/squirrel/squirrel-3.1.ebuild 
b/dev-lang/squirrel/squirrel-3.1.ebuild
index 4a149398eb4..761fd5c1da2 100644
--- a/dev-lang/squirrel/squirrel-3.1.ebuild
+++ b/dev-lang/squirrel/squirrel-3.1.ebuild
@@ -1,4 +1,4 @@
-# Copyright 1999-2017 Gentoo Foundation
+# Copyright 1999-2018 Gentoo Foundation
 # Distributed under the terms of the GNU General Public License v2
 
 EAPI=6
@@ -17,6 +17,10 @@ IUSE="examples static-libs"
 RDEPEND=""
 DEPEND="${RDEPEND}"
 
+PATCHES=(
+       "${FILESDIR}"/${P}-declare-options.patch
+)
+
 src_configure() {
        local mycmakeargs=(
                -DINSTALL_LIB_DIR="$(get_libdir)" \

Reply via email to