Skip to content

Conversation

fanquake
Copy link
Member

We don't use the gold linker.
binutils-gold just installs binutils (and the gold linker) in any case.

We don't use the gold linker.
binutils-gold just installs binutils (and the gold linker) in any case.
@fanquake fanquake added the Docs label Nov 25, 2022
@DrahtBot
Copy link
Contributor

DrahtBot commented Nov 25, 2022

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Reviews

See the guideline for information on the review process.

Type Reviewers
ACK hebasto

@hebasto
Copy link
Member

hebasto commented Nov 25, 2022

We don't use the gold linker.

Was it always the case?

@fanquake
Copy link
Member Author

Was it always the case?

As far as I can remember.

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 0953c62.

binutils-gold just installs binutils (and the gold linker) in any case.

On supported Debian and Ubuntu distros, the binutils-gold is a virtual package which provides the binutils one, and the latter in turn does install the gold linker anyway.

Of course, non-Debian-based distros can have different package structure.

FWIW, another virtual package, which we mention in our docs, is libz-dev which actually provides zlib1g-dev (see somewhat related discussion in #19783).

@fanquake fanquake merged commit 9c2854c into bitcoin:master Nov 28, 2022
@fanquake fanquake deleted the not_binutils_gold branch November 28, 2022 10:29
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Dec 1, 2022
0953c62 doc: install binutils, not binutils-gold in depends (fanquake)

Pull request description:

  We don't use the gold linker.
  binutils-gold just installs binutils (and the gold linker) in any case.

ACKs for top commit:
  hebasto:
    ACK 0953c62.

Tree-SHA512: c6e44c8d2fe9ef184aeb3b8aec87bac9c49ed0c3ef67de630b12da2945552b3dc4b9b3e75203a7d24a94d9586d59bb638e98f712dbc477681c95a793ee2327d9
@bitcoin bitcoin locked and limited conversation to collaborators Nov 28, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants