Re: [PATCH] Fix new-workdir (again) to work on bare repositories
- From
Shawn O. Pearce <spearce@spearce.org>
- Date
- Aug 22, 2007, 05:33 UTC
- Message-ID
- <20070822053349.GO27913@spearce.org>
- In-Reply-To
- <7v3ayckx1m.fsf@gitster.siamese.dyndns.org>
Junio C Hamano <gitster@pobox.com> wrote:
Show 7 quoted lines
> "Shawn O. Pearce" <spearce@spearce.org> writes: > > So removing that core.bare thing makes the magic work. > > Ok, that is better than the response I sent (our messages > crossed). In that case, perhaps you would want a warning or a > suggestion at the end of new-workdir script, probably before > checkout, to teach the user about that magic?
-->8-- Suggest unsetting core.bare when using new-workdir on a bare repository
If core.bare is set to true in the config file of a repository that the user is trying to create a working directory from we should abort and suggest to the user that they remove the option first.
If we leave the core.bare=true setting in the config file then working tree operations will get confused when they attempt to execute in the new workdir, as it shares its config file with the bare repository. The working tree operations will assume that the workdir is bare and abort, which is not what the user wants.
If we changed core.bare to be false then working tree operations will function in the workdir but other operations may fail in the bare repository, as it claims to not be bare.
If we remove core.bare from the config then Git can fallback on the legacy guessing behavior. This allows operations in the bare repository to work as though it were bare, while operations in the workdirs to act as though they are not bare.
Signed-off-by: Shawn O. Pearce <spearce@spearce.org> --- contrib/workdir/git-new-workdir | 8 ++++++++ 1 files changed, 8 insertions(+), 0 deletions(-)
diff --git a/contrib/workdir/git-new-workdir b/contrib/workdir/git-new-workdir index 119cff9..c6e154a 100755 --- a/contrib/workdir/git-new-workdir +++ b/contrib/workdir/git-new-workdir @@ -33,6 +33,14 @@ case "$git_dir" in ;; esac +# don't link to a configured bare repository +isbare=$(git --git-dir="$git_dir" config --bool --get core.bare) +if test ztrue = z$isbare +then + die "\"$git_dir\" has core.bare set to true," \ + " remove from \"$git_dir/config\" to use $0" +fi + # don't link to a workdir if test -L "$git_dir/config" then
-- 1.5.3.rc6 -- Shawn.