summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMichał Górny <mgorny@gentoo.org>2019-11-13 15:06:35 +0100
committerMichał Górny <mgorny@gentoo.org>2020-01-10 19:27:18 +0100
commit23717189bcef73c1b4604282de58daaf14a17b43 (patch)
tree7fa5c486ef54fecf836078c2f45f4273fe2b3b9d /eclass/git-r3.eclass
parentmedia-libs/glm: Fix glm.pc (diff)
downloadgentoo-23717189bcef73c1b4604282de58daaf14a17b43.tar.gz
gentoo-23717189bcef73c1b4604282de58daaf14a17b43.tar.bz2
gentoo-23717189bcef73c1b4604282de58daaf14a17b43.zip
git-r3.eclass: Handle recursive submodules in EGIT_SUBMODULES
Match recursive submodules using their full paths rather than path relatively to the parent submodule. Closes: https://bugs.gentoo.org/694494 Signed-off-by: Michał Górny <mgorny@gentoo.org>
Diffstat (limited to 'eclass/git-r3.eclass')
-rw-r--r--eclass/git-r3.eclass27
1 files changed, 19 insertions, 8 deletions
diff --git a/eclass/git-r3.eclass b/eclass/git-r3.eclass
index 144236c6ac38..663fd939b295 100644
--- a/eclass/git-r3.eclass
+++ b/eclass/git-r3.eclass
@@ -401,16 +401,22 @@ _git-r3_set_gitdir() {
}
# @FUNCTION: _git-r3_set_submodules
-# @USAGE: <file-contents>
+# @USAGE: <parent-path> <file-contents>
# @INTERNAL
# @DESCRIPTION:
# Parse .gitmodules contents passed as <file-contents>
# as in "$(cat .gitmodules)"). Composes a 'submodules' array that
# contains in order (name, URL, path) for each submodule.
+#
+# <parent-path> specifies path to current submodule (empty if top repo),
+# and is used to support recursively specifying submodules. The path
+# must include a trailing slash if it's not empty.
_git-r3_set_submodules() {
debug-print-function ${FUNCNAME} "$@"
- local data=${1}
+ local parent_path=${1}
+ local data=${2}
+ [[ -z ${parent_path} || ${parent_path} == */ ]] || die
# ( name url path ... )
submodules=()
@@ -435,12 +441,14 @@ _git-r3_set_submodules() {
l_res=1
fi
- [[ ${subname} == ${p} ]] && res=${l_res}
+ [[ ${parent_path}${subname} == ${p} ]] && res=${l_res}
done
if [[ ! ${res} ]]; then
- einfo "Skipping submodule ${subname}"
+ einfo "Skipping submodule ${parent_path}${subname}"
continue
+ else
+ einfo "Using submodule ${parent_path}${subname}"
fi
fi
@@ -546,7 +554,7 @@ _git-r3_is_local_repo() {
# This default should be fine unless you are fetching multiple trees
# from the same repository in the same ebuild.
#
-# <commit-id> requests attempting to use repository state as of specific
+# <commit-date> requests attempting to use repository state as of specific
# date. For more details, see EGIT_COMMIT_DATE.
#
# The fetch operation will affect the EGIT_STORE only. It will not touch
@@ -817,7 +825,7 @@ git-r3_fetch() {
# recursively fetch submodules
if git cat-file -e "${local_ref}":.gitmodules &>/dev/null; then
local submodules
- _git-r3_set_submodules \
+ _git-r3_set_submodules "${_GIT_SUBMODULE_PATH}" \
"$(git cat-file -p "${local_ref}":.gitmodules || die)"
while [[ ${submodules[@]} ]]; do
@@ -839,7 +847,9 @@ git-r3_fetch() {
local subrepos
_git-r3_set_subrepos "${url}" "${repos[@]}"
- git-r3_fetch "${subrepos[*]}" "${commit}" "${local_id}/${subname}"
+ _GIT_SUBMODULE_PATH=${_GIT_SUBMODULE_PATH}${path}/ \
+ git-r3_fetch "${subrepos[*]}" "${commit}" \
+ "${local_id}/${subname}" ""
fi
submodules=( "${submodules[@]:3}" ) # shift
@@ -975,7 +985,7 @@ git-r3_checkout() {
# recursively checkout submodules
if [[ -f ${out_dir}/.gitmodules && ! ${checkout_paths} ]]; then
local submodules
- _git-r3_set_submodules \
+ _git-r3_set_submodules "${_GIT_SUBMODULE_PATH}" \
"$(<"${out_dir}"/.gitmodules)"
while [[ ${submodules[@]} ]]; do
@@ -989,6 +999,7 @@ git-r3_checkout() {
local subrepos
_git-r3_set_subrepos "${url}" "${repos[@]}"
+ _GIT_SUBMODULE_PATH=${_GIT_SUBMODULE_PATH}${path}/ \
git-r3_checkout "${subrepos[*]}" "${out_dir}/${path}" \
"${local_id}/${subname}"
fi