xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
From: Fantu <fantonifabio@tiscali.it>
To: xen-devel@lists.xensource.com
Subject: Re: xl_cmdimpl.c errors
Date: Fri, 17 Feb 2012 05:14:22 -0800 (PST)	[thread overview]
Message-ID: <1329484462005-5492438.post@n5.nabble.com> (raw)
In-Reply-To: <1329481575617-5492321.post@n5.nabble.com>

With only second patch fail:

gcc  -O1 -fno-omit-frame-pointer -m64 -g -fno-strict-aliasing -std=gnu99
-Wall -Wstrict-prototypes -Wdeclaration-after-statement
-Wno-unused-but-set-variable   -D__XEN_TOOLS__ -MMD -MF .xl_cmdimpl.o.d 
-D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -fno-optimize-sibling-calls
-Werror -Wno-format-zero-length -Wmissing-declarations
-Wno-declaration-after-statement -Wformat-nonliteral -I. -fPIC
-DHAVE_YAJL_VERSION
-I/mnt/vm/xen/xen-unstable.hg/tools/libxl/../../tools/libxc
-I/mnt/vm/xen/xen-unstable.hg/tools/libxl/../../tools/include 
-I/mnt/vm/xen/xen-unstable.hg/tools/libxl/../../tools/libxl
-I/mnt/vm/xen/xen-unstable.hg/tools/libxl/../../tools/libxc
-I/mnt/vm/xen/xen-unstable.hg/tools/libxl/../../tools/include
-I/mnt/vm/xen/xen-unstable.hg/tools/libxl/../../tools/include  -c -o
xl_cmdimpl.o xl_cmdimpl.c 
xl_cmdimpl.c: In function âprintf_infoâ:
xl_cmdimpl.c:305:5: error: implicit declaration of function
âlibxl_yajl_gen_allocâ [-Werror=implicit-function-declaration]
xl_cmdimpl.c:305:10: error: assignment makes pointer from integer without a
cast [-Werror]
cc1: all warnings being treated as errors

make[3]: *** [xl_cmdimpl.o] Error 1
make[3]: Leaving directory `/mnt/vm/xen/xen-unstable.hg/tools/libxl'
make[2]: *** [subdir-install-libxl] Error 2
make[2]: Leaving directory `/mnt/vm/xen/xen-unstable.hg/tools'
make[1]: *** [subdirs-install] Error 2
make[1]: Leaving directory `/mnt/vm/xen/xen-unstable.hg/tools'
make: *** [install-tools] Error 2

Probably need also the first, not add after read this:
> tools/libxl: rename libxl__yajl_gen_alloc
>
> libxl__yajl_gen_alloc() is called by generic code,
> rename it to libx__yajl_gen_alloc().

Other than this typo both patches in this series look good to me,
thanks.

Acked-by: Ian Campbell <[hidden email]> 

Now I try also first.

--
View this message in context: http://xen.1045712.n5.nabble.com/xl-cmdimpl-c-errors-tp5491537p5492438.html
Sent from the Xen - Dev mailing list archive at Nabble.com.

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

  reply	other threads:[~2012-02-17 13:14 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-02-17  5:36 xl_cmdimpl.c errors jacek burghardt
2012-02-17  8:22 ` Ian Campbell
2012-02-17  8:37   ` Fantu
2012-02-17 11:06     ` Fantu
2012-02-17 12:01       ` Ian Campbell
2012-02-17 12:26         ` Fantu
2012-02-17 13:14           ` Fantu [this message]
2012-02-17 13:34             ` Ian Campbell
2012-02-17 13:36               ` Fantu
2012-02-17 17:39                 ` jacek burghardt
2012-02-21 13:56                   ` Fantu

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1329484462005-5492438.post@n5.nabble.com \
    --to=fantonifabio@tiscali.it \
    --cc=xen-devel@lists.xensource.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).