Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 23 Jan 2014 13:54:30 -0800
From:      "Simon J. Gerraty" <sjg@juniper.net>
To:        Garrett Cooper <yaneurabeya@gmail.com>
Cc:        "freebsd-testing@freebsd.org" <freebsd-testing@freebsd.org>, brooks@freebsd.org, sjg@juniper.net
Subject:   Re: Makefile.inc1.patch
Message-ID:  <20140123215430.4B7B15807E@chaos.jnpr.net>
In-Reply-To: <EBDCAEEC-9485-4EA5-AA60-943EA70A3171@gmail.com>
References:  <B4D2A908-715F-484F-8028-A1F38884AF3F@gmail.com> <CAOtMX2jQ24JCR2Ct8YKob4MKcHWMhVVv5XG-1usoPWqEOA2OQg@mail.gmail.com> <4A3E3984-73D3-4441-97A7-D58679EFF978@gmail.com> <9775878D-91AB-4BE4-ADFA-32D8DB582AA6@gmail.com> <20140123210308.0E1D65807E@chaos.jnpr.net> <EBDCAEEC-9485-4EA5-AA60-943EA70A3171@gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help

On Thu, 23 Jan 2014 13:30:12 -0800, Garrett Cooper writes:
>> Not crazy about frobbing ${MAKE}
>
>Neither am I, but .export is a bmake only feature. I=92m still using =
>fmake :(.

Not necessarily a good excuse ;-)

>> The semantics in bsd.own.mk are quite broken and result in a lot of =
>complex
>> dancing to keep things working.
>
>In this case though, this is complex dancing due to how the different =
>stages stack upon one another in the build process and the fact that =
>meta make isn=92t here (yet), so things have to be built in the right =
>order. This method is one that I=92ve been using for quite some time =
>without any side effects on multiple machines...

Actually a simple tweak to the bsd.own.mk semantics would alleviate of
lot of the pain.  Throwing errors if MK_* is already set, or if both
WITH_* and WITHOUT_* are set makes the usage very messy indeed.

For options.mk I allow MK_* to already be set and WITHOUT_* to take
precedence over WITH_*.  I also allow makefiles to have their own lists
of options - separate the policy from the mechanism.

I guess you could even allow a per-knob setting as to which takes
precedence. 

By simply allowing WITHOUT_* to overrule WITH_*, the Makefile.inc1 usage
would be greatly simplified.




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20140123215430.4B7B15807E>