Re: [Xen-devel] [PATCH 0/2] tools: split xl into a separate directory

2017-02-23 Thread Wei Liu
On Thu, Feb 23, 2017 at 05:25:32PM +, Ian Jackson wrote: > Wei Liu writes ("Re: [PATCH 0/2] tools: split xl into a separate directory"): > > On Thu, Feb 23, 2017 at 04:59:34PM +, Wei Liu wrote: > > > But I am now very confused how we encode this in our build system. > > > Obviously there is

Re: [Xen-devel] [PATCH 0/2] tools: split xl into a separate directory

2017-02-23 Thread Ian Jackson
Wei Liu writes ("Re: [PATCH 0/2] tools: split xl into a separate directory"): > On Thu, Feb 23, 2017 at 04:59:34PM +, Wei Liu wrote: > > But I am now very confused how we encode this in our build system. > > Obviously there is already such need. Say, all the tools under misc > > would require l

Re: [Xen-devel] [PATCH 0/2] tools: split xl into a separate directory

2017-02-23 Thread Wei Liu
On Thu, Feb 23, 2017 at 04:59:34PM +, Wei Liu wrote: > On Thu, Feb 23, 2017 at 04:29:46PM +, Ian Jackson wrote: > > Wei Liu writes ("Re: [PATCH 0/2] tools: split xl into a separate > > directory"): > > > On Thu, Feb 23, 2017 at 03:54:02PM +, Ian Jackson wrote: > > > > No, this is entir

Re: [Xen-devel] [PATCH 0/2] tools: split xl into a separate directory

2017-02-23 Thread Wei Liu
On Thu, Feb 23, 2017 at 04:29:46PM +, Ian Jackson wrote: > Wei Liu writes ("Re: [PATCH 0/2] tools: split xl into a separate directory"): > > On Thu, Feb 23, 2017 at 03:54:02PM +, Ian Jackson wrote: > > > No, this is entirely wrong. You will end up running make in > > > tools/libxl twice si

Re: [Xen-devel] [PATCH 0/2] tools: split xl into a separate directory

2017-02-23 Thread Ian Jackson
Wei Liu writes ("Re: [PATCH 0/2] tools: split xl into a separate directory"): > On Thu, Feb 23, 2017 at 03:54:02PM +, Ian Jackson wrote: > > No, this is entirely wrong. You will end up running make in > > tools/libxl twice simultaneously. > > Libacpi can be built in different directories by s

Re: [Xen-devel] [PATCH 0/2] tools: split xl into a separate directory

2017-02-23 Thread Wei Liu
On Thu, Feb 23, 2017 at 03:54:02PM +, Ian Jackson wrote: > Wei Liu writes ("Re: [PATCH 0/2] tools: split xl into a separate directory"): > > Basically this needs to be folded into xl/Makefile: > > > > +# libxenlight should be built before building xl > > +.PHONY: libxl > > +libxl: > > +

Re: [Xen-devel] [PATCH 0/2] tools: split xl into a separate directory

2017-02-23 Thread Ian Jackson
Wei Liu writes ("Re: [PATCH 0/2] tools: split xl into a separate directory"): > Basically this needs to be folded into xl/Makefile: > > +# libxenlight should be built before building xl > +.PHONY: libxl > +libxl: > + $(MAKE) -C $(XEN_ROOT)/tools/libxl > +$(XL_OBJS): libxl OMG. > This is in

Re: [Xen-devel] [PATCH 0/2] tools: split xl into a separate directory

2017-02-23 Thread Wei Liu
On Thu, Feb 23, 2017 at 01:20:25PM +, Wei Liu wrote: > On Thu, Feb 23, 2017 at 12:17:38PM +, Ian Jackson wrote: > > Wei Liu writes ("[PATCH 0/2] tools: split xl into a separate directory"): > > > Xl has grown sufficiently large to warrnant its own directory. We also > > > need > > > clear

Re: [Xen-devel] [PATCH 0/2] tools: split xl into a separate directory

2017-02-23 Thread Wei Liu
On Thu, Feb 23, 2017 at 12:17:38PM +, Ian Jackson wrote: > Wei Liu writes ("[PATCH 0/2] tools: split xl into a separate directory"): > > Xl has grown sufficiently large to warrnant its own directory. We also need > > clear separation between the client (xl) and library (libxl). > > This is a f

Re: [Xen-devel] [PATCH 0/2] tools: split xl into a separate directory

2017-02-23 Thread Ian Jackson
Wei Liu writes ("[PATCH 0/2] tools: split xl into a separate directory"): > Xl has grown sufficiently large to warrnant its own directory. We also need > clear separation between the client (xl) and library (libxl). This is a fine idea. Both patches, Acked-by: Ian Jackson Thanks, Ian. ___

[Xen-devel] [PATCH 0/2] tools: split xl into a separate directory

2017-02-21 Thread Wei Liu
Xl has grown sufficiently large to warrnant its own directory. We also need clear separation between the client (xl) and library (libxl). This patch series moves xl into tools/xl directory. Use find to generate a list of files to be installed from staging and wip branch, then `diff -q staging w