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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




Andreas Schwab <schwab@xxxxxxxxxxxxxx> a écrit :

konglu@xxxxxxxxxxxxxxx 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

--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]