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

Allow enabling/disabling the use of genpeimg binary #87

Merged
merged 1 commit into from
Jul 3, 2024

Conversation

jameshilliard
Copy link
Contributor

In some cases we may want to force the use of python3-pefile instead of genpeimg such as when cross compiling, add a feature option to allow disabling genpeimg.

In some cases we may want to force the use of python3-pefile instead
of genpeimg such as when cross compiling, add a feature option to
allow disabling genpeimg.

Signed-off-by: James Hilliard <[email protected]>
@hughsie hughsie requested a review from superm1 July 3, 2024 06:31
@superm1 superm1 merged commit fb1cf07 into fwupd:main Jul 3, 2024
4 of 5 checks passed
@superm1
Copy link
Member

superm1 commented Jul 3, 2024

Apparently I should have waited for Fedora CI to finish. @jameshilliard can you fix up Fedora CI?

@jameshilliard jameshilliard deleted the genpeimg-feature branch July 3, 2024 22:15
@jameshilliard
Copy link
Contributor Author

@jameshilliard can you fix up Fedora CI?

I think #88 should fix that failure.

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

Successfully merging this pull request may close these issues.

2 participants