From e96a68e53ea2c945af30af983c24714e21e0968a Mon Sep 17 00:00:00 2001
From: Luke Shumaker <lukeshu@lukeshu.com>
Date: Tue, 2 Oct 2018 00:47:01 -0400
Subject: libremakepkg: Tidy funny syntax

I don't know what I was thinking when I wrote it that way.

For one, the "<&0" bit is a no-op.  It redirects stdin to be stdin.  Maybe
I had meant to write "<&-" and close it?

For two, just write it normally.  It's simple enough, don't make it look
like magic.
---
 src/chroot-tools/libremakepkg | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

(limited to 'src')

diff --git a/src/chroot-tools/libremakepkg b/src/chroot-tools/libremakepkg
index 248eab6..00eb2e0 100755
--- a/src/chroot-tools/libremakepkg
+++ b/src/chroot-tools/libremakepkg
@@ -76,9 +76,10 @@ run_hook() {
 	local fails=()
 
 	for hook in "${!hookvar}"; do
-		# The "<&0&wait $!" trick is to prevent "||" from
+		# The "& wait $!" trick is to prevent "||" from
 		# disabling "set -e"
-		{ "$hook" "$@" |& indent; } <&0&wait $! || fails+=("$hook")
+		{ "$hook" "$@" |& indent; } &
+		wait $! || fails+=("$hook")
 	done
 
 	if [[ ${#fails[@]} -gt 0 ]]; then
-- 
cgit v1.2.2