Jacek Konieczny <jajcus@xxxxxxxxxx> writes: > It is not C. The code would break under mksh when 'pull.ff' is set: > > $ git pull > /usr/lib/git-core/git-pull[67]: break: can't break > Already up-to-date. > > Signed-off-by: Jacek Konieczny <jajcus@xxxxxxxxxx> > --- Thanks for spotting; I cannot believe this somehow got through unnoticed. As this fixes b814da89 (pull: add pull.ff configuration, 2014-01-15), it will eventually have to go to both master for 2.1 and maint for 2.0.1. > git-pull.sh | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/git-pull.sh b/git-pull.sh > index 6cd8ebc..7358fac 100755 > --- a/git-pull.sh > +++ b/git-pull.sh > @@ -58,11 +58,9 @@ pull_ff=$(git config pull.ff) > case "$pull_ff" in > false) > no_ff=--no-ff > - break > ;; > only) > ff_only=--ff-only > - break > ;; > esac -- 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