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

Re: [PATCH] run-command: report exec failure

From
John Passaro <john.a.passaro@gmail.com>
Date
Dec 12, 2018, 15:27 UTC
Message-ID
<CAJdN7Khf+Y_jyyG2qoqiMHYPCHHSws15EDftOQni=gFJ7SoMXg@mail.gmail.com>
In-Reply-To
<xmqqsgz4jkgl.fsf@gitster-ct.c.googlers.com>
Thank you for this incredibly quick fix.

I see the fix made it to pu as 6b206be3e5 ("run-command: report exec failure" 2018-12-11). For what it's worth, it fixes the issue as far as I'm concerned and I'm very glad to see the behavior is covered by tests now.

As a procedural question: I'd like to reference this patch in one of my own. Can I reference it as I typed it above? Or is there a chance of the SHA1 changing before it goes into some sort of a main history?

John Passaro (917) 678-8293 On Tue, Dec 11, 2018 at 7:50 AM Junio C Hamano <gitster@pobox.com> wrote:

Show 7 quoted lines
>
> Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:
>
> > This breaks on Windows (on Windows, the error message says "cannot spawn", see
>
> Thanks for a quick feedback.  Let's update to look for the pathname
> of the command, as Peff suggested earlier.
Previous: Junio C HamanoNext: Jeff King
Message 5 of 10 in “run-command: report exec failure”
  1. run-command: report exec failureJunio C Hamano, Dec 11, 2018
  2. Jeff KingDec 11, 2018
  3. Johannes SchindelinDec 11, 2018
  4. Junio C HamanoDec 11, 2018
  5. John PassaroDec 12, 2018
  6. Jeff KingDec 13, 2018
  7. run-command: report exec failureJunio C Hamano, Dec 12, 2018
  8. Jeff KingDec 13, 2018
  9. Junio C HamanoDec 13, 2018
  10. Johannes SchindelinDec 13, 2018

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.