Subject: Re: pkg/33428: broken package wm/icecc fix
To: None <gnats-bugs@netbsd.org>
From: Iain Hibbert <plunky@rya-online.net>
List: pkgsrc-bugs
Date: 05/08/2006 21:23:21
On Mon, 8 May 2006, joerg@britannica.bec.de wrote:

>  OK, so we talk about the same problem. While I don't think that the
>  patch is wrong, it should not be needed since PTHREAD_LIBS +
>  PTHREAD_LDFLAGS should be self-contained in my understanding. If it is
>  missing the rpath entry e.g. in case of pth, this looks wrong.

My thought was that it only happens because the gmakefile is hardwired,
and the pthread.buildlink3.mk/builtin.mk would just assume that if
pthreads are built in then nothing needs to be added because LDFLAGS
should be enough.

In fact, the patch would technically be slightly wrong I just realised, it
should be using
	$(CXX) $(CXXFLAGS) ..

instead of
	$(CC) $(CFLAGS) ..

?

iain