Skip to content

Comments

Add Patina Config Enforcement#1324

Draft
os-d wants to merge 1 commit intoOpenDevicePartnership:mainfrom
os-d:config
Draft

Add Patina Config Enforcement#1324
os-d wants to merge 1 commit intoOpenDevicePartnership:mainfrom
os-d:config

Conversation

@os-d
Copy link
Contributor

@os-d os-d commented Feb 17, 2026

Description

Per RFC 25, this patch adds config.toml enforcement of the Patina config version.

config.toml is updated to reflect the various settings Patina wants to use. Each is documented on why.

build.rs verifies the patina config version is the correct one and will fail the build if not.

Docs are updated.

Note: This is in draft form as a companion to the RFC. When the RFC is completed, this will be published, the config.toml changes will go to patina-devops and the build.rs/doc changes will target the major branch.

  • Impacts functionality?
  • Impacts security?
  • Breaking change?
  • Includes tests?
  • Includes documentation?

How This Was Tested

Tested building Q35 and SBSA and booting them to Windows. Settings were validated in build.rs and by outside factors such as binary size.

Integration Instructions

Patina's .cargo/config.toml must be copied to the platform's .cargo/config.toml. See the docs for more details.

@os-d os-d linked an issue Feb 17, 2026 that may be closed by this pull request
@github-actions github-actions bot added impact:breaking-change Requires integration attention type:documentation Improvements or additions to documentation labels Feb 17, 2026
@codecov
Copy link

codecov bot commented Feb 17, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@cfernald cfernald self-requested a review February 18, 2026 01:01
Per RFC 25, this patch adds config.toml enforcement
of the Patina config version.

config.toml is updated to reflect the various settings Patina wants
to use. Each is documented on why.

build.rs verifies the patina config version is the correct one and
will fail the build if not.

Docs are updated.

Signed-off-by: Oliver Smith-Denny <osde@microsoft.com>
serde_json = { version = "1.0", default-features = false, features = ["alloc"] }
lzma-rs = { version = "0.3" }

[profile.release]
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note, this part will go in the file sync PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

impact:breaking-change Requires integration attention type:documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature]: Consider Downstream Build Options for Patina Consumers

4 participants