Gentoo Archives: gentoo-portage-dev

From: Mike Frysinger <vapier@g.o>
To: gentoo-portage-dev@l.g.o
Subject: Re: [gentoo-portage-dev] [PATCH] ebuild: set up bash compat levels
Date: Wed, 11 Nov 2015 21:11:17
Message-Id: 20151111211114.GF5154@vapier.lan
In Reply to: Re: [gentoo-portage-dev] [PATCH] ebuild: set up bash compat levels by Zac Medico
1 On 11 Nov 2015 13:04, Zac Medico wrote:
2 > On 11/11/2015 12:55 PM, Mike Frysinger wrote:
3 > > On 11 Nov 2015 11:42, Zac Medico wrote:
4 > >> On 11/10/2015 08:39 PM, Mike Frysinger wrote:
5 > >>> +# Set up the bash version compatibility level.
6 > >>> +__check_bash_version() {
7 > >>
8 > >> Please unset all new internal function inside bin/save-ebuild-env.sh.
9 > >> Note that it already uses this line to unset functions beginning with
10 > >> ___eapi:
11 > >>
12 > >> unset -f $(compgen -A function ___eapi_)
13 > >
14 > > why don't we create a new namespace for portage funcs ? something like __e* ?
15 >
16 > That works for me. According to PMS, we're free to do anything we want
17 > as long as it begins with at least 2 underscores.
18
19 interesting. why don't we just unmap all things that begin with 2 underscores
20 then ? or maybe 3 ?
21 -mike

Attachments

File name MIME type
signature.asc application/pgp-signature

Replies