qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Fam Zheng <famz@redhat.com>
To: Peter Maydell <peter.maydell@linaro.org>
Cc: "Michael Tokarev" <mjt@tls.msk.ru>,
	"QEMU Developers" <qemu-devel@nongnu.org>,
	"Stefan Hajnoczi" <stefanha@redhat.com>,
	"Paolo Bonzini" <pbonzini@redhat.com>,
	"Lluís Vilanova" <vilanova@ac.upc.edu>,
	"Richard Henderson" <rth@twiddle.net>
Subject: Re: [Qemu-devel] [PATCH v6 0/8] Shared Library Module Support
Date: Thu, 12 Sep 2013 11:08:52 +0800	[thread overview]
Message-ID: <20130912030852.GL9134@T430s.nay.redhat.com> (raw)
In-Reply-To: <CAFEAcA_8DxUoN87C5rPArpcV-buXXCpUh1Zz-P8hzstbjpxD4A@mail.gmail.com>

On Wed, 09/11 17:26, Peter Maydell wrote:
> On 11 September 2013 14:34, Fam Zheng <famz@redhat.com> wrote:
> > This series implements feature of shared object building as described in:
> >
> > http://wiki.qemu.org/Features/Modules
> 
> > v6: Dropping RFC.
> 
> ...which I guess makes it a reasonable time to mention that
> this doesn't build on MacOSX/clang:
> 
> cc -m64 -DOS_OBJECT_USE_OBJC=0 -arch x86_64 -D_GNU_SOURCE
> -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -Wstrict-prototypes
> -Wredundant-decls -Wall -Wundef -Wwrite-strings -Wmissing-prototypes
> -fno-strict-aliasing  -Wno-initializer-overrides -Wendif-labels
> -Wmissing-include-dirs -Wempty-body -Wnested-externs -Wformat-security
> -Wformat-y2k -Winit-self -Wignored-qualifiers -Wold-style-definition
> -Wtype-limits -fstack-protector-all -I/sw/include
> -I/sw/include/libpng15   -I/opt/X11/include/pixman-1
> -I/Users/pm215/src/qemu/dtc/libfdt -I/Users/pm215/src/qemu/tests -I
> qga/qapi-generated -I/sw/include/glib-2.0 -I/sw/lib/glib-2.0/include
> -I/sw/include/glib-2.0 -I/sw/lib/glib-2.0/include -g  -m64 -framework
> CoreFoundation -framework IOKit -arch x86_64 -g  -o qemu-ga
> -L/sw/lib -lgmodule-2.0 -lglib-2.0 -lintl -L/sw/lib -lgthread-2.0
> -lglib-2.0 -lintl
> Undefined symbols for architecture x86_64:
>   "_main", referenced from:
>      -u command line option
> ld: symbol(s) not found for architecture x86_64
> clang: error: linker command failed with exit code 1 (use -v to see invocation)
> 
> ...which looks like we're trying to link qemu-ga but not specifying
> any object files at all, with the result that the linker complains that
> we haven't provided a main() function.
> 
Yes, the same bug with patch 03 without libtool, fixed here. Thanks.

Fam

      reply	other threads:[~2013-09-12  3:09 UTC|newest]

Thread overview: 40+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-09-11 13:34 [Qemu-devel] [PATCH v6 0/8] Shared Library Module Support Fam Zheng
2013-09-11 13:34 ` [Qemu-devel] [PATCH v6 1/8] make.rule: fix $(obj) to a real relative path Fam Zheng
2013-09-11 13:41   ` Paolo Bonzini
2013-09-12  2:32     ` Fam Zheng
2013-09-11 14:59   ` Daniel P. Berrange
2013-09-12  1:57     ` Fam Zheng
2013-09-12  2:22     ` Fam Zheng
2013-09-11 16:38   ` Peter Maydell
2013-09-11 16:40     ` Paolo Bonzini
2013-09-11 16:43       ` Peter Maydell
2013-09-11 13:34 ` [Qemu-devel] [PATCH v6 2/8] rule.mak: allow per object cflags and libs Fam Zheng
2013-09-11 13:43   ` Paolo Bonzini
2013-09-12  2:52     ` Fam Zheng
2013-09-12  6:34       ` Paolo Bonzini
2013-09-12  7:12         ` Fam Zheng
2013-09-11 13:34 ` [Qemu-devel] [PATCH v6 3/8] Makefile: introduce common-obj-m and block-obj-m for DSO Fam Zheng
2013-09-11 20:01   ` Peter Maydell
2013-09-12  2:48     ` Fam Zheng
2013-09-12  2:50   ` Fam Zheng
2013-09-11 13:34 ` [Qemu-devel] [PATCH v6 4/8] module: implement module loading function Fam Zheng
2013-09-11 15:48   ` Daniel P. Berrange
2013-09-11 18:33     ` Alex Bligh
2013-09-11 18:46     ` Richard Henderson
2013-09-12  3:02       ` Fam Zheng
2013-09-12  5:36         ` Michael Tokarev
2013-09-12  9:13           ` Daniel P. Berrange
2013-09-12 11:59           ` Eric Blake
2013-09-12 12:44             ` Daniel P. Berrange
2013-09-11 13:34 ` [Qemu-devel] [PATCH v6 5/8] configure: introduce --enable-modules Fam Zheng
2013-09-11 13:46   ` Paolo Bonzini
2013-09-12  2:06     ` Fam Zheng
2013-09-11 13:34 ` [Qemu-devel] [PATCH v6 6/8] Makefile: install modules with "make install" Fam Zheng
2013-09-11 13:50   ` Paolo Bonzini
2013-09-12  2:00     ` Fam Zheng
2013-09-11 13:34 ` [Qemu-devel] [PATCH v6 7/8] .gitignore: ignore module related files (dll, so, mo) Fam Zheng
2013-09-11 13:34 ` [Qemu-devel] [PATCH v6 8/8] block: convert block drivers linked with libs to modules Fam Zheng
2013-09-11 15:41   ` Daniel P. Berrange
2013-09-12  2:07     ` Fam Zheng
2013-09-11 16:26 ` [Qemu-devel] [PATCH v6 0/8] Shared Library Module Support Peter Maydell
2013-09-12  3:08   ` Fam Zheng [this message]

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=20130912030852.GL9134@T430s.nay.redhat.com \
    --to=famz@redhat.com \
    --cc=mjt@tls.msk.ru \
    --cc=pbonzini@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    --cc=rth@twiddle.net \
    --cc=stefanha@redhat.com \
    --cc=vilanova@ac.upc.edu \
    /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).