]> git.proxmox.com Git - libgit2.git/commit - src/libgit2/repository.c
repository: make check if repo is a worktree more strict
authorPatrick Steinhardt <ps@pks.im>
Fri, 19 May 2017 07:21:17 +0000 (09:21 +0200)
committerPatrick Steinhardt <ps@pks.im>
Fri, 19 May 2017 07:44:44 +0000 (09:44 +0200)
commit2696c5c3ece1a28c545d6e59b0d121480e9f977d
tree5a9f2a5fe87ce832fb7d9cb9b752c1f82703220f
parent9f9fd05f1cb9278a34a265c49c8567b526e48afd
repository: make check if repo is a worktree more strict

To determine if a repository is a worktree or not, we currently check
for the existence of a "gitdir" file inside of the repository's gitdir.
While this is sufficient for non-broken repositories, we have at least
one case of a subtly broken repository where there exists a gitdir file
inside of a gitmodule. This will cause us to misidentify the submodule
as a worktree.

While this is not really a fault of ours, we can do better here by
observing that a repository can only ever be a worktree iff its common
directory and dotgit directory are different. This allows us to make our
check whether a repo is a worktree or not more strict by doing a simple
string comparison of these two directories. This will also allow us to
do the right thing in the above case of a broken repository, as for
submodules these directories will be the same. At the same time, this
allows us to skip the `stat` check for the "gitdir" file for most
repositories.
src/repository.c
tests/submodule/open.c