Remove unnecessary change to Mk/bsd.port.mk (: svn commit: r303619 - head/Mk)

[Gerald Pfeifer]

On Sun, 4 Nov 2012, Eitan Adler wrote:
> On 4 November 2012 09:09, Gerald Pfeifer <gerald at pfeifer.com> wrote:
>> I understand Mk/bsd.port.mk is not applicable for maintainer
>> timeouts or I would have gone ahead and committed the patch
>> below.
> to change b.p.m please file a PR and request an exp-run

Well, my point was more that the original commit was problematic
and should be addressed, rather than me suggesting a change.

But, since the bogus change still is in Mk/bsd.port.mk, I now filed
  ports/174790
  Mk/bsd.port.mk: revert part of r303619, CPPFLAGS and LDFLAGS already are part of CONFIGURE_ENV
and hope portmgr@ is going to pick this up when they have a minute.

On Mon, 5 Nov 2012, Baptiste Daroussin wrote:
>> Maybe fjoe@ can explain why CONFIGURE_ENV was placed here.
>> The line was part of PR 162995.

I didn't see any explanation, but may have missed it?

> Speaking of this, USE_READLINE is anyway buggy, because it forces 
> linking over libreadline at first but to not take care of the related 
> ncurses, which could be the one from the ports or the one from base.

I also find the following interesting for such a general flag:

  CPPFLAGS+=		-I${LOCALBASE}/include
  LDFLAGS+=		-L${LOCALBASE}/lib -lreadline

Now, as some of you know, I have made the point that CPPFLAGS and
LDFLAGS should be set like this in general (and those hundreds, if
not thousands of ports doing so each on its own simplified accordingly).

To now see this as part of standard flag in bsd.mk.port, is interesting.

Gerald