* [Bitcoin-development] libcoin (HEAD) now supports boost < 1.47 - please test
@ 2012-02-02 13:46 Michael Grønager
2012-02-02 16:30 ` Luke-Jr
0 siblings, 1 reply; 4+ messages in thread
From: Michael Grønager @ 2012-02-02 13:46 UTC (permalink / raw)
To: Bitcoin Dev
I have added a simplified fall back class to the boost::asio::signal_set. This should enable compilation on platforms with less than bleeding edge versions of Boost. Most notably most of the currently deployed Linux'es that use Boost 1.42.
I also updated the root CMakeLists.txt to only require 1.42. It works for me, but I also recognize the fact that committers machines has some intrinsic magic that just makes things work, hiding actual errors for the them ;)
Please test and feed back.
Cheers,
Michael
Michael Gronager, PhD
Director, Ceptacle
Jens Juels Gade 33
2100 Copenhagen E
Mobile: +45 31 45 14 01
E-mail: gronager@ceptacle•com
Web: http://www.ceptacle.com/
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [Bitcoin-development] libcoin (HEAD) now supports boost < 1.47 - please test
2012-02-02 13:46 [Bitcoin-development] libcoin (HEAD) now supports boost < 1.47 - please test Michael Grønager
@ 2012-02-02 16:30 ` Luke-Jr
2012-02-02 22:43 ` Michael Grønager
0 siblings, 1 reply; 4+ messages in thread
From: Luke-Jr @ 2012-02-02 16:30 UTC (permalink / raw)
To: Michael Grønager; +Cc: bitcoin-development
On Thursday, February 02, 2012 8:46:05 AM Michael Grønager wrote:
> Please test and feed back.
I found the problem: you are trying to use static libraries. Best practices
are to use shared libraries (except for specific scenarios like universal
"Linux" binaries) and most distros do not have static libraries installed by
default.
The coinQt stuff was also creating a problem.
I am able to build with this patch:
diff --git a/CMakeLists.txt b/CMakeLists.txt
index dea37c4..b876881 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -352,7 +352,7 @@ ENDIF(BDB_FOUND)
#Note: We need as a minimum Boost 1.47 to support the signal_set used in
Server. A backup signal_set has been created, though.
SET(Boost_NO_BOOST_CMAKE ON)
SET(Boost_ADDITIONAL_VERSIONS "1.47" "1.47.0" "1.48" "1.48.0")
-SET(Boost_USE_STATIC_LIBS ON)
+#SET(Boost_USE_STATIC_LIBS ON)
SET(Boost_USE_MULTITHREADED ON)
SET(Boost_USE_STATIC_RUNTIME OFF)
@@ -804,9 +804,9 @@ SET(PKGCONFIG_FILES
# libcoin-coinMine
)
-IF(QT4_FOUND)
- SET(PKGCONFIG_FILES ${PKGCONFIG_FILES} libcoin-coinQt)
-ENDIF(QT4_FOUND)
+#IF(QT4_FOUND)
+# SET(PKGCONFIG_FILES ${PKGCONFIG_FILES} libcoin-coinQt)
+#ENDIF(QT4_FOUND)
FOREACH(PKGCONFIG_FILE ${PKGCONFIG_FILES})
CONFIGURE_FILE(${PROJECT_SOURCE_DIR}/packaging/pkgconfig/${PKGCONFIG_FILE}.pc.in
diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt
index 0011392..67044d1 100644
--- a/src/CMakeLists.txt
+++ b/src/CMakeLists.txt
@@ -19,9 +19,9 @@ FOREACH( mylibfolder
ENDFOREACH()
-IF (QT4_FOUND AND NOT ANDROID)
- ADD_SUBDIRECTORY(coinQt)
-ENDIF()
+#IF (QT4_FOUND AND NOT ANDROID)
+# ADD_SUBDIRECTORY(coinQt)
+#ENDIF()
IF(ANDROID)
configure_file("${LIBCOIN_ANDROID_TEMPLATES}/Android.mk.src.in"
"${CMAKE_CURRENT_BINARY_DIR}/Android.mk")
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [Bitcoin-development] libcoin (HEAD) now supports boost < 1.47 - please test
2012-02-02 16:30 ` Luke-Jr
@ 2012-02-02 22:43 ` Michael Grønager
2012-02-02 23:27 ` Luke-Jr
0 siblings, 1 reply; 4+ messages in thread
From: Michael Grønager @ 2012-02-02 22:43 UTC (permalink / raw)
To: Luke-Jr; +Cc: bitcoin-development
Thanks for the patch and the detective work!
Enabling dynamic libs was on my TODO, but on the Redmond_OS_not_to_be_mentioned you need to :
* prepend class definitions with __declspec(dllexport) when you compile the dll
* prepend class definitions with __declspec(dllimport) when you use the dll
I just love the way they spoil their developers...
I have the framework to automate this with CMake from a former project, but I havn't tested it for libcoin yet, hence the static build. And well, iOS also has this fetich for static libs.
I also recall another issue with dll's: If you define a global variable it is shared between all executables using this dll. I have still a handful of globals to clean out, namely those related to logging - I will do so, but it has not yet been a top priority.
So, feel free to use dynamic libs on unix'es, but on windows it is .libs for a little while longer.
Will fix the Qt stuff in CMake - thanks!
Cheers,
Michael
On 02/02/2012, at 17:30, Luke-Jr wrote:
> On Thursday, February 02, 2012 8:46:05 AM Michael Grønager wrote:
>> Please test and feed back.
>
> I found the problem: you are trying to use static libraries. Best practices
> are to use shared libraries (except for specific scenarios like universal
> "Linux" binaries) and most distros do not have static libraries installed by
> default.
>
> The coinQt stuff was also creating a problem.
>
> I am able to build with this patch:
> diff --git a/CMakeLists.txt b/CMakeLists.txt
> index dea37c4..b876881 100644
> --- a/CMakeLists.txt
> +++ b/CMakeLists.txt
> @@ -352,7 +352,7 @@ ENDIF(BDB_FOUND)
> #Note: We need as a minimum Boost 1.47 to support the signal_set used in
> Server. A backup signal_set has been created, though.
> SET(Boost_NO_BOOST_CMAKE ON)
> SET(Boost_ADDITIONAL_VERSIONS "1.47" "1.47.0" "1.48" "1.48.0")
> -SET(Boost_USE_STATIC_LIBS ON)
> +#SET(Boost_USE_STATIC_LIBS ON)
> SET(Boost_USE_MULTITHREADED ON)
> SET(Boost_USE_STATIC_RUNTIME OFF)
>
> @@ -804,9 +804,9 @@ SET(PKGCONFIG_FILES
> # libcoin-coinMine
> )
>
> -IF(QT4_FOUND)
> - SET(PKGCONFIG_FILES ${PKGCONFIG_FILES} libcoin-coinQt)
> -ENDIF(QT4_FOUND)
> +#IF(QT4_FOUND)
> +# SET(PKGCONFIG_FILES ${PKGCONFIG_FILES} libcoin-coinQt)
> +#ENDIF(QT4_FOUND)
>
> FOREACH(PKGCONFIG_FILE ${PKGCONFIG_FILES})
> CONFIGURE_FILE(${PROJECT_SOURCE_DIR}/packaging/pkgconfig/${PKGCONFIG_FILE}.pc.in
> diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt
> index 0011392..67044d1 100644
> --- a/src/CMakeLists.txt
> +++ b/src/CMakeLists.txt
> @@ -19,9 +19,9 @@ FOREACH( mylibfolder
>
> ENDFOREACH()
>
> -IF (QT4_FOUND AND NOT ANDROID)
> - ADD_SUBDIRECTORY(coinQt)
> -ENDIF()
> +#IF (QT4_FOUND AND NOT ANDROID)
> +# ADD_SUBDIRECTORY(coinQt)
> +#ENDIF()
>
> IF(ANDROID)
> configure_file("${LIBCOIN_ANDROID_TEMPLATES}/Android.mk.src.in"
> "${CMAKE_CURRENT_BINARY_DIR}/Android.mk")
Michael Gronager, PhD
Director, Ceptacle
Jens Juels Gade 33
2100 Copenhagen E
Mobile: +45 31 45 14 01
E-mail: gronager@ceptacle•com
Web: http://www.ceptacle.com/
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [Bitcoin-development] libcoin (HEAD) now supports boost < 1.47 - please test
2012-02-02 22:43 ` Michael Grønager
@ 2012-02-02 23:27 ` Luke-Jr
0 siblings, 0 replies; 4+ messages in thread
From: Luke-Jr @ 2012-02-02 23:27 UTC (permalink / raw)
To: Michael Grønager; +Cc: bitcoin-development
On Thursday, February 02, 2012 5:43:07 PM Michael Grønager wrote:
> Enabling dynamic libs was on my TODO, but on the
> Redmond_OS_not_to_be_mentioned you need to : * prepend class definitions
> with __declspec(dllexport) when you compile the dll * prepend class
> definitions with __declspec(dllimport) when you use the dll I just love
> the way they spoil their developers...
I hadn't even thought of this. Sounds like a pain :/
The problem I had was related to *using* static libraries; ie, boost.
I have libboost*.so, but libboost*.a
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2012-02-02 23:27 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-02-02 13:46 [Bitcoin-development] libcoin (HEAD) now supports boost < 1.47 - please test Michael Grønager
2012-02-02 16:30 ` Luke-Jr
2012-02-02 22:43 ` Michael Grønager
2012-02-02 23:27 ` Luke-Jr
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox