diff mbox

[Xen-devel,osstest] ts-xen-build: collect built seabios and ovmf versions

Message ID 1398442268-18183-1-git-send-email-ian.campbell@citrix.com
State New
Headers show

Commit Message

Ian Campbell April 25, 2014, 4:11 p.m. UTC 
The seabios path was incorrect and ovmf was not present.

Take the opportunity to rename collectversion_qemu to collectversions.

Signed-off-by: Ian Campbell <ian.campbell@citrix.com>
Cc: Wei Liu <wei.liu2@citrix.com>
---
 ts-xen-build | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

Comments

Ian Jackson April 30, 2014, 3:16 p.m. UTC | #1 
Ian Campbell writes ("[PATCH osstest] ts-xen-build: collect built seabios and ovmf versions"):
> The seabios path was incorrect and ovmf was not present.

Acked-by: Ian Jackson <ian.jackson@eu.citrix.com>
Ian Campbell May 1, 2014, 4 p.m. UTC | #2 
On Wed, 2014-04-30 at 16:16 +0100, Ian Jackson wrote:
> Ian Campbell writes ("[PATCH osstest] ts-xen-build: collect built seabios and ovmf versions"):
> > The seabios path was incorrect and ovmf was not present.
> 
> Acked-by: Ian Jackson <ian.jackson@eu.citrix.com>

Thanks, pushed.
diff mbox

Patch

diff --git a/ts-xen-build b/ts-xen-build
index 7e0b78a..7ac46ea 100755
--- a/ts-xen-build
+++ b/ts-xen-build
@@ -116,12 +116,13 @@  END
 END
 }
 
-sub collectversion_qemu () {
+sub collectversions () {
     my $tools="$builddir/xen-unstable/tools";
     store_revision($ho, 'qemu', "$tools/ioemu-dir", 1);
     store_revision($ho, 'qemu', "$tools/qemu-xen-traditional-dir", 1);
     store_revision($ho, 'qemuu', "$tools/qemu-xen-dir", 1);
-    store_revision($ho, 'seabios', "$tools/seabios-dir", 1);
+    store_revision($ho, 'seabios', "$tools/firmware/seabios-dir", 1);
+    store_revision($ho, 'ovmf', "$tools/firmware/ovmf-dir", 1);
 }
 
 sub divide () {
@@ -185,7 +186,7 @@  sub trapping ($) {
 checkout();
 
 trapping(\&build);
-trapping(\&collectversion_qemu);
+trapping(\&collectversions);
 
 die "*** something failed:\n\n".(join "\n\n",@probs)."\n** something failed"
     if @probs;