This should fix #10836,
Add stdlib include in qt/test/test_main (for setenv) #10846
pull TheBlueMatt wants to merge 1 commits into bitcoin:master from TheBlueMatt:2017-07-env-include changing 1 files +2 −0-
TheBlueMatt commented at 6:03 PM on July 16, 2017: member
-
Add stdlib include in qt/test/test_main (for setenv) 3510f02d79
-
TheBlueMatt commented at 6:24 PM on July 16, 2017: member
Heh, apparently the guy was building wrong.
- TheBlueMatt closed this on Jul 16, 2017
-
globaltoken commented at 11:25 AM on July 17, 2017: none
No, I want to build Windows Wallets not Linux,
-
TheBlueMatt commented at 3:02 PM on July 17, 2017: member
@globaltoken can you test with this PR? Does it work afterwards?
-
globaltoken commented at 7:23 PM on July 18, 2017: none
@TheBlueMatt You mean I should try with this Fix the Compiling for Windows ?
-
TheBlueMatt commented at 8:50 PM on July 19, 2017: member
Yes, try building this branch and see if it works for you.
-
globaltoken commented at 11:09 PM on July 20, 2017: none
Hey mate,
No its not working :( Same issue here.
CXX qt/test/qt_test_test_bitcoin_qt-test_main.o qt/test/test_main.cpp: In function ‘int main(int, char**)’: qt/test/test_main.cpp:63:43: error: ‘setenv’ was not declared in this scope setenv("QT_QPA_PLATFORM", "minimal", 0);
-
globaltoken commented at 7:32 AM on July 26, 2017: none
The issue is now solved - thanks anyway!
- MarcoFalke locked this on Sep 8, 2021
Contributors