From 90af3d7ad4a72aab90e2b3bca853a91cf199a753 Mon Sep 17 00:00:00 2001
From: =?utf8?q?Thibaut=20VAR=C3=88NE?= <hacks@slashdirt.org>
Date: Mon, 22 May 2023 20:20:12 +0200
Subject: [PATCH] phase1: cleanup redundant steps
MIME-Version: 1.0
Content-Type: text/plain; charset=utf8
Content-Transfer-Encoding: 8bit

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@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",
-- 
2.30.2