From 2b9170746945e5bfc09e2a235c70fee359a224a5 Mon Sep 17 00:00:00 2001 From: Brandon Dahler Date: Sun, 10 Nov 2013 21:11:33 -0600 Subject: [PATCH] Update build docs that refer to old makefile.* files. Also update pull-tester's error message. --- doc/build-msw.md | 6 ++++-- doc/build-osx.md | 26 ++++++++++---------------- qa/pull-tester/pull-tester.py | 2 +- 3 files changed, 15 insertions(+), 19 deletions(-) diff --git a/doc/build-msw.md b/doc/build-msw.md index e7e902eaa56..c9f01045eef 100644 --- a/doc/build-msw.md +++ b/doc/build-msw.md @@ -76,6 +76,8 @@ Bitcoin ------- DOS prompt: - cd \bitcoin\src - mingw32-make -f makefile.mingw + cd \bitcoin + sh autogen.sh + sh configure + mingw32-make strip bitcoind.exe diff --git a/doc/build-osx.md b/doc/build-osx.md index 6ded75d340f..f70da685e96 100644 --- a/doc/build-osx.md +++ b/doc/build-osx.md @@ -56,12 +56,13 @@ Installing the dependencies using MacPorts is very straightforward. 2. Build bitcoind: - cd src - make -f makefile.osx + ./autogen.sh + ./configure + make 3. It is a good idea to build and run the unit tests, too: - make -f makefile.osx test + make test Instructions: HomeBrew ---------------------- @@ -89,22 +90,15 @@ Rerunning "openssl version" should now return the correct version. git clone git@github.com:bitcoin/bitcoin.git bitcoin cd bitcoin -2. Modify source in order to pick up the `openssl` library. - - Edit `makefile.osx` to account for library location differences. There's a - diff in `contrib/homebrew/makefile.osx.patch` that shows what you need to - change, or you can just patch by doing - - patch -p1 < contrib/homebrew/makefile.osx.patch - -3. Build bitcoind: +2. Build bitcoind: - cd src - make -f makefile.osx + ./autogen.sh + ./configure + make -4. It is a good idea to build and run the unit tests, too: +3. It is a good idea to build and run the unit tests, too: - make -f makefile.osx test + make test Creating a release build ------------------------ diff --git a/qa/pull-tester/pull-tester.py b/qa/pull-tester/pull-tester.py index 6a7c45ccd12..b05ac2c5272 100755 --- a/qa/pull-tester/pull-tester.py +++ b/qa/pull-tester/pull-tester.py @@ -81,7 +81,7 @@ This pull does not merge cleanly onto current master""" + common_message} post_data = { "body" : "Automatic sanity-testing: FAILED BUILD/TEST, see " + linkUrl + " for binaries and test log." + """ This could happen for one of several reasons: -1. It chanages paths in makefile.linux-mingw or otherwise changes build scripts in a way that made them incompatible with the automated testing scripts (please tweak those patches in qa/pull-tester) +1. It chanages changes build scripts in a way that made them incompatible with the automated testing scripts (please tweak those patches in qa/pull-tester) 2. It adds/modifies tests which test network rules (thanks for doing that), which conflicts with a patch applied at test time 3. It does not build on either Linux i386 or Win32 (via MinGW cross compile) 4. The test suite fails on either Linux i386 or Win32