[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-trivial] [PATCH 0/2 v2] Fix static linking for cURL and SDL
From: |
Yann E. MORIN |
Subject: |
[Qemu-trivial] [PATCH 0/2 v2] Fix static linking for cURL and SDL |
Date: |
Tue, 28 Aug 2012 01:01:32 +0200 |
Hello All!
Currently, configure checks for cURL and SDL with either pkg-config (the
default), or with {curl,sdl}-config (as a fallback).
But pkg-config and {curl,sdl}-config do not have the same set of options:
- to check for shared libs, both use the option: --libs
- to check for static libs:
- pkg-config uses : --static --libs
- {curl,sdl}-config use: --static-libs
To add to the complexity, pkg-config is called through the querry_pkg_config
wrapper, that already passes --static when static linking is required, but
there is no such wrapper for {curl,sdl}-config, so we miss the occasion to
pass --static-libs.
To fix this:
- introduce a new variable QEMU_XXX_CONFIG_LIBS_FLAGS that mirrors the
behavior of QEMU_PKG_CONFIG_FLAGS; this variable can be used by all
xxx-config scripts (eg. curl-config, but later sdl-config too).
Default it to '--libs', which is for shared linking;
- properly use either --libs for pkg-config (--static is already taken
care of in the wrapper), or $QEMU_XXX_CONFIG_LIBS_FLAGS for
{curl,sdl}-config.
Changes since v1:
- drop the spice fix, it is not needed (bad env locally)
- drop the added --static to calls to pkg-config, as it's already in the
wrapper (Stefan Hajnoczi)
Regards,
Yann E. MORIN.
- [Qemu-trivial] [PATCH 0/2 v2] Fix static linking for cURL and SDL,
Yann E. MORIN <=
- [Qemu-trivial] [PATCH 2/2] configure: fix detection for SDL libs when static linking, Yann E. MORIN, 2012/08/27
- [Qemu-trivial] [PATCH 1/2] configure: fix detection for cURL libs when static linking, Yann E. MORIN, 2012/08/27
- Re: [Qemu-trivial] [PATCH 0/2 v2] Fix static linking for cURL and SDL, Stefan Hajnoczi, 2012/08/31
- Re: [Qemu-trivial] [PATCH 0/2 v2] Fix static linking for cURL and SDL, Yann E. MORIN, 2012/08/31
- Re: [Qemu-trivial] [PATCH 0/2 v2] Fix static linking for cURL and SDL, Yann E. MORIN, 2012/08/31