* [Qemu-devel] [Qemu-trivial] [PATCH] configure: try pkg-config for curses
@ 2012-12-31 16:30 Vadim Evard
2013-01-11 12:51 ` Vadim Evard
0 siblings, 1 reply; 4+ messages in thread
From: Vadim Evard @ 2012-12-31 16:30 UTC (permalink / raw)
To: qemu-devel; +Cc: qemu-trivial
configure: try pkg-config for curses
Static linkikng against ncurses may require explicit -ltinfo.
In case -lcurses and -lncurses both didn't work give pkg-config a
chance.
Signed-off-by: Vadim Evard <v.e.evard@gmail.com>
---
configure | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/configure b/configure
index b0c7e54..16280e2 100755
--- a/configure
+++ b/configure
@@ -2030,7 +2030,7 @@ fi
if test "$mingw32" = "yes" ; then
curses_list="-lpdcurses"
else
- curses_list="-lncurses -lcurses"
+ curses_list="-lncurses:-lcurses:$($pkg_config --libs ncurses)"
fi
if test "$curses" != "no" ; then
@@ -2043,7 +2043,9 @@ 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"
--
1.7.10.4
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [Qemu-devel] [Qemu-trivial] [PATCH] configure: try pkg-config for curses
2012-12-31 16:30 [Qemu-devel] [Qemu-trivial] [PATCH] configure: try pkg-config for curses Vadim Evard
@ 2013-01-11 12:51 ` Vadim Evard
2013-01-15 9:49 ` Stefan Hajnoczi
0 siblings, 1 reply; 4+ messages in thread
From: Vadim Evard @ 2013-01-11 12:51 UTC (permalink / raw)
To: qemu-devel; +Cc: qemu-trivial
ping
Пнд 31 Дек 2012 20:30:59, Vadim Evard писал:
> configure: try pkg-config for curses
>
> Static linkikng against ncurses may require explicit -ltinfo.
> In case -lcurses and -lncurses both didn't work give pkg-config a
> chance.
>
> Signed-off-by: Vadim Evard <v.e.evard@gmail.com>
> ---
> configure | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/configure b/configure
> index b0c7e54..16280e2 100755
> --- a/configure
> +++ b/configure
> @@ -2030,7 +2030,7 @@ fi
> if test "$mingw32" = "yes" ; then
> curses_list="-lpdcurses"
> else
> - curses_list="-lncurses -lcurses"
> + curses_list="-lncurses:-lcurses:$($pkg_config --libs ncurses)"
> fi
>
> if test "$curses" != "no" ; then
> @@ -2043,7 +2043,9 @@ 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"
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [Qemu-devel] [Qemu-trivial] [PATCH] configure: try pkg-config for curses
2013-01-11 12:51 ` Vadim Evard
@ 2013-01-15 9:49 ` Stefan Hajnoczi
2013-01-15 12:17 ` [Qemu-devel] [Qemu-trivial] [PATCH v2] " Vadim Evard
0 siblings, 1 reply; 4+ messages in thread
From: Stefan Hajnoczi @ 2013-01-15 9:49 UTC (permalink / raw)
To: Vadim Evard; +Cc: qemu-trivial, qemu-devel
On Fri, Jan 11, 2013 at 04:51:24PM +0400, Vadim Evard wrote:
> ping
>
> Пнд 31 Дек 2012 20:30:59, Vadim Evard писал:
> >configure: try pkg-config for curses
> >
> >Static linkikng against ncurses may require explicit -ltinfo.
> >In case -lcurses and -lncurses both didn't work give pkg-config a
> >chance.
> >
> >Signed-off-by: Vadim Evard <v.e.evard@gmail.com>
> >---
> >configure | 4 +++-
> >1 file changed, 3 insertions(+), 1 deletion(-)
> >
> >diff --git a/configure b/configure
> >index b0c7e54..16280e2 100755
> >--- a/configure
> >+++ b/configure
> >@@ -2030,7 +2030,7 @@ fi
> >if test "$mingw32" = "yes" ; then
> >curses_list="-lpdcurses"
> >else
> >- curses_list="-lncurses -lcurses"
> >+ curses_list="-lncurses:-lcurses:$($pkg_config --libs ncurses)"
> >fi
> >
> >if test "$curses" != "no" ; then
> >@@ -2043,7 +2043,9 @@ 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"
Please also unset IFS after the loop so there will never be a problem if
curses_list="".
Stefan
^ permalink raw reply [flat|nested] 4+ messages in thread
* [Qemu-devel] [Qemu-trivial] [PATCH v2] configure: try pkg-config for curses
2013-01-15 9:49 ` Stefan Hajnoczi
@ 2013-01-15 12:17 ` Vadim Evard
0 siblings, 0 replies; 4+ messages in thread
From: Vadim Evard @ 2013-01-15 12:17 UTC (permalink / raw)
To: Stefan Hajnoczi; +Cc: qemu-trivial, qemu-devel
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 <v.e.evard@gmail.com>
---
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)"
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.7.10.4
On 15.01.2013 13:49, Stefan Hajnoczi wrote:
> On Fri, Jan 11, 2013 at 04:51:24PM +0400, Vadim Evard wrote:
>> ping
>>
>> Пнд 31 Дек 2012 20:30:59, Vadim Evard писал:
>>> configure: try pkg-config for curses
>>>
>>> Static linkikng against ncurses may require explicit -ltinfo.
>>> In case -lcurses and -lncurses both didn't work give pkg-config a
>>> chance.
>>>
>>> Signed-off-by: Vadim Evard<v.e.evard@gmail.com>
>>> ---
>>> configure | 4 +++-
>>> 1 file changed, 3 insertions(+), 1 deletion(-)
>>>
>>> diff --git a/configure b/configure
>>> index b0c7e54..16280e2 100755
>>> --- a/configure
>>> +++ b/configure
>>> @@ -2030,7 +2030,7 @@ fi
>>> if test "$mingw32" = "yes" ; then
>>> curses_list="-lpdcurses"
>>> else
>>> - curses_list="-lncurses -lcurses"
>>> + curses_list="-lncurses:-lcurses:$($pkg_config --libs ncurses)"
>>> fi
>>>
>>> if test "$curses" != "no" ; then
>>> @@ -2043,7 +2043,9 @@ 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"
>
> Please also unset IFS after the loop so there will never be a problem if
> curses_list="".
>
> Stefan
^ permalink raw reply related [flat|nested] 4+ messages in thread
end of thread, other threads:[~2013-01-15 12:17 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-12-31 16:30 [Qemu-devel] [Qemu-trivial] [PATCH] configure: try pkg-config for curses Vadim Evard
2013-01-11 12:51 ` Vadim Evard
2013-01-15 9:49 ` Stefan Hajnoczi
2013-01-15 12:17 ` [Qemu-devel] [Qemu-trivial] [PATCH v2] " Vadim Evard
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).