Re: [PATCH] t5570: forward git-daemon messages in a different way
From: Clemens Buchacher <hidden>
Date: 2016-06-15 22:53:40
On Thu, Apr 26, 2012 at 08:16:37PM +0200, Johannes Sixt wrote:
quoted hunk ↗ jump to hunk
@@ -30,10 +30,10 @@ start_git_daemon() { "$@" "$GIT_DAEMON_DOCUMENT_ROOT_PATH" \ >&3 2>git_daemon_output & GIT_DAEMON_PID=$! + exec 7<git_daemon_output && { - read line + read line <&7 echo >&4 "$line" - cat >&4 & # Check expected output if test x"$(expr "$line" : "\[[0-9]*\] \(.*\)")" != x"Ready to rumble"@@ -43,7 +43,9 @@ start_git_daemon() { trap 'die' EXIT error "git daemon failed to start" fi - } <git_daemon_output + cat <&7 >&4 & + exec 7<&- + }
I won't pretend to understand why this works. I have to study this some more. But if this is 'correct', then it is obviously preferable to the comparatively complicated wrapper. We should move the cat <&7 >&4 & and exec 7<&- part in front of the output check, otherwise output would be truncated in an error condition. This can be tested by passing an invalid argument to git daemon above, for example. Clemens