config: drop dependency on libnv
ClosedPublic
Actions

Authored by kevans on Jan 30 2023, 3:54 PM.

Details

Reviewers

imp
jrtc27

Commits

rG29c5f8bf9a01: config: drop dependency on libnv
rG4e8636941785: tools: cleanup stale dependencies from recent config(8) change

Summary

Compile mkmakefile.c as C++ instead and use an std::unordered_map to
accomplish the same goal.

Sponsored by: Klara, Inc.
Sponsored by: NetApp, Inc.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 49323
Build 46213: arc lint + arc unit

Event Timeline

kevans requested review of this revision.Jan 30 2023, 3:54 PM

kevans created this revision.

Harbormaster completed remote builds in B49323: Diff 116045.Jan 30 2023, 3:54 PM

kevans added a child revision: D38276: config: drop dependency on libsbuf.Jan 30 2023, 3:54 PM

kevans added a parent revision: D38274: config: make changes to allow some parts to build as C++.Jan 30 2023, 3:55 PM

My C++ fu is weak these days... but such as it is, I see no issues and think this is a great leap sideways...
Since config doesn't have to run on the TARGET system until it's self hosting, and since we need C++ for that anyway
and since we can run config on a host with C++ fully realized, this shouldn't affect early bring up of new targets.