Vladislav Sekulic wrote:
> On 26 Jan 2005 09:11:47 -0500 Greg Troxel <gdt@ir.bbn.com> wrote:
>>You may want to set
>>
>>USE_GNU_READLINE=yes
>>
>>and include readline/builtin.mk instead
>>
> 
> Ah, including builtin.mk fixes the problem:
> 
> ---
> ===> Checking for vulnerabilities in ratpoison-1.3.0
> => Checksum OK for ratpoison-1.3.0.tar.gz.
> ===> Extracting for ratpoison-1.3.0
> ===> Required installed package bash-[0-9]*: bash-3.0nb1 found
> ===> Required installed package readline>=5.0: readline-5.0 found
> ===> Required installed package x11-links>=0.24: x11-links-0.24 found
> ===> Patching for ratpoison-1.3.0
> <snip>
> ---
> 
> It compiles and installs normally now.  Looking at the pkgsrc 
> guide section 9.3, it states that in this kind of a situation,
> builtin.mk should be included in addition to buildlink3.mk, which
> is a point that I had previously missed.

No, the builtin.mk files should never need to be included by a package
Makefile, and I consider it a bug if it's required.  It looks like the
problem is that "-lhistory" is being converted into "-ledit", but
libedit requires "-ltermcap" to also be on the command line.  I will fix
this in readline/builtin.mk.  Your package Makefile should only need to
include readline/buildlink3.mk to note that it wants readline functionality.

	Cheers,

	-- Johnny Lam <jlam@NetBSD.org>