Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Write generated Rust code to the source tree; apply fixups #6

Merged
merged 6 commits into from
Aug 4, 2021
Merged

Write generated Rust code to the source tree; apply fixups #6

merged 6 commits into from
Aug 4, 2021

Conversation

bgilbert
Copy link
Contributor

@bgilbert bgilbert commented Aug 3, 2021

Store generated Rust code in Git, and regenerate it only when built with cargo build --features regenerate. Then apply fixups:

Requires Git version of Schemafy.

Prepare to keep generated Rust code in Git.  Regenerate it only when
built with `cargo build --features regenerate`.

Add anyhow as an unconditional build dependency because it makes the
top-level error handling in build.rs easier.

Requires Git version of Schemafy.
start_mi_b => start_mib
size_mi_b  => size_mib
tpm_2      => tpm2

Fixes #2.
Copy link
Member

@jlebon jlebon left a comment

Choose a reason for hiding this comment

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

One comment, but LGTM otherwise.
Nice work!

build.rs Show resolved Hide resolved
Schemafy generates both a context-specific
DirectoryUser/DirectoryGroup/etc. and a NodeUser/NodeGroup it doesn't use.
Switch all references to the latter.

Works around Marwes/schemafy#50.
@bgilbert bgilbert merged commit 7b9a6d7 into coreos:main Aug 4, 2021
@bgilbert bgilbert deleted the generate branch August 4, 2021 16:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Strange field names
2 participants