Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

missing check for readline.h in configure.ac

#59

Topic says pretty much all, but dev/lckdclient.c includes headers from readline and configure.ac doesn't check these headers are available. This patch fixes that.

Reported by teuf · July 24th, 2009 @ 07:06 PM

State: resolved
Milestone: none
Assigned to: nobody

Activity

  1. Martin S.
    Martin S.
    • State changed from new to open
    • Tag set to libiphone, lockdown, patch

    lckdclient is only built if --enable-dev-tools is passed.

    July 24th, 2009 @ 10:39 PM

  2. teuf
    teuf

    Ah ok, the attached patch is better then. autogen.sh passes --enable-dev-tools to the configure it generates, that's why I was hit by that.

    July 25th, 2009 @ 07:43 AM

  3. Matt Colyer
    Matt Colyer
    • State changed from open to resolved

    (from [8ebfd7d8eea89bb27e4e6dbb1f37fd90d98b439c]) check for readline availability

    dev/lckdclient.c includes headers from readline but configure.ac doesn't
    make sure these headers are available. This commit makes sure
    readline/readline.h is present and assumes readline is properly installed
    if this header is present

    [#59 missing check for readline.h in configure.ac state:resolved]

    Signed-off-by: Matt Colyer matt@colyer.name
    http://github.com/MattColyer/libiphone/commit/8ebfd7d8eea89bb27e4e6...

    July 27th, 2009 @ 02:28 AM

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.