bryson/pacman
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

libmakepkg: only save the shellopts we need

Browse source 
micro-optimization: We only care about temporarily enforcing extglob, so
that is the only one we need to explicitly restore.

Signed-off-by: Eli Schwartz <eschwartz@archlinux.org>
Signed-off-by: Allan McRae <allan@archlinux.org>
This commit is contained in:
Eli Schwartz 2018-05-02 14:24:44 -04:00 • committed by Allan McRae
parent 2c94118dc1
commit 542c3cf812
7 changed files with 13 additions and 13 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
scripts/libmakepkg/lint_pkgbuild/checkdepends.sh.in
View file

@ -37,8 +37,8 @@ lint_checkdepends() {
get_pkgbuild_all_split_attributes checkdepends checkdepends_list
# save our shell options and turn on extglob
local shellopts=$(shopt -p)
# this function requires extglob - save current status to restore later
local shellopts=$(shopt -p extglob)
shopt -s extglob
for checkdepend in "${checkdepends_list[@]}"; do

4
scripts/libmakepkg/lint_pkgbuild/conflicts.sh.in
View file

@ -37,8 +37,8 @@ lint_conflicts() {
get_pkgbuild_all_split_attributes conflicts conflicts_list
# save our shell options and turn on extglob
local shellopts=$(shopt -p)
# this function requires extglob - save current status to restore later
local shellopts=$(shopt -p extglob)
shopt -s extglob
for conflict in "${conflicts_list[@]}"; do

4
scripts/libmakepkg/lint_pkgbuild/depends.sh.in
View file

@ -37,8 +37,8 @@ lint_depends() {
get_pkgbuild_all_split_attributes depends depends_list
# save our shell options and turn on extglob
local shellopts=$(shopt -p)
# this function requires extglob - save current status to restore later
local shellopts=$(shopt -p extglob)
shopt -s extglob
for depend in "${depends_list[@]}"; do

4
scripts/libmakepkg/lint_pkgbuild/makedepends.sh.in
View file

@ -37,8 +37,8 @@ lint_makedepends() {
get_pkgbuild_all_split_attributes makedepends makedepends_list
# save our shell options and turn on extglob
local shellopts=$(shopt -p)
# this function requires extglob - save current status to restore later
local shellopts=$(shopt -p extglob)
shopt -s extglob
for makedepend in "${makedepends_list[@]}"; do

4
scripts/libmakepkg/util/pkgbuild.sh.in
View file

@ -80,8 +80,8 @@ extract_function_variable() {
printf -v attr_regex '^[[:space:]]* %s\+?=[^(]' "$2"
fi
# save our shell options and turn on extglob
local shellopts=$(shopt -p)
# this function requires extglob - save current status to restore later
local shellopts=$(shopt -p extglob)
shopt -s extglob
while read -r; do

4
scripts/libmakepkg/util/util.sh.in
View file

@ -42,8 +42,8 @@ is_array() {
local v=$1
local ret=1
# this function requires extglob - save current options to restore later
local shellopts=$(shopt -p)
# this function requires extglob - save current status to restore later
local shellopts=$(shopt -p extglob)
shopt -s extglob
if [[ $(declare -p "$v") == declare\ -*([[:alnum:]])a*([[:alnum:]])\ * ]]; then

2
scripts/makepkg.sh.in
View file

@ -275,7 +275,7 @@ handle_deps() {
# we might need the new system environment
# save our shell options and turn off extglob
local shellopts=$(shopt -p)
local shellopts=$(shopt -p extglob)
shopt -u extglob
source /etc/profile &>/dev/null
eval "$shellopts"