[buildbot] phase1: cleanup redundant steps

LEDE Commits lede-commits at lists.infradead.org
Mon May 22 20:55:30 PDT 2023


ynezz pushed a commit to buildbot.git, annotated tag v5:
https://git.openwrt.org/90af3d7ad4a72aab90e2b3bca853a91cf199a753

commit 90af3d7ad4a72aab90e2b3bca853a91cf199a753
Author: Thibaut VARÈNE <hacks at slashdirt.org>
AuthorDate: Mon May 22 20:20:12 2023 +0200

    phase1: cleanup redundant steps
    
    These 3 rm steps are unnecessary since these folders are already cleaned
    by the Git() step. Thus reduce clutter in already pretty busy Factory.
    
    Signed-off-by: Thibaut VARÈNE <hacks at slashdirt.org>
---
 phase1/master.cfg | 19 -------------------
 1 file changed, 19 deletions(-)

diff --git a/phase1/master.cfg b/phase1/master.cfg
index 1658a91..bbd3734 100644
--- a/phase1/master.cfg
+++ b/phase1/master.cfg
@@ -754,19 +754,6 @@ for target in targets:
 		haltOnFailure = True,
 	))
 
-	factory.addStep(ShellCommand(
-		name = "rmtmp",
-		description = "Remove tmp folder",
-		command=["rm", "-rf", "tmp/"],
-	))
-
-	# feed
-	factory.addStep(ShellCommand(
-		name = "rmfeedlinks",
-		description = "Remove feed symlinks",
-		command=["rm", "-rf", "package/feeds/"],
-	))
-
 	factory.addStep(StringDownload(
 		name = "ccachecc",
 		s = '#!/bin/sh\nexec ${CCACHE} ${CCC} "$@"\n',
@@ -815,12 +802,6 @@ for target in targets:
 		command = Interpolate("printf 'CONFIG_TARGET_%(kw:target)s=y\\nCONFIG_TARGET_%(kw:target)s_%(kw:subtarget)s=y\\nCONFIG_SIGNED_PACKAGES=%(kw:usign:#?|y|n)s\\n' >> .config", target=ts[0], subtarget=ts[1], usign=GetUsignKey),
 	))
 
-	factory.addStep(ShellCommand(
-		name = "delbin",
-		description = "Removing output directory",
-		command = ["rm", "-rf", "bin/"],
-	))
-
 	factory.addStep(ShellCommand(
 		name = "defconfig",
 		description = "Populating .config",




More information about the lede-commits mailing list