[Pkg-xen-devel] [PATCH 09/13] xen init script: move init_dom0 into xenstored start

[Ian Jackson]

Hans van Kranenburg writes ("[PATCH 09/13] xen init script: move init_dom0 into xenstored start"):
> This little xen-init-dom0 program is present in both our 4.8 and 4.11
> packages, so there's no need to keep the if else code. Also, it only
> makes sense to run it after starting xenstored, and not all other times
> start is called.

I think xen-init-dom0 is not in jessie and although skip-upgrades are
not "supported" I don't see a reason to gratuitously break them.
Would you mind if we deferred removing this compat code until after
buster ?

> @@ -307,11 +299,6 @@ case "$1" in
>  		0|1) ;;
>  		*) log_end_msg 1; exit ;;
>  	esac
> -	init_dom0
> -	case "$?" in
> -		0|1) ;;
> -		*) log_end_msg 1; exit ;;
> -	esac

It would have been less confusing to me if you had separated out the
change to remove the open-coded fallback, from the change to when this
is called.

Also the latter change seems right and here I am complaining about
the former...

Thanks,
Ian.

-- 
Ian Jackson <ijackson at chiark.greenend.org.uk>   These opinions are my own.

If I emailed you from an address @fyvzl.net or @evade.org.uk, that is
a private address which bypasses my fierce spamfilter.