[devtools,v2,1/2] makechrootpkg: Fix unconditionally running namcap

Message ID 20170903075340.13311-1-eschwartz@archlinux.org
State Accepted
Headers show
Series [devtools,v2,1/2] makechrootpkg: Fix unconditionally running namcap | expand

Commit Message

Eli Schwartz Sept. 3, 2017, 7:53 a.m. UTC
Fixes regression in 2fd5931a8c67289a8a4acd327b3ce99a5d64c8c7

$run_namcap will always be set to ""
`if $not_a_var; then ...; fi` is always truthful when $not_a_var is
unset or equal to "" and the `then` clause will always be run.

I'm not sure why global state variables need to be cloned locally for
their sole explicit purpose.

But for now this patch implements the minimum necessary work to properly
pass the "do I want namcap" variable into prepare_chroot() according to
the current logic flow.
Note that I have still not thorougly tested makechrootpkg.

Signed-off-by: Eli Schwartz <eschwartz@archlinux.org>

v2: somewhat more accurate commit message

 makechrootpkg.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)


diff --git a/makechrootpkg.in b/makechrootpkg.in
index ef3f2ec..fe9410d 100644
--- a/makechrootpkg.in
+++ b/makechrootpkg.in
@@ -408,7 +408,7 @@  main() {
 	download_sources "$copydir" "$makepkg_user"
-	prepare_chroot "$copydir" "$USER_HOME" "$keepbuilddir"
+	prepare_chroot "$copydir" "$USER_HOME" "$keepbuilddir" "$run_namcap"
 	if arch-nspawn "$copydir" \
 		--bind="$PWD:/startdir" \