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

[RFC PATCH 0/2] Report remote helper exec failures

From
ILIlari Liusvaara <ilari.liusvaara@elisanet.fi>
Date
Dec 24, 2009, 17:49 UTC
Message-ID
<1261676971-3285-1-git-send-email-ilari.liusvaara@elisanet.fi>

Actually give useful error messages if executing git remote helper fails for some reason.

The previous error message was:
git: 'remote-fail' is not a git-command. See 'git --help'
This changes the error message to:
'Unable to find remote helper for "nonexistent"'
or

'Unable to run helper HelperThatGetsEACCESS: Permission denied' (or whatever the errno is).

Patch series is based on v1.6.6.

One of the changes is adjacent to changes in external helper dispatch support change and gets merge conflict. Here's how I resolved that conflict for testing:

        helper->argv[2] = remove_ext_force(transport->url);
        helper->git_cmd = 0;
        if (start_command(helper)) {
                if (errno == ENOENT)
                        die("Unable to find remote helper for \"%s\"",
                                data->name);
                else
                        die("Unable to run helper %s: %s", helper->argv[0],
                                strerror(errno));
        }

The first line in that comes from dispatch support and the rest come from exec failure reporting.

Ilari Liusvaara (2):
  Report exec errors from run-command
  Improve transport helper exec failure reporting
 run-command.c      |   72 +++++++++++++++++++++++++++++++++++++++++++++++++--
 transport-helper.c |   14 +++++++---
 2 files changed, 79 insertions(+), 7 deletions(-)
Next: Ilari Liusvaara
Message 1 of 11 in “Report remote helper exec failures”
  1. 0/2 Report remote helper exec failuresIlari Liusvaara, Dec 24, 2009
  2. 1/2 Report exec errors from run-commandIlari Liusvaara, Dec 24, 2009
  3. Junio C HamanoDec 25, 2009
  4. Junio C HamanoDec 25, 2009
  5. Junio C HamanoDec 25, 2009
  6. Ilari LiusvaaraDec 25, 2009
  7. Johannes SixtDec 25, 2009
  8. Ilari LiusvaaraDec 25, 2009
  9. 2/2 Improve transport helper exec failure reportingIlari Liusvaara, Dec 24, 2009
  10. Junio C HamanoDec 25, 2009
  11. Ilari LiusvaaraDec 25, 2009

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.