]> git.ipfire.org Git - thirdparty/git.git/commit - git-submodule.sh
submodule: support reading .gitmodules when it's not in the working tree
authorAntonio Ospite <ao2@ao2.it>
Thu, 25 Oct 2018 16:18:12 +0000 (18:18 +0200)
committerJunio C Hamano <gitster@pobox.com>
Wed, 31 Oct 2018 06:01:30 +0000 (15:01 +0900)
commit76e9bdc437ca3111883eb57a479e5d8f582dd670
tree70442b23252b1c0a3441d41dd9c62e955a3b4c9f
parentb5c259f226ba06f64b399ff4c6b843542352395f
submodule: support reading .gitmodules when it's not in the working tree

When the .gitmodules file is not available in the working tree, try
using the content from the index and from the current branch. This
covers the case when the file is part of the repository but for some
reason it is not checked out, for example because of a sparse checkout.

This makes it possible to use at least the 'git submodule' commands
which *read* the gitmodules configuration file without fully populating
the working tree.

Writing to .gitmodules will still require that the file is checked out,
so check for that before calling config_set_in_gitmodules_file_gently.

Add a similar check also in git-submodule.sh::cmd_add() to anticipate
the eventual failure of the "git submodule add" command when .gitmodules
is not safely writeable; this prevents the command from leaving the
repository in a spurious state (e.g. the submodule repository was cloned
but .gitmodules was not updated because
config_set_in_gitmodules_file_gently failed).

Moreover, since config_from_gitmodules() now accesses the global object
store, it is necessary to protect all code paths which call the function
against concurrent access to the global object store. Currently this
only happens in builtin/grep.c::grep_submodules(), so call
grep_read_lock() before invoking code involving
config_from_gitmodules().

Finally, add t7418-submodule-sparse-gitmodules.sh to verify that reading
from .gitmodules succeeds and that writing to it fails when the file is
not checked out.

NOTE: there is one rare case where this new feature does not work
properly yet: nested submodules without .gitmodules in their working
tree.  This has been documented with a warning and a test_expect_failure
item in t7814, and in this case the current behavior is not altered: no
config is read.

Signed-off-by: Antonio Ospite <ao2@ao2.it>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
builtin/grep.c
builtin/submodule--helper.c
git-submodule.sh
submodule-config.c
t/t7411-submodule-config.sh
t/t7418-submodule-sparse-gitmodules.sh [new file with mode: 0755]
t/t7814-grep-recurse-submodules.sh