-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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-tools: Don't let configure call git, add missing dependency, fix libressl support #18721
Conversation
Pull Request assignmentSubmitter: @salahcoronya app-crypt/tpm2-tools: @salahcoronya, @gentoo/proxy-maint Linked bugsBugs linked: 760567 In order to force reassignment and/or bug reference scan, please append Docs: Code of Conduct ● Copyright policy (expl.) ● Devmanual ● GitHub PRs ● Proxy-maint guide |
Pull request CI reportReport generated at: 2020-12-19 03:24 UTC There are existing issues already. Please look into the report to make sure none of them affect the packages in question: |
Closes: https://bugs.gentoo.org/760567 Package-Manager: Portage-3.0.9, Repoman-3.0.2 Signed-off-by: Salah Coronya <salah.coronya@gmail.com>
Package-Manager: Portage-3.0.9, Repoman-3.0.2 Signed-off-by: Salah Coronya <salah.coronya@gmail.com>
Closes: https://bugs.gentoo.org/760567 Package-Manager: Portage-3.0.9, Repoman-3.0.2 Signed-off-by: Salah Coronya <salah.coronya@gmail.com>
Package-Manager: Portage-3.0.9, Repoman-3.0.2 Signed-off-by: Salah Coronya <salah.coronya@gmail.com>
Package-Manager: Portage-3.0.9, Repoman-3.0.2 Signed-off-by: Salah Coronya <salah.coronya@gmail.com>
Package-Manager: Portage-3.0.9, Repoman-3.0.2 Signed-off-by: Salah Coronya <salah.coronya@gmail.com>
260c29e
to
3935467
Compare
Pull request CI reportReport generated at: 2020-12-30 23:29 UTC There are existing issues already. Please look into the report to make sure none of them affect the packages in question: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well Gentoo is getting rid of libressl, but I guess this ships other fixes too.
configure.ac has the follow line at the top:
AC_INIT([tpm2-tools],
[m4_esyscmd_s([git describe --tags --always --dirty])])
This causes "git" to be invoked when "configure" is run, it is used to embed the version number into the apps. Rather than depend on git, use "sed" in src_prepare to remove the 2nd part and replace it with ${PV}. Also, a missing dependency against sys-devel/autoconf-archive is added.