{"thread":{"id":"14377","subject":"Bug: /usr/local/bin/git-daemon starts /usr/sbin/git-upload-pack?","startedAt":"2008-07-09T20:20:10Z","lastAt":"2008-07-10T08:59:53Z","messageCount":5,"participants":["Jan Wielemaker","Junio C Hamano","Brian Foster","Jeff King"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"82746","messageId":"200807092220.10655.J.Wielemaker@uva.nl","threadId":"14377","inReplyTo":null,"subject":"Bug: /usr/local/bin/git-daemon starts /usr/sbin/git-upload-pack?","fromName":"Jan Wielemaker","fromEmail":"j.wielemaker@uva.nl","sentAt":"2008-07-09T20:20:10Z","receivedAt":"2008-07-09T20:20:10Z","isPatch":false,"sender":{"key":"j.wielemaker@uva.nl","avatar":null},"body":"Hi,\n\nAfter upgrade to SuSE 11.0 I was forced to update GIT (changed libcurl).\nI did (with some trickery) a pull of the latest git and built it using\nmake prefix=/usr/local ..., to find out that /usr/local/bin/git-daemon\nstarts /usr/sbin/git-upload-pack.\n\nAfter creating a symlink to /usr/local/bin/git-upload-pack all works fine\nagain, but I guess this is a mistake?\n\n\tCheers --- Jan\n\nP.s.\tThis is quite nasty to debug.  I was forced to to run git-daemon\n\tstand-alone (not xinetd) another port and run strace -f -p <pid>\n\tto discover the\tcause of this problem.  Even with --verbose, the\n\tonly error response was the client complaining on unexpected EOF.\n"},{"id":"82751","messageId":"7vlk0a7obl.fsf@gitster.siamese.dyndns.org","threadId":"14377","inReplyTo":"200807092220.10655.J.Wielemaker@uva.nl","subject":"Re: Bug: /usr/local/bin/git-daemon starts /usr/sbin/git-upload-pack?","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2008-07-09T21:14:06Z","receivedAt":"2008-07-09T21:14:06Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Jan Wielemaker <J.Wielemaker@uva.nl> writes:\n\n> After upgrade to SuSE 11.0 I was forced to update GIT (changed libcurl).\n> I did (with some trickery) a pull of the latest git and built it using\n> make prefix=/usr/local ..., to find out that /usr/local/bin/git-daemon\n> starts /usr/sbin/git-upload-pack.\n>\n> After creating a symlink to /usr/local/bin/git-upload-pack all works fine\n> again, but I guess this is a mistake?\n\nI think this is a distro issue.\n\nPre-1.6.0 git installs git-everything in $(bindir) and knows things are\nthere.  Current 'master' we are preparing for 1.6.0 instead installs\ngit-mostofthem in $(prefix)/libexec/git-core and git-someofthem in\n$(bindir); our executables know which binaries are installed in $(bindir)\nand which ones are installed in $(prefix)/libexec/git-core.\n\nThe point is we never install $anywhere/sbin unless the person who is\nbuilding explicitly does so (either by futzing the Makefile or giving\nbindir=$somewhere from the command line of \"make\").  The reason your\n/usr/local/bin/git-daemon (we do not know who compiled it and how) spawns\nsomething in /usr/sbin is not our doing.  Find out who configured it and\nwhy, and more importantly, what _other_ changes are made in the build and\ninstallation procedure to support that change in location.\n"},{"id":"82788","messageId":"200807100932.06569.brian.foster@innova-card.com","threadId":"14377","inReplyTo":"200807092220.10655.J.Wielemaker@uva.nl","subject":"Re: Bug: /usr/local/bin/git-daemon starts /usr/sbin/git-upload-pack?","fromName":"Brian Foster","fromEmail":"brian.foster@innova-card.com","sentAt":"2008-07-10T07:32:06Z","receivedAt":"2008-07-10T07:32:06Z","isPatch":false,"sender":{"key":"brian.foster@innova-card.com","avatar":null},"body":"On Wednesday 09 July 2008 22:20:10 Jan Wielemaker wrote:\n>[ ... ]\n> P.s.\tThis [ git-daemon using a weird path to exec git-upload-pack\n>       was ] quite nasty to debug.  I was forced to to run git-daemon\n> \tstand-alone (not xinetd) another port and run strace -f -p <pid>\n> \tto discover the\tcause of this problem.  Even with --verbose, the\n> \tonly error response was the client complaining on unexpected EOF.\n\n Just a hint:  I had some odd exec problems in the past\n (as it happens, also from xinetd and also path-related,\n but not(?) your problem), and found that exporting\n\n    GIT_TRACE=/tmp/LOG-git-daemon\n\n gave some valuable clews.  I'm unaware of GIT_TRACE\n being documented.  I also concur it's annoying neither\n --syslog nor --verbose seems to cause these sort of\n errors to be logged.\n\ncheers!\n\t-blf-\n\n-- \n“How many surrealists does it take to   | Brian Foster\n change a lightbulb? Three. One calms   | somewhere in south of France\n the warthog, and two fill the bathtub  |   Stop E$$o (ExxonMobil)!\n with brightly-coloured machine tools.” |      http://www.stopesso.com\n"},{"id":"82793","messageId":"200807100938.53189.J.Wielemaker@uva.nl","threadId":"14377","inReplyTo":"7vlk0a7obl.fsf@gitster.siamese.dyndns.org","subject":"Re: Bug: /usr/local/bin/git-daemon starts /usr/sbin/git-upload-pack?","fromName":"Jan Wielemaker","fromEmail":"j.wielemaker@uva.nl","sentAt":"2008-07-10T07:38:52Z","receivedAt":"2008-07-10T07:38:52Z","isPatch":false,"sender":{"key":"j.wielemaker@uva.nl","avatar":null},"body":"Hi Junio,\n\nOn Wednesday 09 July 2008 23:14:06 Junio C Hamano wrote:\n> Jan Wielemaker <J.Wielemaker@uva.nl> writes:\n> > After upgrade to SuSE 11.0 I was forced to update GIT (changed libcurl).\n> > I did (with some trickery) a pull of the latest git and built it using\n> > make prefix=/usr/local ..., to find out that /usr/local/bin/git-daemon\n> > starts /usr/sbin/git-upload-pack.\n> >\n> > After creating a symlink to /usr/local/bin/git-upload-pack all works fine\n> > again, but I guess this is a mistake?\n>\n> I think this is a distro issue.\n>\n> Pre-1.6.0 git installs git-everything in $(bindir) and knows things are\n> there.  Current 'master' we are preparing for 1.6.0 instead installs\n> git-mostofthem in $(prefix)/libexec/git-core and git-someofthem in\n> $(bindir); our executables know which binaries are installed in $(bindir)\n> and which ones are installed in $(prefix)/libexec/git-core.\n>\n> The point is we never install $anywhere/sbin unless the person who is\n> building explicitly does so (either by futzing the Makefile or giving\n> bindir=$somewhere from the command line of \"make\").  The reason your\n> /usr/local/bin/git-daemon (we do not know who compiled it and how) spawns\n> something in /usr/sbin is not our doing.  Find out who configured it and\n> why, and more importantly, what _other_ changes are made in the build and\n> installation procedure to support that change in location.\n\nIf you read carefully you see some \"I ...\" :-)  Summarising to avoid all\nconfusion:\n\n\t* I had git installed in /usr/local/bin, this was git from git,\n\tclose to 1.5.6.  The machine was running SuSE 10.2.\n\t* I installed SuSE 11.0, whiping /, but git remained as \n\t/usr/local is on a seperate partition.\n\t* Git refused to run (dependency on libcurl.so.3; 11.0 ships with\n\tlibcurl.so.4).\n\t* I copied libcurl.so.3 from backup into /usr/lib64, which at least\n\tmade git run again.\n\t* Then I did \n\n\t\t% git pull\n\t\t% git clean -x -f\n\t\t% make prefix=/usr/local all doc info \n\t\t# make prefix=/usr/local install install-doc install-info\n\t\t\n\t* All went smooth, but clients to git:// complained that the other site\n\thung up unexpectedly.\n\t* Ran git-daemon --verbose --export-all --user=git --port=4000 /some/path\n\t* Ran strace -f -p <pid>\n\t* Ran git clone git://localhost:4000/some/path to find it was trying to\n\tstart /usr/sbin/git-upload-pack.\n\nI understand bit changes are underway, so it might not be worthwhile to try \nand\nget to the bottom of this.\n\n\tCheers --- Jan\n"},{"id":"82800","messageId":"20080710085952.GA5982@sigill.intra.peff.net","threadId":"14377","inReplyTo":"200807100932.06569.brian.foster@innova-card.com","subject":"Re: Bug: /usr/local/bin/git-daemon starts /usr/sbin/git-upload-pack?","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2008-07-10T08:59:53Z","receivedAt":"2008-07-10T08:59:53Z","isPatch":false,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Thu, Jul 10, 2008 at 09:32:06AM +0200, Brian Foster wrote:\n\n>     GIT_TRACE=/tmp/LOG-git-daemon\n> \n>  gave some valuable clews.  I'm unaware of GIT_TRACE\n>  being documented.  I also concur it's annoying neither\n\nIt's in git(1).\n\n-Peff\n"}]}