diff options
| author | Levente Polyak <anthraxx@archlinux.org> | 2022-08-31 23:49:01 +0200 |
|---|---|---|
| committer | Levente Polyak <anthraxx@archlinux.org> | 2022-09-01 23:40:21 +0200 |
| commit | fcaf3ecec0f835cc480fdeb653d33754b0de7624 (patch) | |
| tree | bfa2e2bf75385cc041fc36233205ea6a9be35087 /src | |
| parent | 280ef10d37e88a9a60d1a4406fc511d09acc3fad (diff) | |
| download | devtools-fcaf3ecec0f835cc480fdeb653d33754b0de7624.tar.xz | |
fix: do not set extglob globally to avoid side-effects
Instead only enable it for whatever operation requires them.
Example sides effects:
commitpkg can accidently execute PKGBUILD functions when sourcing the
PKGBUILD that has function names like package_libsigc++()
Fixes #87
Diffstat (limited to 'src')
| -rw-r--r-- | src/checkpkg.in | 2 | ||||
| -rw-r--r-- | src/diffpkg.in | 2 | ||||
| -rw-r--r-- | src/find-libdeps.in | 3 |
3 files changed, 2 insertions, 5 deletions
diff --git a/src/checkpkg.in b/src/checkpkg.in index 1a7983c..fe5b71a 100644 --- a/src/checkpkg.in +++ b/src/checkpkg.in @@ -2,8 +2,6 @@ # # SPDX-License-Identifier: GPL-3.0-or-later -shopt -s extglob - m4_include(lib/common.sh) usage() { diff --git a/src/diffpkg.in b/src/diffpkg.in index 43af217..10b0904 100644 --- a/src/diffpkg.in +++ b/src/diffpkg.in @@ -2,8 +2,6 @@ # # SPDX-License-Identifier: GPL-3.0-or-later -shopt -s extglob - m4_include(lib/common.sh) usage() { diff --git a/src/find-libdeps.in b/src/find-libdeps.in index e1423b8..5a11580 100644 --- a/src/find-libdeps.in +++ b/src/find-libdeps.in @@ -5,7 +5,6 @@ m4_include(lib/common.sh) set -e -shopt -s extglob IGNORE_INTERNAL=0 @@ -43,7 +42,9 @@ fi process_sofile() { # extract the library name: libfoo.so + shopt -s extglob nullglob soname="${sofile%.so?(+(.+([0-9])))}".so + shopt -u extglob nullglob # extract the major version: 1 soversion="${sofile##*\.so\.}" if [[ "$soversion" = "$sofile" ]] && ((IGNORE_INTERNAL)); then |