[patch] prevent ebuild.sh subprocesses from inheriting file descriptor 9

close_pipe.patch (text/plain), 1.20 KB, created by Zac Medico on 2008-12-10 01:51:31 UTC

Index: bin/ebuild.sh
===================================================================
--- bin/ebuild.sh	(revision 12188)
+++ bin/ebuild.sh	(revision 12189)
@@ -2086,16 +2086,26 @@
 	fi
 }
 
-[[ -n $EBUILD_SH_ARGS ]] && ebuild_main
+if [[ $EBUILD_PHASE = depend ]] ; then
+	ebuild_main
+elif [[ -n $EBUILD_SH_ARGS ]] ; then
+	(
+		# Don't allow subprocesses to inherit the pipe which
+		# emerge uses to monitor ebuild.sh.
+		exec 9>&-
 
-# Save the env only for relevant phases.
-if [ -n "${EBUILD_SH_ARGS}" ] && \
-	! hasq ${EBUILD_SH_ARGS} clean depend help info nofetch ; then
-	# Save current environment and touch a success file. (echo for success)
-	umask 002
-	save_ebuild_env | filter_readonly_variables > "${T}/environment"
-	chown portage:portage "${T}/environment" &>/dev/null
-	chmod g+w "${T}/environment" &>/dev/null
+		ebuild_main
+
+		# Save the env only for relevant phases.
+		if ! hasq "$EBUILD_SH_ARGS" clean help info nofetch ; then
+			umask 002
+			save_ebuild_env | filter_readonly_variables > "$T/environment"
+			chown portage:portage "$T/environment" &>/dev/null
+			chmod g+w "$T/environment" &>/dev/null
+		fi
+		exit 0
+	)
+	exit $?
 fi
 
 # Do not exit when ebuild.sh is sourced by other scripts.