From: Thibaut VARÈNE <hacks@slashdirt.org>
Date: Mon, 24 Oct 2022 14:41:00 +0000 (+0200)
Subject: phase1: abort early if current builder does not apply
X-Git-Tag: v2~34
X-Git-Url: http://git.lede-project.org./?a=commitdiff_plain;h=80da7fa93ba954aa09b2a10a43fea6e8f737657a;p=buildbot.git

phase1: abort early if current builder does not apply

Some builders (target/subtarget) may only apply to particular branches,
but are nevertheless defined for all branches. Detect such case early,
stop the build but do not mark it FAILURE.

Signed-off-by: Thibaut VARÈNE <hacks@slashdirt.org>
---

diff --git a/phase1/master.cfg b/phase1/master.cfg
index 9453958..e3a2d3e 100644
--- a/phase1/master.cfg
+++ b/phase1/master.cfg
@@ -722,15 +722,16 @@ for target in targets:
 		env = MakeEnv()
 	))
 
-	# check arch
+	# check arch - exit early if does not exist - NB: some targets do not define CONFIG_TARGET_target_subtarget
 	factory.addStep(ShellCommand(
 		name = "checkarch",
 		description = "Checking architecture",
-		command = ["grep", "-sq", "CONFIG_TARGET_%s=y" %(ts[0]), ".config"],
+		command = 'grep -sq CONFIG_TARGET_%s=y .config && grep -sq CONFIG_TARGET_SUBTARGET=\\"%s\\" .config' %(ts[0], ts[1]),
 		logEnviron = False,
 		want_stdout = False,
 		want_stderr = False,
-		haltOnFailure = True
+		haltOnFailure = True,
+		flunkOnFailure = False,	# this is not a build FAILURE
 	))
 
 	# find libc suffix