Skip to content

Commit

Permalink
go-module: workaround MissingUnpackerDep QA false positive
Browse files Browse the repository at this point in the history
The go.sum behavior in this eclass triggers a false positive from
pkgcheck, specifically:
MissingUnpackerDep: version ...: missing BDEPEND="app-arch/unzip"

The eclass & package do NOT invoke any direct unpacking of the zipfiles,
and leave that entirely to Go's builtin code, making app-arch/unzip
completely unneeded.

Adding packages using the new eclass functionality will spam the above
MissingUnpackerDep warning, so workaround it for now, until pkgcheck/QA
can not give the false positive.

Reference: pkgcore/pkgcheck#214
Signed-off-by: Robin H. Johnson <[email protected]>
  • Loading branch information
robbat2 committed Mar 7, 2020
1 parent 6158c84 commit 1b302a4
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions eclass/go-module.eclass
Original file line number Diff line number Diff line change
Expand Up @@ -57,6 +57,12 @@ _GO_MODULE=1

BDEPEND=">=dev-lang/go-1.12"

# Workaround for pkgcheck false positive: https://github.com/pkgcore/pkgcheck/issues/214
# MissingUnpackerDep: version ...: missing BDEPEND="app-arch/unzip"
# Added here rather than to each affected package, so it can be cleaned up just
# once when pkgcheck is improved.
BDEPEND+=" app-arch/unzip"

# Force go to build in module mode.
# In this mode the GOPATH environment variable is ignored.
# this will become the default in the future.
Expand Down

0 comments on commit 1b302a4

Please sign in to comment.