Hook not created in repository

[ericpaulbishop]

Hook not created in repository

[s0undt3ch]

And I'm still getting some:

fatal: /usr/lib/git-core/git-pull cannot be used without a working tree.

Didn't you addressed these issues?

[s0undt3ch]

Also testes with a parent'less test project, same non existing hook issue.

[ericpaulbishop]

Is this still broken after merging your latest fix (fixing the pull => fetch, merge in the second location) ? Sorry I missed it in that second (more important) spot the first time around.

[s0undt3ch]

yes, notice the following:

www-data@ubxenweb:~$ sudo -u www-data ruby /srv/chili/projects/chiliproject/script/runner -e production "print \\\"good\\\""
Sorry, user www-data is not allowed to execute '/usr/bin/ruby /srv/chili/projects/chiliproject/script/runner -e production print \"good\"' as www-data on ubxenweb.ufsoft.org.

:)

[s0undt3ch]

The user running is already the web user, do you really want to sudo to him? specially since on current docs, the web user should only be allowed to sudo to the git user.

[ericpaulbishop]

That command should be getting run as the git user -- it's a test
whether the git user is correctly configured to run commands as the
www-data user. The www-data user uses the run_as_git_user command to
run sudo BACK to the www-data user. I suppose I could just remove the
test, since current configuration with sudo requires that git
repos/web server be in the same box, but previous ssh configuration
allowed the case where they were not (hence the reason for
testing...).

One reason I've left the test there is as a general test of whether
the run_as_git_user command is working properly -- I had some problems
until I realized that there are different versions of sudo that
actually behave differently (see my commits yesterday). However,
since I've fixed that it seems to be working fine for me, so I'm not
sure why you're having problems. If you comment out the test in the
git_hosting.rb file and just let the commands that set the post-update
hook run do those get called (and the hook set up) properly?

Eric

On Tue, Jul 12, 2011 at 10:16 AM, s0undt3ch
[email protected]
wrote:

The user running is already the web user, do you really want to sudo to him? specially since on current docs, the web user should only be allowed to sudo to the git user.

Reply to this email directly or view it on GitHub:
#30 (comment)

[s0undt3ch]

#32 fixes this ticket.