Gentoo Archives: gentoo-commits

From: Andreas Sturmlechner <asturm@g.o>
To: gentoo-commits@l.g.o
Subject: [gentoo-commits] repo/gentoo:master commit in: media-libs/musicbrainz/files/, media-libs/musicbrainz/
Date: Thu, 04 Jun 2020 20:07:47
Message-Id: 1591301242.2e3353efaacf80e2ac04c3ef59bc91db4fe62937.asturm@gentoo
1 commit: 2e3353efaacf80e2ac04c3ef59bc91db4fe62937
2 Author: William Breathitt Gray <vilhelm.gray <AT> gmail <DOT> com>
3 AuthorDate: Fri May 1 18:11:10 2020 +0000
4 Commit: Andreas Sturmlechner <asturm <AT> gentoo <DOT> org>
5 CommitDate: Thu Jun 4 20:07:22 2020 +0000
6 URL: https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=2e3353ef
7
8 media-libs/musicbrainz: Fix CMake error
9
10 This fixes a build error caused by improper use of wildcards in the
11 project's src/CMakelists.txt file.
12
13 Closes: https://bugs.gentoo.org/698090
14 Signed-off-by: William Breathitt Gray <vilhelm.gray <AT> gmail.com>
15 Closes: https://github.com/gentoo/gentoo/pull/15600
16 Signed-off-by: Andreas Sturmlechner <asturm <AT> gentoo.org>
17
18 .../files/musicbrainz-5.1.0-no-wildcards.patch | 27 ++++++++++++++++++++++
19 media-libs/musicbrainz/musicbrainz-5.1.0.ebuild | 2 ++
20 2 files changed, 29 insertions(+)
21
22 diff --git a/media-libs/musicbrainz/files/musicbrainz-5.1.0-no-wildcards.patch b/media-libs/musicbrainz/files/musicbrainz-5.1.0-no-wildcards.patch
23 new file mode 100644
24 index 00000000000..d0bf0868d54
25 --- /dev/null
26 +++ b/media-libs/musicbrainz/files/musicbrainz-5.1.0-no-wildcards.patch
27 @@ -0,0 +1,27 @@
28 +From 36262d60fe92fe7a2c9bfb40e736bfcd29a6c3bd Mon Sep 17 00:00:00 2001
29 +From: Abderrahim Kitouni <a.kitouni@×××××.com>
30 +Date: Fri, 13 Apr 2018 09:56:57 +0100
31 +Subject: [PATCH 2/2] src/CMakelists.txt: do not use wildcards for dependencies
32 +
33 +This is discouraged by cmake's documentation and doesn't work with the ninja generator.
34 +---
35 + src/CMakeLists.txt | 3 ++-
36 + 1 file changed, 2 insertions(+), 1 deletion(-)
37 +
38 +diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt
39 +index 68c04e3..f7439d3 100644
40 +--- a/src/CMakeLists.txt
41 ++++ b/src/CMakeLists.txt
42 +@@ -37,7 +37,8 @@ ADD_CUSTOM_COMMAND(
43 + OUTPUT ${CMAKE_CURRENT_BINARY_DIR}/mb5_c.cc ${CMAKE_CURRENT_BINARY_DIR}/mb5_c.h ${CMAKE_CURRENT_BINARY_DIR}/../include/musicbrainz5/mb5_c.h
44 + COMMAND make-c-interface ${CMAKE_CURRENT_SOURCE_DIR} cinterface.xml ${CMAKE_CURRENT_BINARY_DIR} mb5_c.cc mb5_c.h
45 + COMMAND ${CMAKE_COMMAND} -E copy_if_different ${CMAKE_CURRENT_BINARY_DIR}/mb5_c.h ${CMAKE_CURRENT_BINARY_DIR}/../include/musicbrainz5/mb5_c.h
46 +- DEPENDS make-c-interface cinterface.xml *.inc
47 ++ DEPENDS make-c-interface cinterface.xml c-int-medium-defines.inc c-int-query-source.inc c-int-source-funcs.inc
48 ++ c-int-medium-source.inc c-int-release-defines.inc c-int-query-defines.inc c-int-release-source.inc
49 + )
50 +
51 + ADD_CUSTOM_TARGET(src_gen DEPENDS mb5_c.h)
52 +--
53 +2.26.2
54 +
55
56 diff --git a/media-libs/musicbrainz/musicbrainz-5.1.0.ebuild b/media-libs/musicbrainz/musicbrainz-5.1.0.ebuild
57 index 3c7debe1451..25b005bc1bd 100644
58 --- a/media-libs/musicbrainz/musicbrainz-5.1.0.ebuild
59 +++ b/media-libs/musicbrainz/musicbrainz-5.1.0.ebuild
60 @@ -25,6 +25,8 @@ DEPEND="${RDEPEND}
61
62 S="${WORKDIR}/lib${P}"
63
64 +PATCHES=( "${FILESDIR}/${P}-no-wildcards.patch" )
65 +
66 src_prepare() {
67 use test || cmake_comment_add_subdirectory tests
68 cmake-utils_src_prepare