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

Re: [PATCH] cvsserver: Don't send mixed messages to clients

From
Junio C Hamano <junkio@cox.net>
Date
May 14, 2007, 21:28 UTC
Message-ID
<7vmz07m6wy.fsf@assigned-by-dhcp.cox.net>
In-Reply-To
<20070514131528.GC5272@planck.djpig.de>
Frank Lichtenheld <frank@lichtenheld.de> writes:
Show 19 quoted lines
> After looking through all the uses of
> 	die "something"
> and
> 	print "E something";
> 	exit;
> and
> 	print "error 1 something";
> 	exit;
>
> in the current code I fail to see a real pattern there, is there any?
> If yes, we should maybe document it in a comment somewhere...
>
> For the record, cvs (server side) does the following in this particular situation:
> 	buf_output0 (buf_to_net, "I HATE YOU\n");
>         buf_flush (buf_to_net, true);
>
>         /* Don't worry about server_cleanup, server_active isn't set
>            yet.  */
>         exit (EXIT_FAILURE);

Ok, I stand corrected. If there is a pattern to emulate, that should be found in the real cvs server.

Previous: Frank LichtenheldNext: Frank Lichtenheld
Message 2 of 5 in “cvsserver: Don't send mixed messages to clients”
  1. cvsserver: Don't send mixed messages to clientsFrank Lichtenheld, May 13, 2007
  2. Junio C HamanoMay 14, 2007
  3. Frank LichtenheldMay 20, 2007
  4. Junio C HamanoMay 20, 2007
  5. Frank LichtenheldMay 20, 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.