Disable removal of 'target_path' when build starts

diff --git a/ChangeLog b/ChangeLog
index d11a4e544132318d4f165ba2654e31779339839e..d3bc64e23d1b4316cebb3de58b976840e7c95f0f 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,6 +1,10 @@
-# Copyright 2002-2008 Gentoo Foundation; 2008 Chris Gianelloni, Andrew Gaffney
+# Copyright 2002-2009 Gentoo Foundation; 2008 Chris Gianelloni, Andrew Gaffney
 # Distributed under the GPL v2
 
+  02 Jan 2009; Andrew Gaffney <agaffney@gentoo.org>
+  modules/generic_stage_target.py:
+  Disable removal of 'target_path' when build starts
+
   26 Dec 2008; Andrew Gaffney <agaffney@gentoo.org>
   modules/generic_stage_target.py:
   Apply patch from armin76 to actually make use of the busybox_config value

diff --git a/modules/generic_stage_target.py b/modules/generic_stage_target.py
index a0a0aaba69c5b0164b62ab342cbe6c14236b7843..e9938d16314bd92944800aa040d325c7cca5603a 100644 (file)
--- a/modules/generic_stage_target.py
+++ b/modules/generic_stage_target.py
@@ -300,10 +300,13 @@ class generic_stage_target(generic_target):
                                "Resume point detected, skipping target path setup operation..."
                else:
                        """ First clean up any existing target stuff """
-                       if os.path.isfile(self.settings["target_path"]):
-                               cmd("rm -f "+self.settings["target_path"],\
-                                       "Could not remove existing file: "\
-                                       +self.settings["target_path"],env=self.env)
+                       # XXX WTF are we removing the old tarball before we start building the
+                       # XXX new one? If the build fails, you don't want to be left with 
+                       # XXX nothing at all
+#                      if os.path.isfile(self.settings["target_path"]):
+#                              cmd("rm -f "+self.settings["target_path"],\
+#                                      "Could not remove existing file: "\
+#                                      +self.settings["target_path"],env=self.env)
                                touch(self.settings["autoresume_path"]+"setup_target_path")
 
                        if not os.path.exists(self.settings["storedir"]+"/builds/"):