[OSSTEST PATCH] target_cmd_build: Delete build-ok-stamp before starting

[OSSTEST PATCH] target_cmd_build: Delete build-ok-stamp before starting

[Ian Jackson]

Many of the callers of target_cmd_build use a build-ok-stamp idiom to
detect failed builds.  This idiom does not work if the stamp file
exists already, so delete it.

In the future we may move more of the test build-ok-stamp, echo ok,
into TestSupport, but this will do for now.

Signed-off-by: Ian Jackson <Ian.Jackson@eu.citrix.com>
---
 Osstest/TestSupport.pm |    1 +
 1 file changed, 1 insertion(+)

diff --git a/Osstest/TestSupport.pm b/Osstest/TestSupport.pm
index b50ad0a..3ff70a5 100644
--- a/Osstest/TestSupport.pm
+++ b/Osstest/TestSupport.pm
@@ -593,6 +593,7 @@ END
 END
         exec </dev/null
         cd $builddir
+        rm -f build-ok-stamp
 END
 }
 
-- 
1.7.10.4

Re: [OSSTEST PATCH] target_cmd_build: Delete build-ok-stamp before starting

[Ian Campbell]

On Tue, 2015-04-21 at 17:40 +0100, Ian Jackson wrote:
> Many of the callers of target_cmd_build use a build-ok-stamp idiom to
> detect failed builds.  This idiom does not work if the stamp file
> exists already, so delete it.
> 
> In the future we may move more of the test build-ok-stamp, echo ok,
> into TestSupport, but this will do for now.
> 
> Signed-off-by: Ian Jackson <Ian.Jackson@eu.citrix.com>

Acked-by: Ian Campbell <ian.campbell@citrix.com>