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

prepare-root: A few cleanups #2926

Merged
merged 3 commits into from
Jul 13, 2023
Merged

Conversation

cgwalters
Copy link
Member

Add an internal constant for the composefs image name

Just a minor cleanup.


prepare-root: Use otutil and g_print

Now that we link to libotutil (and glib) we don't need separate
handling for conditionalizing on the presence of systemd.

Further, there's no value in sd_journal_send(MESSAGE=) over
just printing to stdout.


prepare-root: Drop unused verity flag querying

This logic got pushed down into libcomposefs.


Copy link
Member

@alexlarsson alexlarsson left a comment

Choose a reason for hiding this comment

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

lgtm

@alexlarsson
Copy link
Member

test-concurrency.py strikes again!

@alexlarsson
Copy link
Member

Oh, and clang-format

Now that we link to libotutil (and glib) we don't need separate
handling for conditionalizing on the presence of systemd.

Further, there's no value in `sd_journal_send(MESSAGE=)` over
just printing to stdout.
This logic got pushed down into libcomposefs.
@cgwalters
Copy link
Member Author

/retest

2 similar comments
@cgwalters
Copy link
Member Author

/retest

@cgwalters
Copy link
Member Author

/retest

@cgwalters cgwalters merged commit 6cdc5ce into ostreedev:main Jul 13, 2023
20 checks passed
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.

2 participants