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

Re: [PATCH 3/3] git --help COMMAND brings up the git-COMMAND man-page., take two

From
Andreas Ericsson <ae@op5.se>
Date
Nov 16, 2005, 23:16 UTC
Message-ID
<437BBDD6.40809@op5.se>
In-Reply-To
<437BAA5F.5060608@zytor.com>
H. Peter Anvin wrote:
Show 20 quoted lines
> Junio C Hamano wrote:
> 
>> "H. Peter Anvin" <hpa@zytor.com> writes:
>>
>>> The way this made it into the actual tree was to call /usr/bin/man, 
>>> but still using execlp().  This is clearly bogus.  There *ARE* good 
>>> reasons to use PATH resolutions for this, since man is one of the 
>>> interactive commands the user may want to wrapper.
>>
>>
>> Oh, that was my call, so please do not blame Andreas.
>> I just sent out a proposed patch to address all the points
>> discussed on the list for the last several hours.
>>
> 
> FWIW, I rarely blame people for bad code; *everyone* does something 
> stupid every now and then, and for most of us, far more often than that. 
>  "This piece of code is stupid" != "the author of this piece of code is 
> stupid", a distinction which unfortunately often gets lost.
> 

True, but it's possible to have a 180 iq and still know nothing about programming. Anyhow, I sent the first patch too, so please blame me if anyone. If I was hoping to be admired I'd go play video-games with my 6 year old cousin.

-- 
Andreas Ericsson                   andreas.ericsson@op5.se
OP5 AB                             www.op5.se
Tel: +46 8-230225                  Fax: +46 8-230231
Previous: H. Peter AnvinNext: H. Peter Anvin
Message 8 of 9 in “git --help COMMAND brings up the git-COMMAND man-page., take two”
  1. 3/3 git --help COMMAND brings up the git-COMMAND man-page., take twoAndreas Ericsson, Nov 16, 2005
  2. H. Peter AnvinNov 16, 2005
  3. Andreas EricssonNov 16, 2005
  4. git wrapper: basic fixes.Junio C Hamano, Nov 16, 2005
  5. Johannes SchindelinNov 16, 2005
  6. Junio C HamanoNov 16, 2005
  7. H. Peter AnvinNov 16, 2005
  8. Andreas EricssonNov 16, 2005
  9. H. Peter AnvinNov 16, 2005

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.