[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 3/3] configure: try pkg-config for curses
From: |
Anthony Liguori |
Subject: |
Re: [Qemu-devel] [PATCH 3/3] configure: try pkg-config for curses |
Date: |
Tue, 15 Jan 2013 16:57:18 -0600 |
User-agent: |
Notmuch/0.13.2+93~ged93d79 (http://notmuchmail.org) Emacs/23.3.1 (x86_64-pc-linux-gnu) |
Stefan Hajnoczi <address@hidden> writes:
> From: Vadim Evard <address@hidden>
>
> Static linkikng against ncurses may require explicit -ltinfo.
> In case -lcurses and -lncurses both didn't work give pkg-config a
> chance.
>
> Fixes #1094786 for me.
>
> Signed-off-by: Vadim Evard <address@hidden>
> Signed-off-by: Stefan Hajnoczi <address@hidden>
> ---
> configure | 5 ++++-
> 1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/configure b/configure
> index c908f66..40473a9 100755
> --- a/configure
> +++ b/configure
> @@ -2039,7 +2039,7 @@ fi
> if test "$mingw32" = "yes" ; then
> curses_list="-lpdcurses"
> else
> - curses_list="-lncurses -lcurses"
> + curses_list="-lncurses:-lcurses:$($pkg_config --libs ncurses)"
Please fix this as a follow up but this is sloppy.
pkg-config spews stuff to stderr when a package isn't found so this ends
up outputting junk in the configure if ncurses isn't found.
Regards,
Anthony Liguori
> fi
>
> if test "$curses" != "no" ; then
> @@ -2052,13 +2052,16 @@ int main(void) {
> return s != 0;
> }
> EOF
> + IFS=:
> for curses_lib in $curses_list; do
> + unset IFS
> if compile_prog "" "$curses_lib" ; then
> curses_found=yes
> libs_softmmu="$curses_lib $libs_softmmu"
> break
> fi
> done
> + unset IFS
> if test "$curses_found" = "yes" ; then
> curses=yes
> else
> --
> 1.8.0.2