Hans van Kranenburg
2020-Dec-01 18:26 UTC
[Pkg-xen-devel] [PATCH 9/9] d/xendomains.init: Remove use of `xen` compat wrapper for `xl`
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/nullWe 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