Gentoo Archives: gentoo-portage-dev

From: Mike Frysinger <vapier@g.o>
To: gentoo-portage-dev@l.g.o
Subject: [gentoo-portage-dev] [PATCH] ebuild: unset all funcs/vars that start with ___
Date: Fri, 13 Nov 2015 01:55:48
Message-Id: 1447379740-6136-1-git-send-email-vapier@gentoo.org
1 Since the __* (two) namespace is reserved, and ___* (three) has rarely
2 (if ever) been used in ebuilds, we can nuke all funcs/vars that start
3 with that. It makes clean up easier for us.
4 ---
5 bin/save-ebuild-env.sh | 4 +++-
6 1 file changed, 3 insertions(+), 1 deletion(-)
7
8 diff --git a/bin/save-ebuild-env.sh b/bin/save-ebuild-env.sh
9 index 31c2d60..ddef1fd 100644
10 --- a/bin/save-ebuild-env.sh
11 +++ b/bin/save-ebuild-env.sh
12 @@ -89,7 +89,9 @@ __save_ebuild_env() {
13 ___eapi_has_package_manager_build_user && unset -f package_manager_build_user
14 ___eapi_has_package_manager_build_group && unset -f package_manager_build_group
15
16 - unset -f $(compgen -A function ___eapi_)
17 + # Clear out the triple underscore namespace as it is reserved by the PM.
18 + unset -f $(compgen -A function ___)
19 + unset ${!___*}
20
21 # portage config variables and variables set directly by portage
22 unset ACCEPT_LICENSE BAD BRACKET BUILD_PREFIX COLS \
23 --
24 2.6.2

Replies