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

Re: [PATCH] Use exit 1 instead of die when req_Root fails.

From
Frank Lichtenheld <frank@lichtenheld.de>
Date
Oct 17, 2007, 19:08 UTC
Message-ID
<20071017190822.GD9041@planck.djpig.de>
In-Reply-To
<118833cc0710170739i179e7389k1e44f70086ca88be@mail.gmail.com>
On Wed, Oct 17, 2007 at 10:39:52AM -0400, Morten Welinder wrote:
Show 8 quoted lines
> >  made it into your repo.  It fixes test failures on my machine that have
> >  been plauging me for months.
> 
> That sounds more like a reason to fix the test.  "die" is the perl
> standard way of
> reporting an error.  It will print the error message on stderr, not on
> stdout like
> your version does.

Please note that git-cvsserver is special in that its output will never be displayed directly to the user but is always interpreted first by the client. So print "E something" is acutally in some cases more correct than print STDERR something.

Gruesse,
-- 
Frank Lichtenheld <frank@lichtenheld.de>
www: http://www.djpig.de/
Previous: Jan HudecNext: Frank Lichtenheld
Message 8 of 12 in “Use exit 1 instead of die when req_Root fails.”
  1. Use exit 1 instead of die when req_Root fails.Brian Gernhardt, Oct 17, 2007
  2. Morten WelinderOct 17, 2007
  3. Brian GernhardtOct 17, 2007
  4. Lars HjemliOct 17, 2007
  5. Brian GernhardtOct 17, 2007
  6. Lars HjemliOct 17, 2007
  7. Jan HudecOct 18, 2007
  8. Frank LichtenheldOct 17, 2007
  9. Frank LichtenheldOct 17, 2007
  10. Brian GernhardtOct 17, 2007
  11. Shawn O. PearceOct 18, 2007
  12. Frank LichtenheldOct 18, 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.