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

Re: [PATCH 2/2] git-p4: fix handling of multi-word P4EDITOR

From
Junio C Hamano <gitster@pobox.com>
Date
May 7, 2015, 20:16 UTC
Message-ID
<xmqqlhh0nny1.fsf@gitster.dls.corp.google.com>
In-Reply-To
<554BBCBE.1020408@diamand.org>
Luke Diamand <luke@diamand.org> writes:
Show 12 quoted lines
> On Linux:
>
> $ export P4EDITOR="gvim -f"
> $ p4 submit
> -- works as you would hope --
>
> $ export P4EDITOR="/home/lgd/My Programs/editor.sh"
> $ p4 submit
> sh: 1: /home/lgd/My: not found
>
> $ export P4EDITOR="/home/lgd/My\ Programs/editor.sh"
> -- works fine --
Good.  That is exactly I wanted to see.
Thanks.
Previous: Luke DiamandNext: Luke Diamand
Message 6 of 13 in “Re: Bug: git-p4 edit_template() and P4EDITOR w/options”
  1. 0/2 Re: Bug: git-p4 edit_template() and P4EDITOR w/optionsLuke Diamand, May 7, 2015
  2. 1/2 git-p4: add failing test for P4EDITOR handlingLuke Diamand, May 7, 2015
  3. 2/2 git-p4: fix handling of multi-word P4EDITORLuke Diamand, May 7, 2015
  4. Junio C HamanoMay 7, 2015
  5. Luke DiamandMay 7, 2015
  6. Junio C HamanoMay 7, 2015
  7. Luke DiamandMay 7, 2015
  8. Junio C HamanoMay 7, 2015
  9. Junio C HamanoMay 7, 2015
  10. Luke DiamandMay 7, 2015
  11. Junio C HamanoMay 7, 2015
  12. Luke DiamandMay 24, 2015
  13. Junio C HamanoMay 26, 2015

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.