Skip to content

Conversation

achow101
Copy link
Member

@achow101 achow101 commented Apr 11, 2020

automake is needed to build osslsigncode otherwise autogen.sh fails with the docker virtualization method.

automake is needed to build osslsigncode otherwise autogen.sh fails.
Copy link
Member

@hebasto hebasto left a 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"
Copy link
Member

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.

Copy link
Member Author

Choose a reason for hiding this comment

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

meh.

Copy link
Member

@fanquake fanquake left a comment

Choose a reason for hiding this comment

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

ACK e44aeef

@hebasto
Copy link
Member

hebasto commented Apr 12, 2020

@achow101
I've managed to sign windows binaries without these changes :)

UPDATE: my base manifest already containes automake_1%3a1.15.1-3ubuntu2_all.deb
Is it due to using LXC?

@achow101
Copy link
Member Author

@achow101
I've managed to sign windows binaries without these changes :)

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.

@hebasto
Copy link
Member

hebasto commented Apr 12, 2020

Can you post your install.log?

It is not quite informative:

mesg: ttyname failed: Inappropriate ioctl for device
Hit:1 http://10.0.3.1:3142/archive.ubuntu.com/ubuntu bionic InRelease
Hit:2 http://10.0.3.1:3142/security.ubuntu.com/ubuntu bionic-security InRelease
Hit:3 http://10.0.3.1:3142/archive.ubuntu.com/ubuntu bionic-updates InRelease
Reading package lists...
mesg: ttyname failed: Inappropriate ioctl for device
Reading package lists...
Building dependency tree...
Reading state information...
faketime is already the newest version (0.9.7-2).
0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.
mesg: ttyname failed: Inappropriate ioctl for device

Which virtualization are you using?

LXC 3.0.3

@hebasto
Copy link
Member

hebasto commented Apr 12, 2020

@achow101

I use docker which has a very minimal dependency base.

Maybe it is worth to mention Docker in the OP?

@maflcko maflcko added this to the 0.20.0 milestone Apr 16, 2020
Copy link
Member

@jonatack jonatack left a 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.

@laanwj laanwj merged commit 8f24979 into bitcoin:master Apr 16, 2020
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Apr 17, 2020
…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
fanquake pushed a commit to fanquake/bitcoin that referenced this pull request Apr 23, 2020
automake is needed to build osslsigncode otherwise autogen.sh fails.

Github-Pull: bitcoin#18598
Rebased-From: e44aeef
@fanquake fanquake mentioned this pull request Apr 23, 2020
laanwj added a commit that referenced this pull request May 11, 2020
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
backpacker69 referenced this pull request in peercoin/peercoin Mar 28, 2021
automake is needed to build osslsigncode otherwise autogen.sh fails.

Github-Pull: #18598
Rebased-From: e44aeef
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Feb 15, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants