-
Notifications
You must be signed in to change notification settings - Fork 37.7k
gitian: Add missing automake package to gitian-win-signer.yml #18598
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
Conversation
automake is needed to build osslsigncode otherwise autogen.sh fails.
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.
ACK e44aeef, for osslsigncode-1.7.1
we did not run autogen.sh
in the past.
It seems the backport to the 0.20 branch is needed.
@@ -8,6 +8,7 @@ architectures: | |||
packages: | |||
- "libssl-dev" | |||
- "autoconf" | |||
- "automake" |
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.
nit: automake
depends on autoconf
, therefore, the latter could be dropped.
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.
meh.
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.
ACK e44aeef
@achow101 UPDATE: my base manifest already containes |
That's interesting. Which virtualization are you using? Can you post your install.log? I use docker which has a very minimal dependency base. So if you are using something else, maybe a dependency is installed that requires automake. |
It is not quite informative:
LXC 3.0.3 |
Maybe it is worth to mention Docker in the OP? |
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.
ACK e44aeef
Came across this when signing 0.20.0.rc1 for win with docker.
…in-signer.yml e44aeef gitian: Add missing automake package to gitian-win-signer.yml (Andrew Chow) Pull request description: automake is needed to build osslsigncode otherwise autogen.sh fails with the docker virtualization method. ACKs for top commit: hebasto: ACK e44aeef, for `osslsigncode-1.7.1` we did not run `autogen.sh` in the past. fanquake: ACK e44aeef jonatack: ACK e44aeef Tree-SHA512: a0e615c1b099ee1c469ce41f886f2ece6746234a5a800743a4e8be671e4114fd30e1c35bc0ddcb75778409564129d0fde7ac4e3d70b0f7691f97f729f34c8e0c
automake is needed to build osslsigncode otherwise autogen.sh fails. Github-Pull: bitcoin#18598 Rebased-From: e44aeef
7f7548d rpc: Do not advertise dumptxoutset as a way to flush the chainstate (MarcoFalke) a9ca65b Fix naming of macOS SDK and clarify version (Andrew Chow) 54d2063 Do not expose and consider -logthreadnames when it does not work (Hennadii Stepanov) 6986b26 build: fix ASLR for bitcoin-cli on Windows (fanquake) 1d1e358 build: Set libevent minimum version to 2.0.21 (Hennadii Stepanov) 842b13a Avoid non-trivial global constants in SHA-NI code (Pieter Wuille) ade4185 gitian: Add missing automake package to gitian-win-signer.yml (Andrew Chow) Pull request description: Currently backports the following to the 0.20 branch: * #18598 - gitian: Add missing automake package to gitian-win-signer.yml * #18702 - build: fix ASLR for bitcoin-cli on Windows * #18676 - build: Check libevent minimum version in configure script * #18665 - Do not expose and consider -logthreadnames when it does not work * #18553 - Avoid non-trivial global constants in SHA-NI code * #18589 - Fix naming of macOS SDK and clarify version ACKs for top commit: laanwj: ACK 7f7548d Tree-SHA512: 2cba748414a440e3fb901940085a7ae059e8b926c9187fbbbdeb7846a32e7374f318cc21e499c911ff803c42aef2c844b04af10b87f9c5a2b3edf6deb03ebb04
automake is needed to build osslsigncode otherwise autogen.sh fails. Github-Pull: #18598 Rebased-From: e44aeef
automake is needed to build osslsigncode otherwise autogen.sh fails with the docker virtualization method.