Re: [Powertop] [PATCH] Optionally check for ncurses pkgconfig file.
Alexandra Yates <alexandra.yates at linux.intel.com>
| Newsgroups | dev.linux.lists.powertop |
|---|---|
| Message-ID | <[email protected]> |
>> 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. > I tested in ubuntu I'm running libncurses5 powertop configured and installed with out issues. Looking for someone to test on chrome. Thank you, Alexandra.