Re: [Powertop] [PATCH] Optionally check for ncurses pkgconfig file.

[Alexandra Yates <alexandra.yates at linux.intel.com>]

[-- Attachment #1: Type: text/plain, Size: 1399 bytes --]

> On Tue, May 6, 2014 at 12:53 AM, Sergey Senozhatsky
> <sergey.senozhatsky(a)gmail.com> wrote:
>> On (05/05/14 23:42), Auke Kok wrote:
>>> ---
>>>  configure.ac | 3 ++-
>>>  1 file changed, 2 insertions(+), 1 deletion(-)
>>>
>>> diff --git a/configure.ac b/configure.ac
>>> index baa5faa..5a90fe3 100644
>>> --- a/configure.ac
>>> +++ b/configure.ac
>>> @@ -46,7 +46,8 @@ AC_FUNC_REALLOC
>>>  AC_FUNC_STRTOD
>>>  AC_CHECK_FUNCS([fdatasync getpagesize gettimeofday memmove memset
>>> mkdir munmap pow realpath regcomp select setlocale socket sqrt
>>> strcasecmp strchr strdup strerror strncasecmp strstr strtoul
>>> strtoull])
>>>
>>> -AC_SEARCH_LIBS([delwin], [ncursesw ncurses], [], AC_MSG_ERROR([ncurses
>>> is required but was not found]), [])
>>> +PKG_CHECK_MODULES([NCURSES], [ncurses], [LIBS="$LIBS $ncurses_LIBS"],
>>                                 ^^^^^^^^^
>>                                 [ncursesw ncurses]?
>
> Good question, I actually do not know, and I don't have an ncurses
> pkgconfig around to see if that works.
>
> Alexandra, can you see if this works with:
>
> +PKG_CHECK_MODULES([NCURSES], [ncursesw ncurses], [LIBS="$LIBS
> $ncurses_LIBS"],
>
> That needs to be tested by someone with a new ncurses.
>
> Auke
>

I'm on training this week.  I will be able to test at the end of the day
after my class today.

Thank you,
Alexandra.