Fix makefile.unix to link wxWidgets #81

pull mgiuca wants to merge 2 commits into bitcoin:master from mgiuca:master changing 2 files +4 −10

mgiuca commented at 7:51 AM on February 25, 2011: contributor

I tried to build bitcoin following the instructions in build-unix.txt. I suppose I should have read this line: "The build hasn't been updated to work with wxWidgets 2.9.1 yet."

Anyway, the makefile is not very robust in specifically including wx files directly in the INCLUDEPATHS and WXLIBS. I had a bunch of build problems which were all solved by simply using the utility that comes with wx, wx-config. This utility is automatically installed by wx's 'make install', and will provide the correct command-line arguments for including and linking correctly.

I have updated the makefile to use these tools rather than hand-coding the paths. It works for me, but I haven't tested it on multiple configurations. If it works for other people (and I figure it will be much more robust in the long run), then it should be merged.

My branch also adds .gitignore, which I find useful.

makefile.unix: Fix WX include and library paths.

    Use wx-config utility, instead of hard-coding filenames (which can change
    across WX versions or even with different build options).

e158f9f786

Added .gitignore, which prevents 'bitcoin' and 'bitcoind' from showing up in git. e8c5134e0d
tcatm commented at 2:24 PM on February 25, 2011: none

There's already a very similar pull request: #52
zathras-crypto referenced this in commit 6b0b617450 on Jun 12, 2015
CryptAxe referenced this in commit 7a61b377bb on Mar 11, 2018
cryptapus referenced this in commit 4230bc3a78 on Aug 13, 2018
lateminer referenced this in commit 3d2b4200ef on Feb 23, 2019
attilaaf referenced this in commit 19654159fe on Sep 20, 2019
Losangelosgenetics referenced this in commit 09d58e9a40 on Mar 12, 2020
DrahtBot locked this on Sep 8, 2021

Contributors