Re: [gentoo-dev] [PATCH v2 4/4] dev-vcs/cli: new package

2020-02-18 Thread Robin H. Johnson
On Wed, Feb 19, 2020 at 12:18:24AM -0600, William Hubbs wrote: > > +RDEPEND+=">=dev-vcs/git-1.7.3" > > +BDEPEND+=">=dev-lang/go-1.13" > > +GOPATH="${WORKDIR}" > You don't need += here I've taken to += as a cleaner variant than xDEPEND="${xDEPEND} cat/pn" when the eclass MAY be setting some

Re: [gentoo-dev] [PATCH v2 3/4] app-admin/kube-bench: convert to go-module go.sum

2020-02-18 Thread Robin H. Johnson
(WilliamH: apologies for duplicate, I replied directly to you instead of the list the first time around) On Wed, Feb 19, 2020 at 12:10:46AM -0600, William Hubbs wrote: > > +EGO_PN=github.com/aquasecurity/kube-bench > You shouldn't need EGO_PN Copied from th previous -r0. > > >

Re: [gentoo-dev] [PATCH v2 1/4] eclass/go-module: add support for building based on go.sum

2020-02-18 Thread Robin H. Johnson
On Tue, Feb 18, 2020 at 11:46:45PM -0600, William Hubbs wrote: > > -# If it does not have a vendor directory, you should use the EGO_VENDOR > > +# Alternatively, older versions of this eclass used the EGO_VENDOR > > # variable and the go-module_vendor_uris function as shown in the > > # example

Re: [gentoo-dev] [PATCH v2 4/4] dev-vcs/cli: new package

2020-02-18 Thread William Hubbs
On Mon, Feb 17, 2020 at 01:22:32AM -0800, Robin H. Johnson wrote: > Package-Manager: Portage-2.3.84, Repoman-2.3.18 > Signed-off-by: Robin H. Johnson > --- > dev-vcs/cli/Manifest | 137 +++ > dev-vcs/cli/cli-0.5.5.ebuild | 177 +++ >

Re: [gentoo-dev] [PATCH v2 3/4] app-admin/kube-bench: convert to go-module go.sum

2020-02-18 Thread William Hubbs
On Mon, Feb 17, 2020 at 01:22:31AM -0800, Robin H. Johnson wrote: > Signed-off-by: Robin H. Johnson > --- > app-admin/kube-bench/Manifest | 351 > .../kube-bench/kube-bench-0.2.3-r1.ebuild | 394 ++ > 2 files changed, 745 insertions(+) >

Re: [gentoo-dev] [PATCH v2 2/4] dev-go/go-tour: convert to go-module go.sum

2020-02-18 Thread William Hubbs
This ebuild isn't fully convirted, so it probably isn't the best example. My comments are just a couple of aspects of it. On Mon, Feb 17, 2020 at 01:22:30AM -0800, Robin H. Johnson wrote: > Signed-off-by: Robin H. Johnson > --- > dev-go/go-tour/Manifest | 7 ++ >

Re: [gentoo-dev] [PATCH v2 1/4] eclass/go-module: add support for building based on go.sum

2020-02-18 Thread William Hubbs
On Mon, Feb 17, 2020 at 01:22:29AM -0800, Robin H. Johnson wrote: > EGO_SUM mode now supplements the existing EGO_VENDOR mode. > > EGO_SUM should be populated by the maintainer, directly from the go.sum > file of the root package. See eclass and conversion examples for further > details:

Re: [gentoo-dev] [PATCH 3/3] app-admin/kube-bench: convert to go-module go.sum

2020-02-18 Thread desultory
On 02/18/20 02:36, Matt Turner wrote: > On Mon, Feb 17, 2020 at 11:47 PM desultory wrote: >> > > You've got a particular knack for this kind of argumentative nonsense. > > While I will gladly accept that post being described as "argumentative", as after all I am very much interested in reading

Re: [gentoo-dev] [PATCH] eclass/acct-user.eclass: disable pkg_* on Prefix.

2020-02-18 Thread Benda Xu
Michael 'veremitz' Everitt writes: > Peanut gallery says 'ACK' +1 Thank you veremitz. Let's see :) Benda signature.asc Description: PGP signature

Re: [gentoo-dev] [Policy change] Package masking of live ebuilds

2020-02-18 Thread Michael 'veremitz' Everitt
On 18/02/20 19:52, Ulrich Mueller wrote: > The devmanual says about live ebuilds: > > | CVS ebuilds must be either with empty KEYWORDS or package.masked > | (but not both). Empty KEYWORDS are strongly preferred. This applies > | to "live" ebuilds (-) and to ebuilds that extract a static > |

Re: [gentoo-dev] [Policy change] Package masking of live ebuilds

2020-02-18 Thread William Hubbs
On Tue, Feb 18, 2020 at 08:52:59PM +0100, Ulrich Mueller wrote: > The devmanual says about live ebuilds: > > | CVS ebuilds must be either with empty KEYWORDS or package.masked > | (but not both). Empty KEYWORDS are strongly preferred. This applies > | to "live" ebuilds (-) and to ebuilds that

[gentoo-dev] [Policy change] Package masking of live ebuilds

2020-02-18 Thread Ulrich Mueller
The devmanual says about live ebuilds: | CVS ebuilds must be either with empty KEYWORDS or package.masked | (but not both). Empty KEYWORDS are strongly preferred. This applies | to "live" ebuilds (-) and to ebuilds that extract a static | revision but still use CVS for fetching. As of today,

Re: [gentoo-dev] [PATCH] eclass/acct-user.eclass: disable pkg_* on Prefix.

2020-02-18 Thread Michael 'veremitz' Everitt
On 18/02/20 13:02, hero...@gentoo.org wrote: > From: Benda Xu > > Gentoo Prefix runs with a normal user and cannot manage any other user. > Exit gracefully with a message. > > Closes: https://bugs.gentoo.org/709570 > Signed-off-by: Benda Xu > --- > eclass/acct-user.eclass | 10 ++ >

[gentoo-dev] [PATCH] eclass/acct-user.eclass: disable pkg_* on Prefix.

2020-02-18 Thread heroxbd
From: Benda Xu Gentoo Prefix runs with a normal user and cannot manage any other user. Exit gracefully with a message. Closes: https://bugs.gentoo.org/709570 Signed-off-by: Benda Xu --- eclass/acct-user.eclass | 10 ++ 1 file changed, 10 insertions(+) diff --git