On Mon, Jun 14, 2010 at 07:44:30PM +0200, Anton Berezin wrote:
> On Mon, Jun 14, 2010 at 05:22:29PM +0000, Philip M. Gollucci wrote:
> > On 06/14/10 12:04, Anton Berezin wrote:
> > > | +RUN_DEPENDS= ${BUILD_DEPENDS}
> > I really thought we were going in the other direction of not setting a
> > depends = to another depends?
>
> This particular commit did not change the status quo. The change was done
> to silence portlint.
>
> As for the substance of setting the two to be the same, I am afraid I missed
> the relevant discussion - the one which reached the consensus that it is not
> a good idea. Could you point me towards it?
It is not a good idea because of make(1) being lazy when evaluating
assignments, which in turn can result in rdeps being polluted by bdeps
when the latter is being altered somewhere deep in bpm. The right thing
to do here is probably not to avoid assigning rdeps to bdeps completely,
but to do it correctly with immediate expansion thereof:
RUN_DEPENDS:= ${BUILD_DEPENDS}
This is being very common lack of knowledge for lots of our fellow
committers, to my regret. :-(
Didn't we have portlint(1) check or PHB paragraph about it?
./danfe
_______________________________________________
cvs-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/cvs-all
To unsubscribe, send any mail to "cvs-all-unsubscr...@freebsd.org"
