From 49d9859c052b95ce157d31eaf2906e32869522ad Mon Sep 17 00:00:00 2001
From: Jo-Philipp Wich <jo@mein.io>
Date: Fri, 4 Nov 2016 15:59:50 +0100
Subject: [PATCH] phase1: remove automatic triggering of clean targets

Remove the automatic triggering of clean targets depending on the changed
files since the code for that is incompatible with BuildBot 0.8.9.

Instead, implement the ability to trigger specific clean steps through
build properties.

Signed-off-by: Jo-Philipp Wich <jo@mein.io>
---
 phase1/master.cfg | 50 ++++++++++++++++++++++++-----------------------
 1 file changed, 26 insertions(+), 24 deletions(-)

diff --git a/phase1/master.cfg b/phase1/master.cfg
index 6030def..f92a080 100644
--- a/phase1/master.cfg
+++ b/phase1/master.cfg
@@ -141,23 +141,23 @@ from buildbot.steps.master import MasterShellCommand
 from buildbot.process.properties import WithProperties
 
 
-MakeTargetMap = {
-	"^tools/":				"tools/clean",
-	"^toolchain/":			"toolchain/clean",
-	"^target/linux/":		"target/linux/clean",
-	"^(config|include)/":	"dirclean"
-}
+CleanTargetMap = [
+	[ "tools",	"tools/clean"			],
+	[ "chain",	"toolchain/clean"		],
+	[ "linux",	"target/linux/clean"	],
+	[ "dir",	"dirclean"				],
+	[ "dist",	"distclean"				]
+]
 
-def IsAffected(pattern):
-	def CheckAffected(change):
-		for request in change.build.requests:
-			for source in request.sources:
-				for change in source.changes:
-					for file in change.files:
-						if re.match(pattern, file):
-							return True
-		return False
-	return CheckAffected
+def IsCleanRequested(pattern):
+	def CheckCleanProperty(step):
+		val = step.getProperty("clean")
+		if val and re.match(pattern, val):
+			return True
+		else:
+			return False
+
+	return CheckCleanProperty
 
 
 c['builders'] = []
@@ -249,6 +249,16 @@ for target in targets:
 			haltOnFailure = True,
 			timeout = 2400))
 
+	# user-requested clean targets
+	else:
+		for tuple in CleanTargetMap:
+			factory.addStep(ShellCommand(
+				name = tuple[1],
+				description = 'User-requested "make %s"' % tuple[1],
+				command = ["make", tuple[1], "V=s"],
+				doStepIf = IsCleanRequested(tuple[0])
+			))
+
 	# check out the source
 	factory.addStep(Git(repourl=repo_url, mode='update'))
 
@@ -361,14 +371,6 @@ EOT''' %(ts[0], ts[0], ts[1]) ))
 		command=["make", "package/base-files/clean", "V=s"]
 	))
 
-	# optional clean steps
-	for pattern, maketarget in MakeTargetMap.items():
-		factory.addStep(ShellCommand(
-			name = maketarget,
-			description = maketarget,
-			command=["make", maketarget, "V=s"], doStepIf=IsAffected(pattern)
-		))
-
 	# build
 	factory.addStep(ShellCommand(
 		name = "tools",
-- 
2.30.2