[PATCH] curses: remove hard-wired ncursesw include path

Daniel Kahn Gillmor dkg at fifthhorseman.net
Mon Jun 29 05:27:24 CEST 2015


On Wed 2015-06-24 08:31:19 -0400, Neal H. Walfield wrote:
> Hi,
>
> At Wed, 24 Jun 2015 09:52:56 +0000,
> Gergely Imreh wrote:
>> 
>> * pinentry/pinentry-curses.c: pkg-config finds the correct include
>> paths so should not hard-wire the path for the ncursesw header file.
>
> Thanks!  I've applied and pushed this.

hm, this is a reverse of 21e83f422667e431c1283b9ae3356fded3523e50, which
is a changeset i'm the author on.  the commit log for 21e83f states:

   Avoid implicit declaration of function addnwstr.
    
    * pinentry/pinentry-curses.c [HAVE_NCURSESW]: Include <ncursesw/curses.h>.
    [! HAVE_NCURSESW]: Only include <curses.h> in this case.
    
    --
    
    When built with libncursesw, we see this problem:
    
    pinentry-curses.c:440:8: warning: implicit declaration of function ‘addnwstr’ [-Wimplicit-function-declaration]
            ADDCH (start[i]);
            ^

I ran into this on debian unstable systems.  If we revert it, i'm
concerned that this will mean restoring the warning again; i agree with
Gergely that pkg-config *should* do things properly and the additional
#ifdef shouldn't be needed, but that's not what i saw happening.

Maybe there's some other way to fix this cleanly?

          --dkg



More information about the Gnupg-devel mailing list