diff mbox series

[OSSTEST,02/15] TestSupport: target_var: Use host_V for host variables

Message ID 20190426164002.22381-3-ian.jackson@eu.citrix.com (mailing list archive)
State Superseded
Headers show
Series Do armhf kernel builds on amd64 | expand

Commit Message

Ian Jackson April 26, 2019, 4:39 p.m. UTC 
Change `target_var' to set `IDENT_V' rather than just V.  For
compatibility with older flights and older flight construction,
look for plain V too when looking up the variable.

And, we now look at all_host_V before V.  This has no functional
change with existing flights, because existing flights only have
  all_host_suite
  all_host_di_version
  all_host_os
and we never set the corresponding V form of those variables.

So with existing flights the only functional change is a change to
synth runvars, to add HOST_ to the name.

Signed-off-by: Ian Jackson <ian.jackson@eu.citrix.com>
---
 Osstest/TestSupport.pm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
diff mbox series

Patch

diff --git a/Osstest/TestSupport.pm b/Osstest/TestSupport.pm
index fcbde0a2..6ab64d56 100644
--- a/Osstest/TestSupport.pm
+++ b/Osstest/TestSupport.pm
@@ -2465,8 +2465,7 @@  sub guest_await ($$) {
 
 sub target_var_prefix ($) {
     my ($ho) = @_;
-    if (exists $ho->{Guest}) { return $ho->{Guest}.'_'; }
-    return '';
+    return (exists $ho->{Guest} ? $ho->{Guest} : $ho->{Ident}).'_';
 }
 
 sub target_var ($$) {
@@ -2477,6 +2476,7 @@  sub target_var ($$) {
 	push @prefixes, 'all_guest_';
     } else {
 	push @prefixes, 'all_host_';
+	push @prefixes, '';
     }
     foreach my $prefix (@prefixes) {
 	my $v = $r{ $prefix.$vn };