diff options
| author | imp <imp@FreeBSD.org> | 2014-10-22 03:39:11 +0000 |
|---|---|---|
| committer | imp <imp@FreeBSD.org> | 2014-10-22 03:39:11 +0000 |
| commit | e6fed04442dbd93b06a5c67fdbba446bf5122621 (patch) | |
| tree | 20a7489dbd0a71411bf02b8c35d3dff131507296 /lib/Makefile | |
| parent | 0944c94606d6d05249fac6fbe8c1922d9edf6526 (diff) | |
| download | FreeBSD-src-e6fed04442dbd93b06a5c67fdbba446bf5122621.zip FreeBSD-src-e6fed04442dbd93b06a5c67fdbba446bf5122621.tar.gz | |
My previous commit exposed an issue as it fixed a different
issue. lib/atf isn't a prereq_lib, since it isn't required for other
libraries to build. Remove it. The old kludge of always building it
had effectively been retired. Since we don't want to build the
libraries with the tests when we're bootstrapping, invent
MK_TESTS_SUPPORT which normally defaults to the current MK_TESTS
value, except when explicitly defined. Make lib/atf depend on it being
yes. When building the libraries set MK_TESTS to no, and
MK_TESTS_SUPPORT to the current value of MK_TESTS so that later stages
of the build work correctly. This should fix (and does for me)
people's issues with parallel builds racing between lib/atf and
libexec/atf. Since lib/atf is built during the libraries phase, the
race disappears.
Diffstat (limited to 'lib/Makefile')
| -rw-r--r-- | lib/Makefile | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/lib/Makefile b/lib/Makefile index c558f11..b683a90 100644 --- a/lib/Makefile +++ b/lib/Makefile @@ -276,8 +276,10 @@ _libsmutil= libsmutil _libtelnet= libtelnet .endif -.if ${MK_TESTS} != "no" +.if ${MK_TESTS_SUPPORT} != "no" _atf= atf +.endif +.if ${MK_TESTS} != "no" _tests= tests .endif |
