mirror of
https://github.com/freebsd/freebsd-src
synced 2024-10-15 04:43:53 +00:00
bsd.subdir.mk: Drop broken optimisation for realinstall parallelisation
Not all of the tree is happy for realinstall to be done in parallel. In particular, Makefile.inc1 uses .WAIT to force etc to be installed after earlier subdirectories, since etc calls into share/man's makedb to run makewhatis on the tree and needs all manpages to have been installed. Also, libexec/Makefile doesn't set SUBDIR_PARALLEL, and the link from ld-elf32.1 to ld-elf.1 relies on rtld-elf having been installed before rtld-elf32, otherwise creating the link will fail. In general, core behavioural differences like this between NO_ROOT and "normal" builds are also dangerous and confusing. If this optimisation is deemed important, it should be reintroduced in a more limited and robust manner that doesn't break the above situations. Until then value correctness over slight efficiency gains on high core count machines, the same machines where you're more likely to encounter issues from this optimisation. This reverts commitscd19ecdbdc
("Similar to r296013 for NO_ROOT, force SUBDIR_PARALLEL for buildworld WORLDTMP staging.") andb9c6f31681
("Add more STANDALONE_SUBDIR_TARGETS."). Found by: CheriBSD Jenkins Reviewed by: bdrewery, brooks Fixes:cd19ecdbdc
("Similar to r296013 for NO_ROOT, force SUBDIR_PARALLEL for buildworld WORLDTMP staging.") Fixes:b9c6f31681
("Add more STANDALONE_SUBDIR_TARGETS.") MFC after: 1 week Differential Revision: https://reviews.freebsd.org/D43705
This commit is contained in:
parent
18968b8274
commit
fbae308319
|
@ -54,11 +54,6 @@ STANDALONE_SUBDIR_TARGETS+= \
|
||||||
installconfig installdirs installincludes installfiles print-dir \
|
installconfig installdirs installincludes installfiles print-dir \
|
||||||
maninstall manlint obj objlink
|
maninstall manlint obj objlink
|
||||||
|
|
||||||
# It is safe to install in parallel when staging.
|
|
||||||
.if defined(NO_ROOT) || !empty(SYSROOT)
|
|
||||||
STANDALONE_SUBDIR_TARGETS+= realinstall
|
|
||||||
.endif
|
|
||||||
|
|
||||||
.include <bsd.init.mk>
|
.include <bsd.init.mk>
|
||||||
|
|
||||||
.if ${MK_META_MODE} == "yes"
|
.if ${MK_META_MODE} == "yes"
|
||||||
|
|
Loading…
Reference in a new issue