The makefile for Visual C++ was outdated and broken. I updated it to support UPnP, use the new location of the resource files, and a couple other minor things that prevented it from building.
Updated Visual C++ Makefile #371
pull EricJ2190 wants to merge 1 commits into bitcoin:master from EricJ2190:master changing 1 files +24 −8-
EricJ2190 commented at 5:07 PM on July 1, 2011: contributor
-
Updated Visual C++ makefile. f8b5028cd1
-
TheBlueMatt commented at 8:10 PM on July 1, 2011: member
Does the resulting executable actually work 100%. There were some problems which I cant recall at the moment which cause the move off of vc.
-
EricJ2190 commented at 8:58 PM on July 1, 2011: contributor
It works fine when built without UPnP, but building without UPnP requires a slight modification to net.cpp. I am building without UPnP because I have been unable to make a successful build of MiniUPnP in Visual C++. I'll keep working on that, and let you know if I solve it.
-
EricJ2190 commented at 9:58 PM on July 1, 2011: contributor
Good new: UPnP works fine using the prebuilt DLL. Bad news: I have been building against an old (0.3.23) copy of the source. It looks like I'll need to do a bit more to get things working with the current Git tree.
-
EricJ2190 commented at 10:38 PM on July 1, 2011: contributor
Okay, I successfully got it to build the latest tree, with one issue: Visual C++ doesn't accept that CKeyStore::AddKey doesn't return a value. I had to add a return statement to get it to compile
- jgarzik referenced this in commit 056d2ad312 on Jul 1, 2011
- jgarzik merged this on Jul 1, 2011
- jgarzik closed this on Jul 1, 2011
- jtimon referenced this in commit 91ee21c024 on Mar 11, 2016
- zathras-crypto referenced this in commit e0db02ce64 on May 16, 2016
- laanwj referenced this in commit 0ffb3bd95f on Aug 4, 2016
- laanwj referenced this in commit b2135359b3 on Aug 16, 2016
- MarcoFalke referenced this in commit 64dfdde0aa on Dec 13, 2016
- deadalnix referenced this in commit bd2895fdd9 on Jan 19, 2017
- ptschip referenced this in commit 2399ba684d on Mar 16, 2017
- classesjack referenced this in commit 2a373e127e on Jan 2, 2018
- lateminer referenced this in commit 16326c6921 on Oct 16, 2019
- 0xartem referenced this in commit aec686344d on Oct 21, 2020
- cryptapus referenced this in commit 03c3ef837c on May 3, 2021
- rajarshimaitra referenced this in commit 7379592b99 on Aug 5, 2021
- DrahtBot locked this on Sep 8, 2021
Contributors