* [PATCH] tools/Makefile: fix qemu-xen-traditional build
@ 2015-01-25 15:38 Wei Liu
2015-01-26 8:31 ` Olaf Hering
2015-01-26 9:55 ` Ian Campbell
0 siblings, 2 replies; 7+ messages in thread
From: Wei Liu @ 2015-01-25 15:38 UTC (permalink / raw)
To: xen-devel; +Cc: Ian Jackson, Wei Liu, Ian Campbell
In d9740237a ("tools: unhook blktap1 from the build and remove all
references to it"), one spot was left unchanged, which leads to failure
in building qemu-xen-traditional.
Signed-off-by: Wei Liu <wei.liu2@citrix.com>
Cc: Ian Campbell <ian.campbell@citrix.com>
Cc: Ian Jackson <ian.jackson@eu.citrix.com>
---
tools/Makefile | 1 +
1 file changed, 1 insertion(+)
diff --git a/tools/Makefile b/tools/Makefile
index 1ad7a5d..5d7a75f 100644
--- a/tools/Makefile
+++ b/tools/Makefile
@@ -158,6 +158,7 @@ qemu-xen-traditional-dir-force-update: qemu-xen-traditional-dir-find
subdir-all-qemu-xen-traditional-dir: qemu-xen-traditional-dir-find
set -e; \
$(buildmakevars2shellvars); \
+ export CONFIG_BLKTAP1=n; \
cd qemu-xen-traditional-dir; \
$(QEMU_ROOT)/xen-setup \
$(IOEMU_EXTRA_LDFLAGS) \
--
1.9.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH] tools/Makefile: fix qemu-xen-traditional build
2015-01-25 15:38 [PATCH] tools/Makefile: fix qemu-xen-traditional build Wei Liu
@ 2015-01-26 8:31 ` Olaf Hering
2015-01-26 10:01 ` Wei Liu
2015-01-26 9:55 ` Ian Campbell
1 sibling, 1 reply; 7+ messages in thread
From: Olaf Hering @ 2015-01-26 8:31 UTC (permalink / raw)
To: Wei Liu; +Cc: Ian Jackson, Ian Campbell, xen-devel
On Sun, Jan 25, Wei Liu wrote:
> In d9740237a ("tools: unhook blktap1 from the build and remove all
> references to it"), one spot was left unchanged, which leads to failure
> in building qemu-xen-traditional.
Another one. I suspect there is more like that in qemu-xen-traditional.
Should I spent/waste my time to convert qemu-xen-traditional to handle
options via configure instead of relying on variables from the Xen
buildsystem? The same happend when my --prefix series was applied.
Olaf
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] tools/Makefile: fix qemu-xen-traditional build
2015-01-25 15:38 [PATCH] tools/Makefile: fix qemu-xen-traditional build Wei Liu
2015-01-26 8:31 ` Olaf Hering
@ 2015-01-26 9:55 ` Ian Campbell
2015-01-26 11:05 ` Wei Liu
1 sibling, 1 reply; 7+ messages in thread
From: Ian Campbell @ 2015-01-26 9:55 UTC (permalink / raw)
To: Wei Liu; +Cc: Ian Jackson, xen-devel
On Sun, 2015-01-25 at 15:38 +0000, Wei Liu wrote:
> In d9740237a ("tools: unhook blktap1 from the build and remove all
> references to it"), one spot was left unchanged, which leads to failure
> in building qemu-xen-traditional.
Well spotted, which make invocation hits this path? make dist (what I
test) didn't.
> Signed-off-by: Wei Liu <wei.liu2@citrix.com>
> Cc: Ian Campbell <ian.campbell@citrix.com>
> Cc: Ian Jackson <ian.jackson@eu.citrix.com>
Acked-by: Ian Campbell <ian.campbell@citrix.com>
Should we be worried that the -all- and -install- targets are similar
but not identical? Specifically one of them lack --extra-cflags?
Perhaps we should refactor those so the options are only given ones?
> ---
> tools/Makefile | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/tools/Makefile b/tools/Makefile
> index 1ad7a5d..5d7a75f 100644
> --- a/tools/Makefile
> +++ b/tools/Makefile
> @@ -158,6 +158,7 @@ qemu-xen-traditional-dir-force-update: qemu-xen-traditional-dir-find
> subdir-all-qemu-xen-traditional-dir: qemu-xen-traditional-dir-find
> set -e; \
> $(buildmakevars2shellvars); \
> + export CONFIG_BLKTAP1=n; \
> cd qemu-xen-traditional-dir; \
> $(QEMU_ROOT)/xen-setup \
> $(IOEMU_EXTRA_LDFLAGS) \
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] tools/Makefile: fix qemu-xen-traditional build
2015-01-26 8:31 ` Olaf Hering
@ 2015-01-26 10:01 ` Wei Liu
2015-01-26 10:06 ` Ian Campbell
0 siblings, 1 reply; 7+ messages in thread
From: Wei Liu @ 2015-01-26 10:01 UTC (permalink / raw)
To: Olaf Hering; +Cc: Ian Jackson, Wei Liu, Ian Campbell, xen-devel
On Mon, Jan 26, 2015 at 09:31:44AM +0100, Olaf Hering wrote:
> On Sun, Jan 25, Wei Liu wrote:
>
> > In d9740237a ("tools: unhook blktap1 from the build and remove all
> > references to it"), one spot was left unchanged, which leads to failure
> > in building qemu-xen-traditional.
>
> Another one. I suspect there is more like that in qemu-xen-traditional.
> Should I spent/waste my time to convert qemu-xen-traditional to handle
> options via configure instead of relying on variables from the Xen
> buildsystem? The same happend when my --prefix series was applied.
>
qemu-xen-traditional's configure is quite ancient I don't think that's
worth it.
> Olaf
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] tools/Makefile: fix qemu-xen-traditional build
2015-01-26 10:01 ` Wei Liu
@ 2015-01-26 10:06 ` Ian Campbell
0 siblings, 0 replies; 7+ messages in thread
From: Ian Campbell @ 2015-01-26 10:06 UTC (permalink / raw)
To: Wei Liu; +Cc: Olaf Hering, Ian Jackson, xen-devel
On Mon, 2015-01-26 at 10:01 +0000, Wei Liu wrote:
> On Mon, Jan 26, 2015 at 09:31:44AM +0100, Olaf Hering wrote:
> > On Sun, Jan 25, Wei Liu wrote:
> >
> > > In d9740237a ("tools: unhook blktap1 from the build and remove all
> > > references to it"), one spot was left unchanged, which leads to failure
> > > in building qemu-xen-traditional.
> >
> > Another one. I suspect there is more like that in qemu-xen-traditional.
> > Should I spent/waste my time to convert qemu-xen-traditional to handle
> > options via configure instead of relying on variables from the Xen
> > buildsystem? The same happend when my --prefix series was applied.
> >
>
> qemu-xen-traditional's configure is quite ancient I don't think that's
> worth it.
It's also not autoconf (neither is mainline qemu's AFAIK), it's just a
hand written shell script...
Ian.
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] tools/Makefile: fix qemu-xen-traditional build
2015-01-26 9:55 ` Ian Campbell
@ 2015-01-26 11:05 ` Wei Liu
2015-01-27 17:20 ` Ian Campbell
0 siblings, 1 reply; 7+ messages in thread
From: Wei Liu @ 2015-01-26 11:05 UTC (permalink / raw)
To: Ian Campbell; +Cc: Ian Jackson, Wei Liu, xen-devel
On Mon, Jan 26, 2015 at 09:55:41AM +0000, Ian Campbell wrote:
> On Sun, 2015-01-25 at 15:38 +0000, Wei Liu wrote:
> > In d9740237a ("tools: unhook blktap1 from the build and remove all
> > references to it"), one spot was left unchanged, which leads to failure
> > in building qemu-xen-traditional.
>
> Well spotted, which make invocation hits this path? make dist (what I
> test) didn't.
>
make -C tools build.
>
> > Signed-off-by: Wei Liu <wei.liu2@citrix.com>
> > Cc: Ian Campbell <ian.campbell@citrix.com>
> > Cc: Ian Jackson <ian.jackson@eu.citrix.com>
>
> Acked-by: Ian Campbell <ian.campbell@citrix.com>
>
> Should we be worried that the -all- and -install- targets are similar
> but not identical? Specifically one of them lack --extra-cflags?
>
Perhaps nobody ever provides EXTRA_CFLAGS_QEMU_TRADITIONAL so nobody
ever trips over that.
> Perhaps we should refactor those so the options are only given ones?
>
This is a good idea. I can do another patch later.
Wei.
> > ---
> > tools/Makefile | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/tools/Makefile b/tools/Makefile
> > index 1ad7a5d..5d7a75f 100644
> > --- a/tools/Makefile
> > +++ b/tools/Makefile
> > @@ -158,6 +158,7 @@ qemu-xen-traditional-dir-force-update: qemu-xen-traditional-dir-find
> > subdir-all-qemu-xen-traditional-dir: qemu-xen-traditional-dir-find
> > set -e; \
> > $(buildmakevars2shellvars); \
> > + export CONFIG_BLKTAP1=n; \
> > cd qemu-xen-traditional-dir; \
> > $(QEMU_ROOT)/xen-setup \
> > $(IOEMU_EXTRA_LDFLAGS) \
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] tools/Makefile: fix qemu-xen-traditional build
2015-01-26 11:05 ` Wei Liu
@ 2015-01-27 17:20 ` Ian Campbell
0 siblings, 0 replies; 7+ messages in thread
From: Ian Campbell @ 2015-01-27 17:20 UTC (permalink / raw)
To: Wei Liu; +Cc: Ian Jackson, xen-devel
On Mon, 2015-01-26 at 11:05 +0000, Wei Liu wrote:
> On Mon, Jan 26, 2015 at 09:55:41AM +0000, Ian Campbell wrote:
> > On Sun, 2015-01-25 at 15:38 +0000, Wei Liu wrote:
> > > In d9740237a ("tools: unhook blktap1 from the build and remove all
> > > references to it"), one spot was left unchanged, which leads to failure
> > > in building qemu-xen-traditional.
> >
> > Well spotted, which make invocation hits this path? make dist (what I
> > test) didn't.
> >
>
> make -C tools build.
>
> >
> > > Signed-off-by: Wei Liu <wei.liu2@citrix.com>
> > > Cc: Ian Campbell <ian.campbell@citrix.com>
> > > Cc: Ian Jackson <ian.jackson@eu.citrix.com>
> >
> > Acked-by: Ian Campbell <ian.campbell@citrix.com>
And applied.
> >
> > Should we be worried that the -all- and -install- targets are similar
> > but not identical? Specifically one of them lack --extra-cflags?
> >
>
> Perhaps nobody ever provides EXTRA_CFLAGS_QEMU_TRADITIONAL so nobody
> ever trips over that.
Probably true, or at least those people who do use make dist etc.
> > Perhaps we should refactor those so the options are only given ones?
> >
>
> This is a good idea. I can do another patch later.
Thanks..
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2015-01-27 17:20 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-01-25 15:38 [PATCH] tools/Makefile: fix qemu-xen-traditional build Wei Liu
2015-01-26 8:31 ` Olaf Hering
2015-01-26 10:01 ` Wei Liu
2015-01-26 10:06 ` Ian Campbell
2015-01-26 9:55 ` Ian Campbell
2015-01-26 11:05 ` Wei Liu
2015-01-27 17:20 ` Ian Campbell
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.