Re: pkg/50776 (databases/sqlite3 add mk/readline.buildlink3.mk to buildlink3.mk for devel/subversion-base)

[Joerg Sonnenberger <joerg%bec.de@localhost>]

On Mon, Dec 05, 2016 at 11:07:44AM +0100, Joerg Sonnenberger wrote:
> On Mon, Dec 05, 2016 at 02:05:01AM +0000, David Shao wrote:
> >  When I do:
> >  
> >  # ldd libsqlite3.so.0.8.6
> >  libsqlite3.so.0.8.6:
> >      libreadline.so.7 => /usr/pkg/lib/libreadline.so.7 (0x800781000)
> >      libncurses.so.6 => /usr/pkg/lib/libncurses.so.6 (0x8007d0000)
> >      libpthread.so.0 => /usr/lib/libpthread.so.0 (0x80082c000)
> >      libdl.so.0 => /usr/lib/libdl.so.0 (0x8005c9000)
> >      libc.so.8 => /lib/libc.so.8 (0x800458000)
> >  
> >  We see that pkgsrc libreadline is linked in.
> 
> I don't see this and from the Makefile, it should do it either.
> Readline use should be limited to the sqlite3 binary, it should not be
> used by the library.

Ah, right. It was added early this year to the pkgsrc Makefile. That
change is wrong, at least for current sqlite version and should be
reverted. Ryoon, I think that's your responsibility.

Joerg