cvs commit: src/usr.bin/make cond.c cond.h for.c for.h parse.c parse.h

Greg 'groggy' Lehey grog at FreeBSD.org
Mon Apr 11 16:06:17 PDT 2005


On Monday, 11 April 2005 at  2:11:26 -0700, Kris Kennaway wrote:
> On Mon, Apr 11, 2005 at 11:07:28AM +0200, Harti Brandt wrote:
>
>> KK>I assume you've tested that this doesn't cause problems for anything
>> KK>in the ports collection?
>>
>> I didn't test all of the ports collection - just a number of ports. But I
>> grepped the ports infrastructure and the ports Makefile* for problematic
>> constructs and found one problem that now shows up (under certain
>> conditions) instead of beeing hidden:
>>
>> mail/dspam
>> mail/dspam-devel
>>
>> both of them use
>>
>> . elseif ...
>>
>> This has been intepreted by make as a plain .else without any warning up
>> to now. Now it is just ignored when the .if defined (WITH_MYSQL40) ... is
>> false and give an error if it is true. I'll inform the maintainer about
>> this.
>>
>> There may be of course ports that use our make to build. In any case the
>> change to the .else and .endif clauses just give a warning so the ports
>> maintainer have time to fix this. What could break is the use of
>> .undefFOO but I don't expect many of them.
>
> I really hope this doesn't cause problems, but based on historical
> precedent I expect that it will.  In future, please coordinate
> troublesome make changes with us (portmgr) so we can test them first
> and avoid destabilizing the ports collection for the users.

It would be interesting to understand the necessity of this change.
We have so many different flavours of make already.

Greg
--
See complete headers for address and phone numbers.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 187 bytes
Desc: not available
Url : http://lists.freebsd.org/pipermail/cvs-all/attachments/20050412/d137eada/attachment.bin


More information about the cvs-all mailing list