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.

There was a problem

You must be a member of this account.

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.