git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: Meaning of "fatal: protocol error: bad line length character"?

From
Junio C Hamano <junkio@cox.net>
Date
Jan 22, 2007, 20:15 UTC
Message-ID
<7vr6tmstri.fsf@assigned-by-dhcp.cox.net>
In-Reply-To
<20070120213738.GA8973@spearce.org>
"Shawn O. Pearce" <spearce@spearce.org> writes:
Show 14 quoted lines
> Junio C Hamano <junkio@cox.net> wrote:
>> Funny thing is, at least in recent enough git, I think we set up
>> redirection to force output from hook scripts to stderr, but I
>> do not remember when it happened.  Let's see...
>> 
>> 	...goes and looks...
>> 
>> That was supposed to have been fixed with commit cd83c74c (Dec
>> 30, 2006), but this indicates the fix is not working at all when
>> your login shell is tcsh.
>
> Don't tell us tcsh is doing something ugly like opening the tty for
> stdout/stderr instead of using the ones it inherited from its parent.
> 'cause that's just useless!

It turns out that this was a stupidity on the part of the tester (i.e. me).

I let the distro to install a version of git in /usr/bin/, and my .cshrc is not set up to add /home/junio/git-active/bin (I have $HOME/git-{master,next,pu,maint}/bin and git-active is a symlink to git-master right now -- usually it points at git-next), for non-interactive session like invoking receive-pack. But my bash startup does. The breakage I was seeing was _NOT_ because of differences between tcsh and bash, but because the fix is fairly recent.

I forced the master version of receive-pack to be used on the remote side using --receive-pack command line parameter, and things seem to work fine with either tcsh or bash.

So, no, there might be valid reasons to hate tcsh, but this is not one of them.

Previous: Shawn O. PearceNext: Bill Lear
Message 9 of 17 in “Meaning of "fatal: protocol error: bad line length character"?”
  1. Bill LearJan 20, 2007
  2. Junio C HamanoJan 20, 2007
  3. Junio C HamanoJan 20, 2007
  4. Bill LearJan 20, 2007
  5. Jakub NarebskiJan 20, 2007
  6. Junio C HamanoJan 20, 2007
  7. Junio C HamanoJan 20, 2007
  8. Shawn O. PearceJan 20, 2007
  9. Junio C HamanoJan 22, 2007
  10. Bill LearJan 22, 2007
  11. Junio C HamanoJan 22, 2007
  12. Bill LearJan 22, 2007
  13. Bill LearJan 22, 2007
  14. Junio C HamanoJan 22, 2007
  15. Han-Wen NienhuysJan 23, 2007
  16. Junio C HamanoJan 23, 2007
  17. Han-Wen NienhuysJan 23, 2007

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.