Re: [Qemu-devel] [PATCH v16 2/9] rules.mak: allow per object cflags and libs

2014-01-16 Thread Paolo Bonzini
Il 16/01/2014 13:40, Fam Zheng ha scritto: > , compared to > > $(obj)/foo.mo-objs := $(obj)/bar.o $(obj)/biz.o $(obj)/qux.o > > or slightly better > > $(obj)/foo.mo-objs := $(addprefix $(obj), bar.o, biz.o, qux.o) > > I've already converted block/curl.o's cflags to this, later in this s

Re: [Qemu-devel] [PATCH v16 2/9] rules.mak: allow per object cflags and libs

2014-01-16 Thread Fam Zheng
On Thu, 01/16 11:04, Peter Maydell wrote: > On 15 January 2014 08:48, Fam Zheng wrote:.objs. > > Similarly, > > > > foo.o-cflags := $(FOO_CFLAGS) > > > > is also supported. > > I noticed that we already support per-object cflags via: > > $(obj)/adlib.o $(obj)/fmopl.o: QEMU_CFLAGS += -DBU

Re: [Qemu-devel] [PATCH v16 2/9] rules.mak: allow per object cflags and libs

2014-01-16 Thread Peter Maydell
On 15 January 2014 08:48, Fam Zheng wrote:.objs. > Similarly, > > foo.o-cflags := $(FOO_CFLAGS) > > is also supported. I noticed that we already support per-object cflags via: $(obj)/adlib.o $(obj)/fmopl.o: QEMU_CFLAGS += -DBUILD_Y8950=0 (this example from audio/Makefile.objs). Is your

Re: [Qemu-devel] [PATCH v16 2/9] rules.mak: allow per object cflags and libs

2014-01-16 Thread Fam Zheng
On Wed, 01/15 19:35, Peter Maydell wrote: > On 15 January 2014 08:48, Fam Zheng wrote: > > Adds extract-libs in LINK to expand any "per object libs", the syntax to > > define > > such a libs options is like: > > > > foo.o-libs := $(CURL_LIBS) > > > > in block/Makefile.objs. > > > > Simila

Re: [Qemu-devel] [PATCH v16 2/9] rules.mak: allow per object cflags and libs

2014-01-15 Thread Peter Maydell
On 15 January 2014 08:48, Fam Zheng wrote: > Adds extract-libs in LINK to expand any "per object libs", the syntax to > define > such a libs options is like: > > foo.o-libs := $(CURL_LIBS) > > in block/Makefile.objs. > > Similarly, > > foo.o-cflags := $(FOO_CFLAGS) > > is also sup

[Qemu-devel] [PATCH v16 2/9] rules.mak: allow per object cflags and libs

2014-01-15 Thread Fam Zheng
Adds extract-libs in LINK to expand any "per object libs", the syntax to define such a libs options is like: foo.o-libs := $(CURL_LIBS) in block/Makefile.objs. Similarly, foo.o-cflags := $(FOO_CFLAGS) is also supported. "foo.o" must be listed in a nested var (e.g. common-obj-y