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

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