summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLeah Rowe <leah@libreboot.org>2024-06-06 01:01:22 +0100
committerLeah Rowe <leah@libreboot.org>2024-06-06 01:01:22 +0100
commitbea089bbe4a720e39ba78850f015c67dbe3d1a43 (patch)
treeef02631630323db158f449d07d7865d2621ec465
parent6e1b8087c5133b4f9a384df6ea7de3250ea8e573 (diff)
don't use build.list to detect multi-tree projects
instead, check for the presence of target.cfg files not in config/project/ but config/project/tree/ the way this check is done, it merely returns 1 if config/project/*/target.cfg is detected, and returns 0 in all other cases, even if config/project/target.cfg exists that way, if the maintainer accidentally adds a target.cfg in the main directory, the given multi-tree project will not break Signed-off-by: Leah Rowe <leah@libreboot.org>
-rwxr-xr-xbuild5
-rwxr-xr-xinclude/lib.sh10
-rwxr-xr-xscript/trees2
3 files changed, 12 insertions, 5 deletions
diff --git a/build b/build
index 748ff4f0..74ea1de6 100755
--- a/build
+++ b/build
@@ -117,10 +117,7 @@ fetch_trees()
{
for x in $(items config/git); do
./update trees -f "$x" || $err "$_xm: fetch $x"
- done
- for x in config/*/build.list; do
- [ -f "$x" ] && xp="${x#*/}" && xp="${xp%/*}"
- [ ! -f "$x" ] || [ -L "$xp" ] || x_ rm -Rf "src/$xp/$xp"
+ singletree "$x" || x_ rm -Rf "src/$x/$x"
done
rmgit .
}
diff --git a/include/lib.sh b/include/lib.sh
index 6a4be7ee..bc2232ad 100755
--- a/include/lib.sh
+++ b/include/lib.sh
@@ -228,3 +228,13 @@ e()
eval "$estr"
printf "%s %s\n" "$1" "$es2" 1>&2
}
+
+# return 0 if project is single-tree, otherwise 1
+# e.g. coreboot is multi-tree, so 1
+singletree()
+{
+ for pdir in "config/${1}/"*/target.cfg; do
+ [ ! -e "$pdir" ] && continue
+ [ -f "$pdir" ] && return 1
+ done
+}
diff --git a/script/trees b/script/trees
index e22467f7..897c6ed3 100755
--- a/script/trees
+++ b/script/trees
@@ -39,7 +39,7 @@ main()
remkdir "${tmpgit%/*}"
_cmd="build_projects"
- [ -f "config/$project/build.list" ] && _cmd="build_targets"
+ singletree "$project" || _cmd="build_targets"
$_cmd $@
}