[Pkg-xen-devel] [PATCH 10/16] d/xendomains.init: Use internal `xen` compat wrapper for `xl`

[Hans van Kranenburg]

Hi,

On 10/4/20 2:46 AM, Elliott Mitchell wrote:
> During the transition from `xm` to `xl` there was some benefit to using
> an external `xen` wrapper to call the appropriate one.  Now `xl` is the
> only one in use and has been the only one for some time.  As such adjust
> the `xendomains` script to use an internal wrapper instead.
> 
> I am highly tempted to purge the `xen` wrapper, but could understand
> some users desiring to keep the compatibility around.  Perhaps for a
> future where something replaces `xl`.

If there will ever be a successor to xl, then we'll probably know about
it happening a few Xen releases before it actually becomes the
recommended default.

I'm all for cutting out all the compatibility toolstack wrapper stuff,
using xl instead of xen in all places ourselves, and just making xen a
symlink to xl for backwards compatibility for 'user muscle memory'.

> Signed-off-by: Elliott Mitchell <ehem+debian at m5p.com>
> ---
>  debian/xen-utils-common.xendomains.init | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/debian/xen-utils-common.xendomains.init b/debian/xen-utils-common.xendomains.init
> index ca03ae4cfe..e01d41beb6 100644
> --- a/debian/xen-utils-common.xendomains.init
> +++ b/debian/xen-utils-common.xendomains.init
> @@ -17,6 +17,11 @@
>  . /lib/init/vars.sh
>  . /lib/lsb/init-functions
>  
> +xen()
> +{
> +	xl "$@"
> +}
> +
>  xen list &> /dev/null
>  if test $? -ne 0
>  then
> 

Hans