Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

doc: Remove outdated qt4 install information from README.md #8048

Merged
merged 2 commits into from May 18, 2016

Conversation

Projects
None yet
4 participants
Owner

laanwj commented May 12, 2016

This text is aimed at users installing the binaries.

Now that Qt5 is linked statically, there is no need to install Qt as a run-time dependency.

Needs backport to 0.12.x.

doc: Remove outdated qt4 install information from README.md
This text is aimed at users installing the binaries.

Now that Qt5 is linked statically, there is no need to install Qt
as a run-time dependency.

@MarcoFalke MarcoFalke and 1 other commented on an outdated diff May 12, 2016

doc/README.md
@@ -11,17 +11,11 @@ The following are some helpful notes on how to run Bitcoin on your native platfo
### Unix
-You need the Qt4 run-time libraries to run Bitcoin-Qt. On Debian or Ubuntu:
-
- sudo apt-get install libqtgui4
-
Unpack the files into a directory and run:
- bin/32/bitcoin-qt (GUI, 32-bit) or bin/32/bitcoind (headless, 32-bit)
@MarcoFalke

MarcoFalke May 12, 2016

Member

There is no bin/32?

There are different tars for each arch?

@laanwj

laanwj May 12, 2016

Owner

Yes, good catch, that needs updating too.

Member

fanquake commented May 12, 2016

utACK once nits are fixed

Owner

laanwj commented May 18, 2016

Removed the 32/64 bit reference.

Contributor

paveljanik commented May 18, 2016

ACK 6075bc4

@laanwj laanwj merged commit 6075bc4 into bitcoin:master May 18, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

laanwj added a commit that referenced this pull request May 18, 2016

Merge #8048: doc: Remove outdated qt4 install information from README.md
6075bc4 doc: 32 and 64 bit packages are seperate (Wladimir J. van der Laan)
e5764e6 doc: Remove outdated qt4 install information from README.md (Wladimir J. van der Laan)

laanwj added a commit that referenced this pull request May 18, 2016

doc: Remove outdated qt4 install information from README.md
- doc: Remove outdated qt4 install information from README.md
- doc: 32 and 64 bit packages are seperate

Github-Pull: #8048
Rebased-From: e5764e6 6075bc4

@laanwj laanwj removed the Needs backport label May 18, 2016

nomnombtc added a commit to nomnombtc/bitcoin that referenced this pull request Nov 12, 2016

doc: Remove outdated qt4 install information from README.md
- doc: Remove outdated qt4 install information from README.md
- doc: 32 and 64 bit packages are seperate

Github-Pull: #8048
Rebased-From: e5764e6 6075bc4

nomnombtc added a commit to nomnombtc/bitcoin that referenced this pull request Nov 12, 2016

doc: Remove outdated qt4 install information from README.md
- doc: Remove outdated qt4 install information from README.md
- doc: 32 and 64 bit packages are seperate

Github-Pull: #8048
Rebased-From: e5764e6 6075bc4

nomnombtc added a commit to nomnombtc/bitcoin that referenced this pull request Nov 13, 2016

doc: Remove outdated qt4 install information from README.md
- doc: Remove outdated qt4 install information from README.md
- doc: 32 and 64 bit packages are seperate

Github-Pull: #8048
Rebased-From: e5764e6 6075bc4

sickpig added a commit to sickpig/BitcoinUnlimited that referenced this pull request Nov 14, 2016

doc: Remove outdated qt4 install information from README.md
- doc: Remove outdated qt4 install information from README.md
- doc: 32 and 64 bit packages are seperate

Github-Pull: #8048
Rebased-From: e5764e6 6075bc4
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment