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

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

The following reply was made to PR pkg/50776; it has been noted by GNATS.

From: Joerg Sonnenberger <joerg%bec.de@localhost>
To: Joerg Sonnenberger <joerg%bec.de@localhost>
Cc: gnats-bugs%NetBSD.org@localhost, ryoon%NetBSD.org@localhost, gnats-admin%netbsd.org@localhost,
	pkgsrc-bugs%netbsd.org@localhost, davshao%gmail.com@localhost
Subject: Re: pkg/50776 (databases/sqlite3 add mk/readline.buildlink3.mk to
 buildlink3.mk for devel/subversion-base)
Date: Mon, 5 Dec 2016 11:10:42 +0100

 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