From: GitHub <no...@gi...> - 2012-02-04 15:12:25
|
Branch: refs/heads/master Home: https://github.com/znc/znc Commit: 4d1e97ed4775726c2e5e3207f3a9c866dc5aefe5 https://github.com/znc/znc/commit/4d1e97ed4775726c2e5e3207f3a9c866dc5aefe5 Author: Uli Schlachter <ps...@zn...> Date: 2012-02-04 (Sat, 04 Feb 2012) Changed paths: M m4/ax_pthread.m4 Log Message: ----------- Fix AX_PTHREAD to actually work At first the test was failing with the following error due to my CXXFLAGS: conftest.cpp:20:37: error: parameter 'a' set but not used [-Werror=unused-but-set-parameter] Then it figured out that -lpthread works, but I wondered why the test for "-pthread" still fails. This is where I realized that it is testing with C++ compiler, but puts its stuff in $CFLAGS which is just ignored. So a little search-and-replace later, this now uses $CXXFLAGS. At this point the results of the tests looked good, but znc failed to link, because it didn't use -pthread when linking. One little "PTHREAD_LIBS=" later, that problem was also gone and I can finally commit this. Yay. Signed-off-by: Uli Schlachter <ps...@zn...> |