Introduce bsd.sysdir.mk to consolidate looking for the kernel.

[jhb]

I guess the extra directories don't hurt. I'm less certain about the implications of the missing checks for an empty KERNBUILDDIR.

share/mk/bsd.dtb.mk

3

Does this need to be wrapped in .if empty(KERNBUILDDIR)?

share/mk/bsd.kmod.mk

3

Same.

sys/conf/kmod.mk

82

Same?

[bdrewery]

In D22144#484006, @jhb wrote:

I guess the extra directories don't hurt. I'm less certain about the implications of the missing checks for an empty KERNBUILDDIR.

Same concerns but I like the change otherwise.

[jhb]

Ping. The kmod.opts.mk change is a prerequisite for bhyve save/restore, so I'd like to either commit that one as-is and do this cleanup afterwards, or get this one unstuck to make forward progress. I still have questions about the KERNBUILDDIR stuff as I don't fully understand the implications of removing those checks.

[imp]

share/mk/bsd.dtb.mk

3

No. I don't think so. KERNBUILDDIR I think is actually irrelevant. If SYSDIR being defined or not is all that matters here. When we define it, we also define SYSDIR, so the !defined() is sufficient.

[imp]

In D22144#484174, @bdrewery wrote:

In D22144#484006, @jhb wrote:

I guess the extra directories don't hurt. I'm less certain about the implications of the missing checks for an empty KERNBUILDDIR.

Same concerns but I like the change otherwise.

When we define KERNBUILDDIR, we also define SYSDIR, so checking it is redundant.

[jhb]

Ping.

[imp]

Running a final make universe before committing.
Should be in tomorrow morning if nothing breaks.