* [PATCH] rebase: fix run_specific_rebase's use of "return" on FreeBSD
@ 2013-09-09 8:53 Matthieu Moy
2013-09-09 9:00 ` Ramkumar Ramachandra
2013-09-09 15:44 ` Junio C Hamano
0 siblings, 2 replies; 3+ messages in thread
From: Matthieu Moy @ 2013-09-09 8:53 UTC (permalink / raw
To: git, gitster; +Cc: avg, christoph.mallon, Ramkumar Ramachandra, Matthieu Moy
Since a1549e10, git-rebase--am.sh uses the shell's "return" statement, to
mean "return from the current file inclusion", which is POSIXly correct,
but badly interpreted on FreeBSD, which returns from the current
function, hence skips the finish_rebase statement that follows the file
inclusion.
Make the use of "return" portable by using the file inclusion as the last
statement of a function.
Reported-by: Christoph Mallon <christoph.mallon@gmx.de>
Signed-off-by: Matthieu Moy <Matthieu.Moy@imag.fr>
---
I sent the patch inline in a conversation and got no feedback.
Resending as a proper patch.
git-rebase.sh | 11 ++++++++++-
1 file changed, 10 insertions(+), 1 deletion(-)
diff --git a/git-rebase.sh b/git-rebase.sh
index 8d7659a..226752f 100755
--- a/git-rebase.sh
+++ b/git-rebase.sh
@@ -167,13 +167,22 @@ You can run "git stash pop" or "git stash drop" at any time.
rm -rf "$state_dir"
}
-run_specific_rebase () {
+run_specific_rebase_internal () {
if [ "$interactive_rebase" = implied ]; then
GIT_EDITOR=:
export GIT_EDITOR
autosquash=
fi
+ # On FreeBSD, the shell's "return" returns from the current
+ # function, not from the current file inclusion.
+ # run_specific_rebase_internal has the file inclusion as a
+ # last statement, so POSIX and FreeBSD's return will do the
+ # same thing.
. git-rebase--$type
+}
+
+run_specific_rebase () {
+ run_specific_rebase_internal
ret=$?
if test $ret -eq 0
then
--
1.8.4.5.g8688bea
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] rebase: fix run_specific_rebase's use of "return" on FreeBSD
2013-09-09 8:53 [PATCH] rebase: fix run_specific_rebase's use of "return" on FreeBSD Matthieu Moy
@ 2013-09-09 9:00 ` Ramkumar Ramachandra
2013-09-09 15:44 ` Junio C Hamano
1 sibling, 0 replies; 3+ messages in thread
From: Ramkumar Ramachandra @ 2013-09-09 9:00 UTC (permalink / raw
To: Matthieu Moy; +Cc: Git List, Junio C Hamano, avg, christoph.mallon
Matthieu Moy wrote:
> diff --git a/git-rebase.sh b/git-rebase.sh
> index 8d7659a..226752f 100755
> --- a/git-rebase.sh
> +++ b/git-rebase.sh
> @@ -167,13 +167,22 @@ You can run "git stash pop" or "git stash drop" at any time.
> rm -rf "$state_dir"
> }
>
> -run_specific_rebase () {
> +run_specific_rebase_internal () {
> if [ "$interactive_rebase" = implied ]; then
> GIT_EDITOR=:
> export GIT_EDITOR
> autosquash=
> fi
> + # On FreeBSD, the shell's "return" returns from the current
> + # function, not from the current file inclusion.
> + # run_specific_rebase_internal has the file inclusion as a
> + # last statement, so POSIX and FreeBSD's return will do the
> + # same thing.
> . git-rebase--$type
> +}
Most excellent. Thanks for fixing this.
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] rebase: fix run_specific_rebase's use of "return" on FreeBSD
2013-09-09 8:53 [PATCH] rebase: fix run_specific_rebase's use of "return" on FreeBSD Matthieu Moy
2013-09-09 9:00 ` Ramkumar Ramachandra
@ 2013-09-09 15:44 ` Junio C Hamano
1 sibling, 0 replies; 3+ messages in thread
From: Junio C Hamano @ 2013-09-09 15:44 UTC (permalink / raw
To: Matthieu Moy; +Cc: git, avg, christoph.mallon, Ramkumar Ramachandra
Matthieu Moy <Matthieu.Moy@imag.fr> writes:
> Since a1549e10, git-rebase--am.sh uses the shell's "return" statement, to
> mean "return from the current file inclusion", which is POSIXly correct,
> but badly interpreted on FreeBSD, which returns from the current
> function, hence skips the finish_rebase statement that follows the file
> inclusion.
>
> Make the use of "return" portable by using the file inclusion as the last
> statement of a function.
>
> Reported-by: Christoph Mallon <christoph.mallon@gmx.de>
> Signed-off-by: Matthieu Moy <Matthieu.Moy@imag.fr>
> ---
> I sent the patch inline in a conversation and got no feedback.
> Resending as a proper patch.
I'll replace mm/rebase-continue-freebsd-WB with this version, but it
would be nice to hear from FreeBSD folks that this is sufficient for
their platform to work around the issue (e.g. there could be other
"return"s unfixed outside the codepath this patch fixes).
Thanks.
> git-rebase.sh | 11 ++++++++++-
> 1 file changed, 10 insertions(+), 1 deletion(-)
>
> diff --git a/git-rebase.sh b/git-rebase.sh
> index 8d7659a..226752f 100755
> --- a/git-rebase.sh
> +++ b/git-rebase.sh
> @@ -167,13 +167,22 @@ You can run "git stash pop" or "git stash drop" at any time.
> rm -rf "$state_dir"
> }
>
> -run_specific_rebase () {
> +run_specific_rebase_internal () {
> if [ "$interactive_rebase" = implied ]; then
> GIT_EDITOR=:
> export GIT_EDITOR
> autosquash=
> fi
> + # On FreeBSD, the shell's "return" returns from the current
> + # function, not from the current file inclusion.
> + # run_specific_rebase_internal has the file inclusion as a
> + # last statement, so POSIX and FreeBSD's return will do the
> + # same thing.
> . git-rebase--$type
> +}
> +
> +run_specific_rebase () {
> + run_specific_rebase_internal
> ret=$?
> if test $ret -eq 0
> then
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2013-09-09 15:45 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-09-09 8:53 [PATCH] rebase: fix run_specific_rebase's use of "return" on FreeBSD Matthieu Moy
2013-09-09 9:00 ` Ramkumar Ramachandra
2013-09-09 15:44 ` Junio C Hamano
Code repositories for project(s) associated with this public inbox
https://80x24.org/mirrors/git.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).