diff options
| author | Patrick Georgi <pgeorgi@chromium.org> | 2015-09-15 16:57:04 +0200 |
|---|---|---|
| committer | Patrick Georgi <pgeorgi@google.com> | 2015-09-16 07:43:34 +0000 |
| commit | 098c4a80962437c6d23dc3fde894a1c3a7bf0ca0 (patch) | |
| tree | c49fdeee5c0855f802dc07caddb331f543e337ee | |
| parent | 9763d8e9d1f9e6d4dc26075544d1f9b20d86c614 (diff) | |
abuild: don't create junit tests with empty testclass field
Variable expansion made abuild create board..foo/bar, which are annoying
on jenkins' web UI because it doesn't cope properly with the empty
namespace between the dots. make it create board.foo/bar or
board.$class.foo/bar.
Change-Id: Ifa79cbfd1f263e11a458b3cc320baeed6a3fbc98
Signed-off-by: Patrick Georgi <pgeorgi@chromium.org>
Reviewed-on: http://review.coreboot.org/11640
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Tested-by: build bot (Jenkins)
| -rwxr-xr-x | util/abuild/abuild | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/util/abuild/abuild b/util/abuild/abuild index 9516da51d5..788ed7d0db 100755 --- a/util/abuild/abuild +++ b/util/abuild/abuild @@ -479,7 +479,8 @@ chromeos=false clean_work=false customizing="" configoptions="" -testclass= +# testclass needs to be undefined if not used for variable expansion to work +unset testclass while true ; do case "$1" in -J|--junit) shift; mode=junit; rm -f $XMLFILE ;; |