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

app-crypt/tpm2-pkcs11: Bump to 1.7.0 #22426

Closed
wants to merge 2 commits into from

Conversation

salahcoronya
Copy link
Contributor

The tpm2_pool utility gains a new operation, Import but requires >=app-crypt/tpm2-tool-5.2 (Not released yet).

I'd like to enable more of the tests, but there's a few problems doing so:

  1. Requires dev-python/python-pkcs11 - not in portage
  2. Requires >=app-crypt/tpm2-tool-5.2 - not released yet
  3. One of test downloads stuff off the Internet, so it should be patched out (or fixed, if possible)
  4. But the main issue issue one of the tests is written in Java. None of the java eclasses support a build-only dependency to Java.

@gentoo-bot
Copy link

Pull Request assignment

Submitter: @salahcoronya
Areas affected: ebuilds
Packages affected: app-crypt/tpm2-pkcs11

app-crypt/tpm2-pkcs11: @salahcoronya, @gentoo/proxy-maint

Linked bugs

Bugs linked: 813735


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

Docs: Code of ConductCopyright policy (expl.) ● DevmanualGitHub PRsProxy-maint guide

@gentoo-bot gentoo-bot added self-maintained The PR changes only packages that are maintained by the submitter (i.e. no need to ask anybody else) assigned PR successfully assigned to the package maintainer(s). bug linked Bug/Closes found in footer, and cross-linked with the PR. labels Sep 27, 2021
@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2021-09-27 22:29 UTC
Newest commit scanned: 374e86f
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/6b3a578d32/output.html

Closes: https://bugs.gentoo.org/813735
Package-Manager: Portage-3.0.20, Repoman-3.0.3
Signed-off-by: Christopher Byrne <salah.coronya@gmail.com>
Package-Manager: Portage-3.0.20, Repoman-3.0.3
Signed-off-by: Christopher Byrne <salah.coronya@gmail.com>
@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2021-09-28 03:59 UTC
Newest commit scanned: b7a0984
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/7544cc2a83/output.html

Copy link
Member

@juippis juippis left a comment

Choose a reason for hiding this comment

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

Requires dev-python/python-pkcs11 - not in portage

If you find it worth, you can add that when next modifying this package somehow.

One of test downloads stuff off the Internet, so it should be patched out (or fixed, if possible)

Yes, if possible, you should pre-download the required files with test? ( ... ) USE conditional.

But the main issue issue one of the tests is written in Java. None of the java eclasses support a build-only dependency to Java.

I don't know java at all, but couldn't you just straight up DEPEND/BDEPEND on some java?

@salahcoronya salahcoronya deleted the tpm2-pkcs11 branch September 28, 2021 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
assigned PR successfully assigned to the package maintainer(s). bug linked Bug/Closes found in footer, and cross-linked with the PR. self-maintained The PR changes only packages that are maintained by the submitter (i.e. no need to ask anybody else)
Projects
None yet
4 participants