git.sh: download xtree *before*, not after

downloading it after means that if an error occurs
when downloading the xtree project, the main project
will still be there and nothing will mandate the
downloading of the xtree project. whereas, if we
grab the xtree project first, then the main project
won't get saved to src/

this makes the build system a bit more resilient under
fault conditions, but otherwise doesn't change behaviour.

Signed-off-by: Leah Rowe <leah@libreboot.org>
20240612_branch
Leah Rowe 2024-06-09 11:55:30 +01:00
parent 6a3d8a9600
commit fbcdf33f93
1 changed files with 2 additions and 2 deletions

View File

@ -56,6 +56,8 @@ fetch_project_repo()
[ -z "${loc+x}" ] && $err "fetch_project_repo $project: loc not set" [ -z "${loc+x}" ] && $err "fetch_project_repo $project: loc not set"
[ -z "${url+x}" ] && $err "fetch_project_repo $project: url not set" [ -z "${url+x}" ] && $err "fetch_project_repo $project: url not set"
[ -n "$xtree" ] && [ ! -d "src/coreboot/$xtree" ] && \
x_ ./update trees -f coreboot "$xtree"
[ -z "$depend" ] || for d in $depend ; do [ -z "$depend" ] || for d in $depend ; do
printf "'%s' needs dependency '%s'; grabbing '%s' now\n" \ printf "'%s' needs dependency '%s'; grabbing '%s' now\n" \
"$project" "$d" "$d" "$project" "$d" "$d"
@ -172,8 +174,6 @@ move_repo()
{ {
[ "$1" = "${1%/*}" ] || x_ mkdir -p "${1%/*}" [ "$1" = "${1%/*}" ] || x_ mkdir -p "${1%/*}"
mv "$tmpgit" "$1" || $err "git_prep: !mv $tmpgit $1" mv "$tmpgit" "$1" || $err "git_prep: !mv $tmpgit $1"
[ -n "$xtree" ] && [ ! -d "src/coreboot/$xtree" ] && \
x_ ./update trees -f coreboot "$xtree"; return 0
} }
# can delete from multi- and single-tree projects. # can delete from multi- and single-tree projects.