summaryrefslogtreecommitdiff
path: root/include
diff options
context:
space:
mode:
authorLeah Rowe <leah@libreboot.org>2024-06-07 12:50:55 +0100
committerLeah Rowe <leah@libreboot.org>2024-06-07 12:52:52 +0100
commit9c1ea8f93ac08a0a0aac5e0f71564a376151859e (patch)
tree2c60edb5614e4d098e4194107e18352a00dbd01e /include
parent137321ebc0be7f4f79c7c48ebd6d49f945067849 (diff)
git.sh: *never* run git submodule update
only use config/submodules/ which the build system then uses to run git clones manually, replicating the submodules feature. we must never use a project's own gitmodules feature, because we can't easily control it. better to let it break first, and then figure out what modules to add manually, so that we have only what we need for each project. it's done this way, because git's own submodules feature doesn't have very good error checking in general, nor does it have good redundancy. with the current design, we can declare backup repositories for each submodule. we replicate it precisely. for example: 3rdparty/vboot this is a coreboot submodule, and we handle that in the coreboot trees. however, our current design also allows you to do this even if the upstream repository does not contain a .gitmodules file Signed-off-by: Leah Rowe <leah@libreboot.org>
Diffstat (limited to 'include')
-rwxr-xr-xinclude/git.sh6
1 files changed, 0 insertions, 6 deletions
diff --git a/include/git.sh b/include/git.sh
index 6c5c2718..8af1f0d5 100755
--- a/include/git.sh
+++ b/include/git.sh
@@ -109,9 +109,6 @@ prep_submodules()
if [ -f "$mdir/module.list" ]; then
cat "$mdir/module.list" > "$tmpdir/modules" || \
$err "!cp $mdir/module.list $tmpdir/modules"
- elif [ -f "$tmpgit/.gitmodules" ]; then
- git -C "$tmpgit" submodule status | awk '{print $2}' > \
- "$tmpdir/modules" || $err "$mdir: cannot list submodules"
else
return 0
fi
@@ -141,9 +138,6 @@ fetch_submodule()
[ -d "$tmpgit/$1" ] && break
done
[ -d "$tmpgit/$1" ] || $err "!clone $mod $project $mcfgdir $1"
- else
- git -C "$tmpgit" submodule update --init --checkout -- "$1" \
- || $err "$mdir: !update $1"
fi
}