Skip to content

Added configure scripts.#2195

Closed
CodeShark wants to merge 1 commit intobitcoin:masterfrom
CodeShark:configure_script
Closed

Added configure scripts.#2195
CodeShark wants to merge 1 commit intobitcoin:masterfrom
CodeShark:configure_script

Conversation

@CodeShark
Copy link
Copy Markdown
Contributor

Basic dependency detection and assistance to user in installing missing dependencies. Automatic switch for USE_UPNP in makefile.

@laanwj
Copy link
Copy Markdown
Member

laanwj commented Jan 22, 2013

Nice, much better than a separate makefile for each platform

Edit: we probably want to add mingw32 (crosscompile) to this as well

@BitcoinPullTester
Copy link
Copy Markdown

Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/cb23ca73567011551b09326e616e869826128fd7 for binaries and test log.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This is not correct. uname -m gives the kernel architecture, not the OS architecture (which can be different in some scenarios).

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

uname -m isn't being used to determine the OS version. In fact, it's not being used at all right now. And it doesn't give the kernel version - it gives the architecture. This might be useful at some point if we want to have switches specific to i386 vs x64_64.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Yes, I meant architecture. Kernel arch != OS arch.

@CodeShark
Copy link
Copy Markdown
Contributor Author

@luke-jr: I added instructions for Ubuntu Precise users on how to install db4.8 using the steps you gave earlier.

@BitcoinPullTester
Copy link
Copy Markdown

Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/4c92791df5c96128e03834a8a1a52c07c2164cf5 for binaries and test log.

@gavinandresen
Copy link
Copy Markdown
Contributor

bitcoin-qt.pro.temp is out of date (missing some changes to bitcoin-qt.pro). And should be named bitcoin-qt.pro.template.

@jgarzik
Copy link
Copy Markdown
Contributor

jgarzik commented Jun 19, 2013

Closing, as an autotools effort is actively underway and preferred.

Can reopen, if that effort fails or stalls insurmountably.

@jgarzik jgarzik closed this Jun 19, 2013
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants