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

Re: [Xen-devel] Re: [PATCH V5 2/3] libxl: Introduce libxl_internal_types.idl.


  • To: Ian Campbell <Ian.Campbell@xxxxxxxxxxxxx>
  • From: Anthony PERARD <anthony.perard@xxxxxxxxxx>
  • Date: Mon, 27 Jun 2011 16:40:16 +0100
  • Cc: Anthony PERARD <anthony.perard@xxxxxxxxxx>, Xen Devel <xen-devel@xxxxxxxxxxxxxxxxxxx>, Stefano Stabellini <Stefano.Stabellini@xxxxxxxxxxxxx>
  • Delivery-date: Mon, 27 Jun 2011 08:49:03 -0700
  • Domainkey-signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:sender:in-reply-to:references:from:date :x-google-sender-auth:message-id:subject:to:cc:content-type :content-transfer-encoding; b=Uf1Eg2MjU0ZqwPTYhg0jcoQ1NqMxL1gGQhu454mZjlhmz1IHStuiKfXH3iCHt9nBKW Qb5RI9rww+cKO1VpqT5xqDdGaeNqnvnhhr73yup8Soe5gVQuniJt7wIXH5OclOUKqWD5 FMOqKGFFsDGLk5YwK6+f0I2QXoDGrRkRdQJTQ=
  • List-id: Xen developer discussion <xen-devel.lists.xensource.com>

On Mon, Jun 27, 2011 at 16:20, Ian Campbell <Ian.Campbell@xxxxxxxxxxxxx> wrote:
> On Mon, 2011-06-27 at 16:12 +0100, Anthony PERARD wrote:
>> Signed-off-by: Anthony PERARD <anthony.perard@xxxxxxxxxx>
>> ---
>> Âtools/libxl/Makefile         |  10 +++++++++-
>> Âtools/libxl/gentypes.py       Â|  15 ++++++++-------
>> Âtools/libxl/libxl_internal.h     |  Â1 +
>> Âtools/libxl/libxl_internal_types.idl | Â 10 ++++++++++
>
> Do you not need to update .hgignore?

I don't use mercurial! :)
But no, the file generated will be catch by:
^tools/libxl/_.*\.h$

>> Â4 files changed, 28 insertions(+), 8 deletions(-)
>> Âcreate mode 100644 tools/libxl/libxl_internal_types.idl
>>
>> diff --git a/tools/libxl/Makefile b/tools/libxl/Makefile
>> index bfe9c58..f148ad3 100644
>> --- a/tools/libxl/Makefile
>> +++ b/tools/libxl/Makefile
>> @@ -35,7 +35,7 @@ LIBXL_OBJS-$(CONFIG_IA64) += libxl_nocpuid.o
>> ÂLIBXL_OBJS = flexarray.o libxl.o libxl_create.o libxl_dm.o libxl_pci.o \
>> Â Â Â Â Â Â Â Â Â Â Â libxl_dom.o libxl_exec.o libxl_xshelp.o libxl_device.o 
>> \
>> Â Â Â Â Â Â Â Â Â Â Â libxl_internal.o libxl_utils.o libxl_uuid.o 
>> $(LIBXL_OBJS-y)
>> -LIBXL_OBJS += _libxl_types.o libxl_flask.o
>> +LIBXL_OBJS += _libxl_types.o libxl_flask.o _libxl_internal_types.o
>>
>> Â$(LIBXL_OBJS): CFLAGS += $(CFLAGS_libxenctrl) $(CFLAGS_libxenguest) 
>> $(CFLAGS_libxenstore) $(CFLAGS_libblktapctl)
>>
>> @@ -79,14 +79,22 @@ _libxl_paths.h: genpath
>> Âlibxl_paths.c: _libxl_paths.h
>>
>> Âlibxl.h: _libxl_types.h
>> +libxl_internal.h: _libxl_internal_types.h
>>
>> Â$(LIBXL_OBJS) $(LIBXLU_OBJS) $(XL_OBJS): libxl.h
>> +$(LIBXL_OBJS): libxl_internal.h
>>
>> Â_libxl_%.h _libxl_%.c: libxl.idl gen%.py libxl%.py
>> Â Â Â $(PYTHON) gen$*.py libxl.idl __libxl_$*.h __libxl_$*.c
>
> If you simply s/gen$*.py/gentypes.py/ and s/libxl.idl/libxl$*.idl/g and
> leave the other patterns does this rule work for both IDL files?

Yes, it's works, with a `mv libxl.idl libxl_types.idl`. I thought this
rules was used by something else.

>> Â Â Â mv __libxl_$*.h _libxl_$*.h
>> Â Â Â mv __libxl_$*.c _libxl_$*.c
>>
>> +.NOTPARALLEL: _libxl_internal_types.h _libxl_internal_types.c
>
> I'd been wondering how to do that!

Better than a -j1, first time I use it but seams to work fine.

-- 
Anthony PERARD

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel


 


Rackspace

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