Skip to content

Conversation

@hebasto
Copy link
Member

@hebasto hebasto commented Oct 6, 2019

Since libtool 1.5.2, on Linux libtool no longer sets RPATH for any directories in the dynamic linker search path, so there is no longer an issue.

This commit reverts a98356f.

Refs:

Since libtool 1.5.2, on Linux libtool no longer sets RPATH for any
directories in the dynamic linker search path, so there is no longer an
issue.
This commit reverts a98356f.
@practicalswift
Copy link
Contributor

Concept ACK -- old cruft should be pruned :)

@maflcko maflcko requested a review from theuni October 6, 2019 20:42
@DrahtBot
Copy link
Contributor

DrahtBot commented Oct 7, 2019

Gitian builds for commit 7b701fe (master):

Gitian builds for commit 817977f50417728efa320bcb38fa7005eef08e6b (master and this pull):

@laanwj
Copy link
Member

laanwj commented Oct 8, 2019

ACK 30fc1a3

laanwj added a commit that referenced this pull request Oct 8, 2019
30fc1a3 build: Remove workaround for ancient libtool (Hennadii Stepanov)
6ca01b9 build: Ensure a minimal version of libtool (Hennadii Stepanov)

Pull request description:

  Since libtool 1.5.2, on Linux libtool no longer sets RPATH for any directories in the dynamic linker search path, so there is no longer an issue.

  This commit reverts a98356f.

  Refs:
  - https://wiki.debian.org/RpathIssue
  - [Debian jessie has libtool 2.4.2](https://packages.debian.org/jessie/libtool)

ACKs for top commit:
  laanwj:
    ACK 30fc1a3

Tree-SHA512: fab56265d4d2c96216a353cc076c6f510e15748d8134f97bae2f67b6d8c0b6a1a9f362d2ab23b19ccc3a8bba8eac3bb1668fc3e42037590f63a7ab4819c9ee15
@laanwj laanwj merged commit 30fc1a3 into bitcoin:master Oct 8, 2019
@hebasto hebasto deleted the 20191006-rpath-issue branch October 8, 2019 09:56
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Oct 8, 2019
30fc1a3 build: Remove workaround for ancient libtool (Hennadii Stepanov)
6ca01b9 build: Ensure a minimal version of libtool (Hennadii Stepanov)

Pull request description:

  Since libtool 1.5.2, on Linux libtool no longer sets RPATH for any directories in the dynamic linker search path, so there is no longer an issue.

  This commit reverts a98356f.

  Refs:
  - https://wiki.debian.org/RpathIssue
  - [Debian jessie has libtool 2.4.2](https://packages.debian.org/jessie/libtool)

ACKs for top commit:
  laanwj:
    ACK 30fc1a3

Tree-SHA512: fab56265d4d2c96216a353cc076c6f510e15748d8134f97bae2f67b6d8c0b6a1a9f362d2ab23b19ccc3a8bba8eac3bb1668fc3e42037590f63a7ab4819c9ee15
MarkLTZ added a commit to litecoinz-core/litecoinz that referenced this pull request Apr 6, 2020
- Fix gcc 9 warnings bitcoin#16995
- guix: Remove now-unnecessary gcc make flag bitcoin#18320
- build: Remove workaround for ancient libtool bitcoin#17066
- build: Add variable printing target to Makefiles bitcoin#17087
- Disable _FORTIFY_SOURCE when enable-debug bitcoin#17033
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Jul 1, 2021
30fc1a3 build: Remove workaround for ancient libtool (Hennadii Stepanov)
6ca01b9 build: Ensure a minimal version of libtool (Hennadii Stepanov)

Pull request description:

  Since libtool 1.5.2, on Linux libtool no longer sets RPATH for any directories in the dynamic linker search path, so there is no longer an issue.

  This commit reverts a98356f.

  Refs:
  - https://wiki.debian.org/RpathIssue
  - [Debian jessie has libtool 2.4.2](https://packages.debian.org/jessie/libtool)

ACKs for top commit:
  laanwj:
    ACK 30fc1a3

Tree-SHA512: fab56265d4d2c96216a353cc076c6f510e15748d8134f97bae2f67b6d8c0b6a1a9f362d2ab23b19ccc3a8bba8eac3bb1668fc3e42037590f63a7ab4819c9ee15
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Jul 13, 2021
30fc1a3 build: Remove workaround for ancient libtool (Hennadii Stepanov)
6ca01b9 build: Ensure a minimal version of libtool (Hennadii Stepanov)

Pull request description:

  Since libtool 1.5.2, on Linux libtool no longer sets RPATH for any directories in the dynamic linker search path, so there is no longer an issue.

  This commit reverts a98356f.

  Refs:
  - https://wiki.debian.org/RpathIssue
  - [Debian jessie has libtool 2.4.2](https://packages.debian.org/jessie/libtool)

ACKs for top commit:
  laanwj:
    ACK 30fc1a3

Tree-SHA512: fab56265d4d2c96216a353cc076c6f510e15748d8134f97bae2f67b6d8c0b6a1a9f362d2ab23b19ccc3a8bba8eac3bb1668fc3e42037590f63a7ab4819c9ee15
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Dec 16, 2021
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.

6 participants