[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH 42/46] Use XEN_RUN_DIR instead of /var/run/xen



On Tue, Sep 23, Ian Campbell wrote:

> On Mon, 2014-09-22 at 15:00 +0200, Olaf Hering wrote:
> > Also remove duplicate XEN_RUN_DIR from m4/paths.m4.
> > Fix XEN_RUN_DIR expansion in toplevel Makefile
> > 
> > Signed-off-by: Olaf Hering <olaf@xxxxxxxxx>
> > ---
> >  Makefile                                                          | 2 +-
> >  m4/paths.m4                                                       | 3 ---
> >  tools/hotplug/Linux/systemd/xen-qemu-dom0-disk-backend.service.in | 2 +-
> >  3 files changed, 2 insertions(+), 5 deletions(-)
> > 
> > diff --git a/Makefile b/Makefile
> > index 7558262..b0742cf 100644
> > --- a/Makefile
> > +++ b/Makefile
> > @@ -221,7 +221,7 @@ uninstall:
> >     rm -f  $(D)$(XEN_SYSTEMD_DIR)/*.service
> >     rm -f  $(D)$(XEN_SYSTEMD_DIR)/*.mount
> >     rm -f  $(D)$(XEN_SYSTEMD_MODULES_LOAD)/*.conf
> > -   rm -rf $(D)$(XEN_RUN_DIR)* $(D)/var/lib/xen*
> > +   rm -rf $(D)$(XEN_RUN_DIR)/* $(D)/var/lib/xen*
> 
> This would previous have removed XEN_RUN_DIR itself and not just the
> contents. It would also have historically also cleaned up
> $(XEN_RUN_DIR)d (i.e. xend...)!
> 
> Perhaps just add an rmdir to follow this, globbing the dir is indeed
> pretty skanky (and probably annoies e.g. the xenomai maintainers!)

The uninstall target is indeed special. It leaves half of the make
install result on the system. And the globbing needs more work.

I will address this in the next version of the patch.

Olaf

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.