[Xen-devel] [OSSTEST PATCH] cr-daily-branch: seabios: "usually" use xen-tested-master

Ian Jackson posted 1 patch 4 years, 10 months ago
Failed in applying to current master (apply log)
cr-daily-branch | 11 +++++++----
1 file changed, 7 insertions(+), 4 deletions(-)
[Xen-devel] [OSSTEST PATCH] cr-daily-branch: seabios: "usually" use xen-tested-master
Posted by Ian Jackson 4 years, 10 months ago
This branch is supposed to be suitable for all versions of Xen.
Conversely, older versions of seabios do not build on newer
compilers (as provided, eg, in stretch).

So, for "branches" other than xen-unstable and xen-unstable-smoke, use
the usual "determine_version" machinery, which will select
xen-tested-master for branches other than the ovmf branch itself.

No change for the seabios "branch", nor for xen-unstable*.  The effect
is to switch xen-*-testing, qemu-*, linux-*, etc., to all use ovmf
xen-tested-master.

This commit is identical to the corresponding one relating to ovmf,
  c9d1e5896fe2246d17d436f05a9960a66ea55426
  cr-daily-branch: ovmf: "usually" use xen-tested-master
mutandis mutandi.  Verified with
  diff -u <(git-show c9d1e5896fe2246d17d436f05a9960a66ea55426) <(git-diff | sed 's/SEABIOS/OVMF/g; s/seabios/ovmf/g') |less

A cursory examination of the standalone-generate-dump-flight-runvars
output revealed what seemed to be the expected changes and nothing
untoward.

CC: Wei Liu <wei.liu2@citrix.com>
CC: Anthony PERARD <anthony.perard@citrix.com>
Signed-off-by: Ian Jackson <ian.jackson@eu.citrix.com>
---
 cr-daily-branch | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/cr-daily-branch b/cr-daily-branch
index 00b713b3..82e20c79 100755
--- a/cr-daily-branch
+++ b/cr-daily-branch
@@ -194,14 +194,17 @@ if [ "x$REVISION_LINUX_ARM" = x ]; then
     fi
 fi
 if [ "x$REVISION_SEABIOS" = x ]; then
-    if [ "x$tree" = "xseabios" ]; then
+    case "$branch" in
+    xen-unstable*)
+	: REVISION_SEABIOS from Config.mk
+	;;
+    *)
 	TREE_SEABIOS=$TREE_SEABIOS_UPSTREAM
 	export TREE_SEABIOS
 	determine_version REVISION_SEABIOS seabios SEABIOS
 	export REVISION_SEABIOS
-    else
-	: REVISION_SEABIOS from Config.mk
-    fi
+	;;
+    esac
 fi
 if [ "x$REVISION_OVMF" = x ]; then
     case "$branch" in
-- 
2.11.0


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
Re: [Xen-devel] [OSSTEST PATCH] cr-daily-branch: seabios: "usually" use xen-tested-master
Posted by Wei Liu 4 years, 10 months ago
On Wed, May 15, 2019 at 11:07:23AM +0100, Ian Jackson wrote:
> This branch is supposed to be suitable for all versions of Xen.
> Conversely, older versions of seabios do not build on newer
> compilers (as provided, eg, in stretch).
> 
> So, for "branches" other than xen-unstable and xen-unstable-smoke, use
> the usual "determine_version" machinery, which will select
> xen-tested-master for branches other than the ovmf branch itself.
> 
> No change for the seabios "branch", nor for xen-unstable*.  The effect
> is to switch xen-*-testing, qemu-*, linux-*, etc., to all use ovmf
> xen-tested-master.
> 
> This commit is identical to the corresponding one relating to ovmf,
>   c9d1e5896fe2246d17d436f05a9960a66ea55426
>   cr-daily-branch: ovmf: "usually" use xen-tested-master
> mutandis mutandi.  Verified with
>   diff -u <(git-show c9d1e5896fe2246d17d436f05a9960a66ea55426) <(git-diff | sed 's/SEABIOS/OVMF/g; s/seabios/ovmf/g') |less
> 
> A cursory examination of the standalone-generate-dump-flight-runvars
> output revealed what seemed to be the expected changes and nothing
> untoward.
> 
> CC: Wei Liu <wei.liu2@citrix.com>
> CC: Anthony PERARD <anthony.perard@citrix.com>
> Signed-off-by: Ian Jackson <ian.jackson@eu.citrix.com>

Acked-by: Wei Liu <wei.liu2@citrix.com>

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel