Re: [PATCH v2] Fix fetch/pull when run without --update-head-ok
From: Shawn O. Pearce <hidden>
Date: 2016-06-15 22:45:28
Johannes Schindelin [off-list ref] wrote:
Some confusing tutorials suggested that it would be a good idea to fetch into the current branch with something like this: git fetch origin master:master (or even worse: the same command line with "pull" instead of "fetch"). While it might make sense to store what you want to pull, it typically is plain wrong when the current branch is "master". As noticed by Junio, this behavior should be triggered by _not_ passing the --update-head-ok option, but somewhere along the lines we lost that behavior. NOTE: this patch does not completely resurrect the original behavior without --update-head-ok: the check for the current branch is now _only_ performed in non-bare repositories. Signed-off-by: Johannes Schindelin <redacted>
Acked-by: Shawn O. Pearce <redacted>
Strangely, some more tests refused to pass this time, because they did not use --update-head-ok; this was fixed, too.
Not strange, --update-head-ok was busted and the tests took advantage of it. :-\ -- Shawn.