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.

Sign up for GitHub

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

Jump to bottom

app-crypt/tpm-tools: fix build on musl #29384

Closed
wants to merge 2 commits into from
Closed

app-crypt/tpm-tools: fix build on musl #29384

wants to merge 2 commits into from

Conversation

stintel
Copy link
Contributor

@stintel stintel commented Feb 1, 2023

No description provided.

@gentoo-bot
Copy link

Pull Request assignment

Submitter: @stintel
Areas affected: ebuilds
Packages affected: app-crypt/tpm-tools, app-crypt/trousers

app-crypt/tpm-tools: @salahcoronya, @gentoo/proxy-maint
app-crypt/trousers: @salahcoronya, @gentoo/proxy-maint

Linked bugs

Bugs linked: 713444

In order to force reassignment and/or bug reference scan, please append [please reassign] to the pull request title.

Docs: Code of Conduct ● Copyright policy (expl.) ● Devmanual ● GitHub PRs ● Proxy-maint guide

@gentoo-bot gentoo-bot added assigned PR successfully assigned to the package maintainer(s). bug linked Bug/Closes found in footer, and cross-linked with the PR. labels Feb 1, 2023
thesamesam
thesamesam reviewed Feb 1, 2023
View reviewed changes
app-crypt/trousers/files/trousers-0.3.15-tspi-drop-the-use-of-getpwent_r.patch
@@ -0,0 +1,67 @@
From 2299eadf77ae7a7ec52148b6a8f4ea37e217eafa Mon Sep 17 00:00:00 2001
Copy link
Member

Choose a reason for hiding this comment

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

Please send it upstream if you didn't already and include the link at the top of the patch.

Please also include a link to the relevant Gentoo bug at the top of the patch.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Gentoo link added. Unfortunately my upstream submission is still awaiting moderator approval.

@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2023-02-01 23:33 UTC
Newest commit scanned: e5786ff
Status: ✅ good

There are existing issues already. Please look into the report to make sure none of them affect the packages in question:
https://qa-reports.gentoo.org/output/gentoo-ci/7dfdbbf8a9/output.html

@stintel
app-crypt/trousers: add musl patch
35fb7a5 
Reference: https://bugs.gentoo.org/713444
Signed-off-by: Stijn Tintel <stijn@linux-ipv6.be>
@stintel
app-crypt/tpm-tools: revbump to depend on app-crypt/trousers-0.3.15-r1
1861983 
Closes: https://bugs.gentoo.org/713444
Signed-off-by: Stijn Tintel <stijn@linux-ipv6.be>
@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2023-02-14 03:13 UTC
Newest commit scanned: 1861983
Status: ✅ good

There are existing issues already. Please look into the report to make sure none of them affect the packages in question:
https://qa-reports.gentoo.org/output/gentoo-ci/aa228b4687/output.html

@stintel stintel deleted the tpm branch February 14, 2023 04:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@thesamesam thesamesam thesamesam left review comments

Assignees
No one assigned
Labels
assigned PR successfully assigned to the package maintainer(s). bug linked Bug/Closes found in footer, and cross-linked with the PR.
Projects
None yet
Milestone
No milestone
4 participants
@stintel @gentoo-bot @gentoo-repo-qa-bot @thesamesam