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

status: Fix build without GPGME #3132

Merged
merged 1 commit into from
Jan 5, 2024
Merged

Conversation

ericcurtin
Copy link
Collaborator

@ericcurtin ericcurtin commented Jan 5, 2024

If OSTREE_DISABLE_GPGME is not built in set remote to NULL.

The ostree_repo_signature_verify_commit_data path is irrelevant in the
no gpg case anyway. Having this set as NULL ensures an error gets
thrown early.

@ericcurtin
Copy link
Collaborator Author

Fixes broken build from:

#3131

@cgwalters
Copy link
Member

cgwalters commented Jan 5, 2024

BTW procedural point; we try to use "topic prefixes" in this repo and also the subject should be in the "imperative mood". So the commit message could look more like (but also rewording with the new suggestion):

status: Fix build without GPGME

Regression from previous change.

@ericcurtin ericcurtin changed the title If OSTREE_DISABLE_GPGME is not built in use osname/stateroot status: Fix build without GPGME Jan 5, 2024
@ericcurtin
Copy link
Collaborator Author

Repushed with changes.

If OSTREE_DISABLE_GPGME is not built in set remote to NULL.

The ostree_repo_signature_verify_commit_data path is irrelevant in the
no gpg case anyway. Having this set as NULL ensures an error gets
thrown early.

Signed-off-by: Eric Curtin <ecurtin@redhat.com>
@ericcurtin ericcurtin force-pushed the query-boot branch 2 times, most recently from 916e4cd to 37971a9 Compare January 5, 2024 16:03
@cgwalters cgwalters merged commit 501a077 into ostreedev:main Jan 5, 2024
45 checks passed
@ericcurtin ericcurtin deleted the query-boot branch January 6, 2024 12:39
@cgwalters cgwalters mentioned this pull request Jan 19, 2024
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