Re: [gentoo-dev] [PATCH] vala.eclass: make has_version aware of ROOT for EAPI 7

2021-01-06 Thread Ulrich Mueller
> On Thu, 07 Jan 2021, Matt Turner wrote: > + has_version $([[ $EAPI == [1-6] ]] || echo -b) > "dev-lang/vala:${v}${u}" && echo "${v}" && return > + has_version $([[ $EAPI == [1-6] ]] || echo -b) > "dev-lang/vala:${version}" || die "No installed vala:${version}" Cur

[gentoo-dev] [PATCH] vala.eclass: make has_version aware of ROOT for EAPI 7

2021-01-06 Thread Matt Turner
From: David Michael The vala dependencies are declared in BDEPEND since EAPI 7 so that the valac command is natively executable. With no arguments, the has_version function would look for a cross-compiled vala package in the target ROOT and always fail. Signed-off-by: David Michael Signed-off-

Re: [gentoo-dev] [PATCH] systemd.eclass: remove SYSROOT from pkg-config output

2021-01-06 Thread Mike Gilbert
On Wed, Jan 6, 2021 at 5:47 PM James Le Cuirot wrote: > > On Mon, 4 Jan 2021 19:28:37 -0500 > Mike Gilbert wrote: > > > On Mon, Jan 4, 2021 at 6:45 PM Mike Gilbert wrote: > > > > > > On Mon, Jan 4, 2021 at 6:18 PM James Le Cuirot wrote: > > > > $ PKG_CONFIG_SYSROOT_DIR=/foo pkg-config --variabl

Re: [gentoo-dev] [PATCH] systemd.eclass: remove SYSROOT from pkg-config output

2021-01-06 Thread James Le Cuirot
On Mon, 4 Jan 2021 19:28:37 -0500 Mike Gilbert wrote: > On Mon, Jan 4, 2021 at 6:45 PM Mike Gilbert wrote: > > > > On Mon, Jan 4, 2021 at 6:18 PM James Le Cuirot wrote: > > > $ PKG_CONFIG_SYSROOT_DIR=/foo pkg-config --variable=udevdir udev > > > /lib/udev > > > > > > The udevdir variable is n

[gentoo-dev] Last-rites: sys-power/pm-utils, sys-power/pm-quirks, app-admin/cgmanager and sys-libs/libnih

2021-01-06 Thread Andreas Sturmlechner
# Andreas Sturmlechner (2021-01-06) # Post-consolekit cleanup, dead upstream since 2016. # https://s3hh.wordpress.com/2016/06/18/whither-cgmanager/ # Masked for removal in 30 days. app-admin/cgmanager sys-libs/libnih # Andreas Sturmlechner (2021-01-06) # Abandoned upstream, countless bugs. Remov

Re: [gentoo-dev] [PATCH 1/2] acct-user.eclass: Support ACCT_USER_ID override

2021-01-06 Thread Ulrich Mueller
> On Wed, 06 Jan 2021, Michał Górny wrote: > On Wed, 2021-01-06 at 14:25 +0100, Ulrich Mueller wrote: >> I wonder about this line. Both hyphen and underscore are valid >> characters in user names. >> >> So, ACCT_USER_FOO_BAR_ID would override the id for both foo_bar and >> foo-bar users. > I

Re: [gentoo-dev] [PATCH 1/2] acct-user.eclass: Support ACCT_USER_ID override

2021-01-06 Thread Thomas Deutschmann
Hi, On 2021-01-06 20:05, Patrick McLean wrote: This is so ACCT_USER_$foo can be set in make.conf, and not have to be specified as an environment variable whenever portage is run. This helps when automated systems are building Gentoo images or systems. Please see my reply to Alec for more deta

Re: [gentoo-dev] [PATCH 1/2] acct-user.eclass: Support ACCT_USER_ID override

2021-01-06 Thread Thomas Deutschmann
On 2021-01-06 20:12, Alec Warner wrote: Not sure I follow. Whether your automation sets a variable in /etc/portage/make.conf or /etc/portage/package.env; it's basically the same problem space; no? No. Assuming we will always stick to same variables, ACCT_USER_ID ACCT_USER_GROUPS ACCT_USER_SHE

Re: [gentoo-dev] [PATCH 1/2] acct-user.eclass: Support ACCT_USER_ID override

2021-01-06 Thread Alec Warner
On Wed, Jan 6, 2021 at 11:05 AM Patrick McLean wrote: > > On Wed, 6 Jan 2021 15:02:12 +0100 > Thomas Deutschmann wrote: > > > Hi, > > > > is there a specific reason why we want to support dynamic variables > > (ACCT_USER_$foo) at all? > > > > Isn't package.env support enough, i.e. use ACCT_USER_I

Re: [gentoo-dev] [PATCH 1/2] acct-user.eclass: Support ACCT_USER_ID override

2021-01-06 Thread Michał Górny
On Wed, 2021-01-06 at 14:25 +0100, Ulrich Mueller wrote: > > > > > > On Tue, 05 Jan 2021, Michał Górny wrote: > > > + # check for the override > > + local override_name=${ACCT_USER_NAME^^} > > + local override_var=ACCT_USER_${override_name//-/_}_ID > > I wonder about this line. Both hyphen

Re: [gentoo-dev] [PATCH 1/2] acct-user.eclass: Support ACCT_USER_ID override

2021-01-06 Thread Patrick McLean
On Wed, 6 Jan 2021 15:02:12 +0100 Thomas Deutschmann wrote: > Hi, > > is there a specific reason why we want to support dynamic variables > (ACCT_USER_$foo) at all? > > Isn't package.env support enough, i.e. use ACCT_USER_ID from environment > if set (which we should detect and log, maybe thi

[gentoo-dev] Towards getting GCC 10 stable

2021-01-06 Thread Andreas K. Huettel
Hey all, it would be great if we could get gcc-10 stable at some point in the near future... For that purpose we now have a new tracker: https://bugs.gentoo.org/762907 ("gcc-10-stable", depends on 127 open bugs). Please have a look if you can help anywhere with the blocking bugs. Thanks a lo

Re: [gentoo-dev] [PATCH 1/2] acct-user.eclass: Support ACCT_USER_ID override

2021-01-06 Thread Thomas Deutschmann
Hi, is there a specific reason why we want to support dynamic variables (ACCT_USER_$foo) at all? Isn't package.env support enough, i.e. use ACCT_USER_ID from environment if set (which we should detect and log, maybe this will require a different namespace for the variables at all to be able

Re: [gentoo-dev] [PATCH 1/2] acct-user.eclass: Support ACCT_USER_ID override

2021-01-06 Thread Ulrich Mueller
> On Tue, 05 Jan 2021, Michał Górny wrote: > + # check for the override > + local override_name=${ACCT_USER_NAME^^} > + local override_var=ACCT_USER_${override_name//-/_}_ID I wonder about this line. Both hyphen and underscore are valid characters in user names. So, ACCT_USER_FOO