Skip to content

Commit 75e26c6

Browse files
committed
fix: silence a few more variables in chroot-pip
1 parent d06a4d3 commit 75e26c6

File tree

1 file changed

+7
-8
lines changed

1 file changed

+7
-8
lines changed

chroot-pip.sh

Lines changed: 7 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -89,23 +89,22 @@ fi
8989
case "$@" in
9090
*'--force-reinstall'*|optimize*)
9191
maybe_packages=
92-
index=-1
92+
index='-1'
9393
for package_name in $@
9494
do
95-
if ! case "$package_name" in '-'*) true ;; *) false ;; esac ; then
96-
index="$(expr $index \+ 1)" || true # ARJ: expr 0 returns 1 lol
95+
if [ x"$package_name" != x ] && ! case "$package_name" in '-'*) true ;; *) false ;; esac ; then
96+
index="$(expr $index \+ 1)" || index='0'
97+
if case "$package_name" in *'=='*) true ;; *) false ;; esac ; then
98+
package_name="$(echo "${package_name}" | cut -d'=' -f1 | xargs)"
99+
fi
97100
if [ $index -ne 0 ]; then
98-
maybe_packages="${maybe_packages} $package_name"
101+
maybe_packages="${maybe_packages} ${package_name}"
99102
fi
100103
fi
101104
done
102-
package_name=
103105
if [ x$maybe_packages != x ]; then
104106
for package_name in $maybe_packages
105107
do
106-
if case "$package_name" in *'=='*) true ;; *) false ;; esac ; then
107-
package_name="$(echo "${package_name}" | cut -d'=' -f1 | xargs)"
108-
fi
109108
# if the package is already installed, flag it
110109
# as if it wasn't installed so we can optimize it
111110
if [ x$package_name != x ] && pip show "${package_name}" >/dev/null ; then

0 commit comments

Comments
 (0)