Page MenuHomeFreeBSD

Simplify and speed up the kyua build

Authored by arichardson on Jun 30 2021, 5:49 PM.



Instead of having multiple kyua libraries, just include the files as part
of usr.bin/kyua. Previously, we would build each kyua source up to four
times: once as a .o file and once as a .pieo. Additionally, the kyua
libraries might be built again for compat32. As all the kyua libraries
amount to 102 C++ sources the build time is significant (especially when
using an assertions enabled compiler).

Test Plan

still compiles

Diff Detail

rG FreeBSD src repository
Lint Not Applicable
Tests Not Applicable

Event Timeline

jrtc27 added inline comments.
1–2 ↗(On Diff #91578)

Do you really want this and the one in utils?

1–2 ↗(On Diff #91578)

Forgot to delete these two after prefixing all source files with <lib>/ (to resolve the name conflicts)

I guess we could also merge */ to the main makefile, but this change was simpler

I guess we could also merge */ to the main makefile, but this change was simpler

I tend to think they should be merged into the main makefile, but probably with per-library SRCS+= lines to ease updating if new files are added upstream. I don't feel all that strongly though.

This revision is now accepted and ready to land.Jul 1 2021, 5:43 PM
This revision now requires review to proceed.Jul 2 2021, 8:19 AM
This revision was not accepted when it landed; it landed in state Needs Review.Jul 2 2021, 8:29 AM
This revision was automatically updated to reflect the committed changes.