Gentoo Archives: gentoo-commits

From: "Robin H. Johnson" <robbat2@g.o>
To: gentoo-commits@l.g.o
Subject: [gentoo-commits] repo/gentoo:master commit in: eclass/
Date: Sat, 07 Mar 2020 20:02:19
Message-Id: 1583611330.1b302a4d9361b070f0ace9547b1faa5b0cf5da89.robbat2@gentoo
1 commit: 1b302a4d9361b070f0ace9547b1faa5b0cf5da89
2 Author: Robin H. Johnson <robbat2 <AT> gentoo <DOT> org>
3 AuthorDate: Sat Mar 7 19:54:46 2020 +0000
4 Commit: Robin H. Johnson <robbat2 <AT> gentoo <DOT> org>
5 CommitDate: Sat Mar 7 20:02:10 2020 +0000
6 URL: https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=1b302a4d
7
8 go-module: workaround MissingUnpackerDep QA false positive
9
10 The go.sum behavior in this eclass triggers a false positive from
11 pkgcheck, specifically:
12 MissingUnpackerDep: version ...: missing BDEPEND="app-arch/unzip"
13
14 The eclass & package do NOT invoke any direct unpacking of the zipfiles,
15 and leave that entirely to Go's builtin code, making app-arch/unzip
16 completely unneeded.
17
18 Adding packages using the new eclass functionality will spam the above
19 MissingUnpackerDep warning, so workaround it for now, until pkgcheck/QA
20 can not give the false positive.
21
22 Reference: https://github.com/pkgcore/pkgcheck/issues/214
23 Signed-off-by: Robin H. Johnson <robbat2 <AT> gentoo.org>
24
25 eclass/go-module.eclass | 6 ++++++
26 1 file changed, 6 insertions(+)
27
28 diff --git a/eclass/go-module.eclass b/eclass/go-module.eclass
29 index 08c123873c4..cdd5b07a930 100644
30 --- a/eclass/go-module.eclass
31 +++ b/eclass/go-module.eclass
32 @@ -57,6 +57,12 @@ _GO_MODULE=1
33
34 BDEPEND=">=dev-lang/go-1.12"
35
36 +# Workaround for pkgcheck false positive: https://github.com/pkgcore/pkgcheck/issues/214
37 +# MissingUnpackerDep: version ...: missing BDEPEND="app-arch/unzip"
38 +# Added here rather than to each affected package, so it can be cleaned up just
39 +# once when pkgcheck is improved.
40 +BDEPEND+=" app-arch/unzip"
41 +
42 # Force go to build in module mode.
43 # In this mode the GOPATH environment variable is ignored.
44 # this will become the default in the future.