[Pkg-xen-devel] [PATCH 9/9] d/xendomains.init: Remove use of `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 may have been some benefit
> to using a `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`.
> 
> 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..cd6902d26e 100644
> --- a/debian/xen-utils-common.xendomains.init
> +++ b/debian/xen-utils-common.xendomains.init
> @@ -31,6 +31,11 @@ fi
>  
>  shopt -s nullglob
>  
> +xen()
> +{
> +	xl "$@"
> +}
> +
>  check_config_name()
>  {
>    /usr/lib/xen-common/bin/xen-init-name "$1" 2>/dev/null

We had a small discussion related to this topic yesterday:
https://salsa.debian.org/xen-team/debian-xen/-/issues/29

I'm interested in your opinion/feedback on this. We could also
(sym|hard)link xen to xl now, directly in the filesystem in the packages.

Hans