Weirdness with port-update hook and local push

I have the following post-update hook:

----- #!/bin/sh

unset GIT_DIR
cd /home/barkalow/auto-working/web
if ! git pull /home/barkalow/git/web.git/
then
  exit 1  
fi
make
-----

/home/barkalow/bin/git-pull: line 108: 30608 Broken pipe      git-merge $no_summary $no_commit $strategy_args "$merge_name" HEAD $merge_head

It works fine when pushing over ssh, and when I just run the hook directly. (It does a fast forward merge without any trouble.) Any ideas on what's confusing it?

	-Daniel
*This .sig left intentionally blank*

Hi,

On Mon, 5 Dec 2005, Daniel Barkalow wrote:

Maybe it runs as a different user? (Sorry if this sounds dumb, but that's exactly the kind of solution I usually find after *days*.)

Hth, Dscho

On Mon, 5 Dec 2005, Johannes Schindelin wrote:

I think it's an environment thing of some sort, most likely, but it can't be the user; nothing's setuid and I'm only one user.

The thing that confuses me is that it works when run from ssh or directly, but not when run from a local push. I'd expect the two that work to be most different.

	-Daniel
*This .sig left intentionally blank*

Daniel Barkalow <barkalow@iabervon.org> writes:

One suspicion and one suggestion (without knowing exactly where that suggestion might lead us to).

 - your stdout/stderr might be connected to somewhere that your
   output gets stuck ("broken pipe"), when your script is run
   from the hook.

 - your environment might be different from what you are
   assuming.

How about doing something like this?

	  #!/bin/sh

	+ exec >/var/tmp/hook-out.$$ 2>/var/tmp/hook-err.$$
	+ echo "** env **"
	+ env
	+ echo "** vars **"
	+ i=0
	+ for v
	+ do
	+	 echo "$i: $v"
	+	 i=$(($i+1))
	+ done
	+ echo "** pwd etc **"
	+ pwd
	+ id -a

	  unset GIT_DIR
	  cd /home/barkalow/auto-working/web
	  if ! git pull /home/barkalow/git/web.git/

and then next replace the whole thing with:

	exec >/dev/null 2>&1

On Mon, 5 Dec 2005, Johannes Schindelin wrote:

Actually, "ls -l /proc/$$/fd" shows that stdin and stdout are sockets with ssh, pipes with local, and ttys when just running the script.

The hooks probably ought to be run with something different for stdin/stdout than the connection to the source of the data. /dev/null, maybe? Or a log file? Maybe dup stderr, which still seems to be going to the pusher's terminal?

	-Daniel
*This .sig left intentionally blank*