Re: [Xen-devel] [PATCH 03/11] tools: install XEN_{LOG,RUN}_DIR

[Wei Liu <wei.liu2@xxxxxxxxxx>]

On Thu, Jun 09, 2016 at 02:13:23PM +0100, Andrew Cooper wrote:
> On 09/06/16 13:57, Wei Liu wrote:
> > Signed-off-by: Wei Liu <wei.liu2@xxxxxxxxxx>
> > ---
> > Cc: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>
> > ---
> >  tools/Makefile | 3 ++-
> >  1 file changed, 2 insertions(+), 1 deletion(-)
> >
> > diff --git a/tools/Makefile b/tools/Makefile
> > index 6440dec..d43a229 100644
> > --- a/tools/Makefile
> > +++ b/tools/Makefile
> > @@ -61,7 +61,8 @@ build all: subdirs-all
> >  .PHONY: install
> >  install: subdirs-install
> >     $(INSTALL_DIR) -m 700 $(DESTDIR)$(XEN_DUMP_DIR)
> > -   $(INSTALL_DIR) $(DESTDIR)/var/log/xen
> > +   $(INSTALL_DIR) $(DESTDIR)$(XEN_LOG_DIR)
> > +   $(INSTALL_DIR) $(DESTDIR)$(XEN_RUN_DIR)
> >     $(INSTALL_DIR) $(DESTDIR)/var/lib/xen
> 
> Looking at a `git grep`, /var/lib/xen is another path in need of a name
> and less hard coding.
> 

Yeah, I started off only looking at /var/log/xen and found issues with
XEN_RUN_DIR as well.

The one you mentioned doesn't seem to have a XEN_*_DIR set, so I left it
untouched. I have it on my list already.

Wei.

> ~Andrew

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