[libvirt] [PATCH] Generate libvirt_qemu.def from libvirt_qemu.syms for MinGW builds

Eric Blake eblake at redhat.com
Fri Aug 13 19:19:28 UTC 2010


On 08/12/2010 05:23 PM, Matthias Bolte wrote:
> @@ -1866,6 +1866,7 @@ case "$host" in
>      # Also set the symbol file to .def, so src/Makefile generates libvirt.def
>      # from libvirt.syms and passes libvirt.def instead of libvirt.syms to the linker
>      LIBVIRT_SYMBOL_FILE=libvirt.def
> +    LIBVIRT_QEMU_SYMBOL_FILE='$(srcdir)/libvirt_qemu.def'

The libvirt_qemu.syms file lives in $(srcdir) because it is
version-controlled, but this line would make the .def file live there
too even though it is generated.  Any reason you can't use just
LIBVIRT_QEMU_SYMBOL_FILE=libvirt_qemu.def, and still have things work in
a VPATH build with it living in $(builddir)?

> +++ b/src/Makefile.am
> @@ -1010,7 +1010,7 @@ EXTRA_DIST += \
>    libvirt_daemon.syms		\
>    libvirt_nwfilter.syms
>  
> -BUILT_SOURCES += libvirt.syms libvirt.def
> +BUILT_SOURCES += libvirt.syms libvirt.def libvirt_qemu.def

Particularly here, since you are treating it as a built source - in
fact, I'm guessing that this won't work in a VPATH build without
tweaking the configure.ac change to drop $(srcdir).

>  
> +libvirt_qemu.def: libvirt_qemu.syms

Which also means that for this line, you may have to make the dependency
be on $(srcdir)/libvirt_qemu.syms, since the whole point of my question
is reading input from $(srcdir) but generating output in $(builddir).

> +	$(AM_V_GEN)rm -f -- $@-tmp $@ ; \
> +	printf 'EXPORTS\n' > $@-tmp && \
> +	sed -e '/^$$/d; /#/d; /:/d; /\}/d; /\*/d; /LIBVIRT_/d; s/[ \t]*\(.*\)\;/    \1/g' $^ >> $@-tmp && \
> +	chmod a-w $@-tmp && \
> +	mv $@-tmp libvirt_qemu.def
> +

-- 
Eric Blake   eblake at redhat.com    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 619 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20100813/6683b84b/attachment-0001.sig>


More information about the libvir-list mailing list