[libvirt] [PATCH] Avoid searching for windres when not building for cygwin or mingw.
Daniel Veillard
veillard at redhat.com
Thu Apr 8 09:54:55 UTC 2010
On Fri, Feb 12, 2010 at 01:29:22PM +0100, Diego Elio Pettenò wrote:
> Just checking for a windres tool might hit even on Linux systems when
> building for Linux (e.g.: when using Gentoo and having built binutils with
> multitarget support), and will then fail to link properly at the end of the
> build.
>
> Check the host string before deciding whether to look for windres or not.
> ---
> configure.ac | 9 +++++++--
> 1 files changed, 7 insertions(+), 2 deletions(-)
>
> diff --git a/configure.ac b/configure.ac
> index 29c6396..e6122b9 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -1676,8 +1676,13 @@ AC_SUBST([CYGWIN_EXTRA_PYTHON_LIBADD])
> AC_SUBST([MINGW_EXTRA_LDFLAGS])
>
> dnl Look for windres to build a Windows icon resource.
> -AC_CHECK_TOOL([WINDRES], [windres], [no])
> -AM_CONDITIONAL([WITH_WIN_ICON], [test "$WINDRES" != "no"])
> +case "$host" in
> + *cygwin* | *mingw* )
> + AC_CHECK_TOOL([WINDRES], [windres], [])
> + ;;
> +esac
> +
> +AM_CONDITIONAL([WITH_WIN_ICON], [test "$WINDRES" != ""])
An old patch that we forgot, it's a cleanup, I just updated
the Windows test to match teh one a few lines before, then applied,
pushed, thanks !
Daniel
--
Daniel Veillard | libxml Gnome XML XSLT toolkit http://xmlsoft.org/
daniel at veillard.com | Rpmfind RPM search engine http://rpmfind.net/
http://veillard.com/ | virtualization library http://libvirt.org/
More information about the libvir-list
mailing list