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

Re: rebase [-i --exec | -ix] <CMD>...

From
Kkonglu@minatec.inpg.fr <konglu@minatec.inpg.fr>
Date
Jun 7, 2012, 20:44 UTC
Message-ID
<20120607224459.Horde.GLRVL3wdC4BP0RLLE1gTU0A@webmail.minatec.grenoble-inp.fr>
In-Reply-To
<m2k3zivp0c.fsf@igel.home>
Andreas Schwab <schwab@linux-m68k.org> a écrit :
Show 11 quoted lines
> konglu@minatec.inpg.fr writes:
>
>> As you pointed out, the problem seems to come from this line
>> 	tmp=$(sed "/^pick .*/i\exec $i" "$1")
>> I think that the dquote here doesn't work on MacOS, though i've
>> never tried it. Anyway, even if it's not the problem, this kind
>> of quoting is quite odd so it will be removed in the next patch
>> (I do not see other reasons of the issue, as sed's i command should
>> work on MacOS).
>
> You need to put a newline after the backslash.
That's what we originally did. Though we did it with indentations:
         tmp=$(sed "/^pick .*/i\
                            exec $i" "$1")
Should it be put that way (without indentations) ?
         tmp=$(sed "/^pick .*/i\
exec $i" "$1")
Thanks,
Lucien Kong
Previous: Andreas SchwabNext: Andreas Schwab
Message 11 of 15 in “commit a63d7ed3017e312ddc752ac89a7d292a745d5f11 broken on MacOS”
  1. Torsten BögershausenJun 7, 2012
  2. konglu@minatec.inpg.frJun 7, 2012
  3. Torsten BögershausenJun 7, 2012
  4. konglu@minatec.inpg.frJun 7, 2012
  5. rebase [-i --exec | -ix] <CMD>...Lucien Kong, Jun 7, 2012
  6. Torsten BögershausenJun 7, 2012
  7. konglu@minatec.inpg.frJun 7, 2012
  8. Torsten BögershausenJun 7, 2012
  9. konglu@minatec.inpg.frJun 7, 2012
  10. Andreas SchwabJun 7, 2012
  11. konglu@minatec.inpg.frJun 7, 2012
  12. Andreas SchwabJun 7, 2012
  13. Matthieu MoyJun 7, 2012
  14. Johannes SixtJun 7, 2012
  15. Junio C HamanoJun 8, 2012

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.