Bug 471916 – net-p2p/bitcoind-0.8.2 version bump

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Matt Whitlock 2013-05-31 16:15:53 UTC]

Created attachment 349742 [details]
net-p2p/bitcoind/bitcoind-0.8.2.ebuild

Attached is a new ebuild for net-p2p/bitcoind-0.8.2, which is almost identical to the ebuild for net-p2p/bitcoind-0.8.1 except that there's a new version of the patch to use the system LevelDB library.

[Comment 1 Matt Whitlock 2013-05-31 16:16:40 UTC]

Created attachment 349744 [details, diff]
net-p2p/bitcoind/files/0.8.2-sys_leveldb.patch

And here's the new version of the patch.

[Comment 2 Matt Whitlock 2013-05-31 16:22:19 UTC]

Probably the ebuild should be keyworded as unstable. Sorry I didn't notice that.

[Comment 3 Jeroen Roovers (RETIRED) 2013-05-31 16:25:24 UTC]

Comment on attachment 349742 [details]
net-p2p/bitcoind/bitcoind-0.8.2.ebuild

--- bitcoind-0.8.1.ebuild       2013-05-14 15:42:39.948907984 +0200
+++ -   2013-05-31 18:25:17.797914474 +0200
@@ -47,7 +47,7 @@
 }
 
 src_prepare() {
-       epatch "${FILESDIR}/0.8.0-sys_leveldb.patch"
+       epatch "${FILESDIR}/0.8.2-sys_leveldb.patch"
        rm -r src/leveldb
 }

[Comment 4 Matt Whitlock 2013-05-31 19:22:17 UTC]

Also this:

--- bitcoind-0.8.2.ebuild~	2013-05-31 16:10:20.000000000 +0000
+++ bitcoind-0.8.2.ebuild	2013-05-31 16:57:35.921611356 +0000
@@ -97,7 +97,7 @@
 	fowners bitcoin:bitcoin /var/lib/bitcoin/.bitcoin
 	dosym /etc/bitcoin/bitcoin.conf /var/lib/bitcoin/.bitcoin/bitcoin.conf
 
-	dodoc doc/README
+	dodoc doc/README.md
 
 	if use examples; then
 		docinto examples

[Comment 5 Matt Whitlock 2013-06-05 07:31:39 UTC]

Created attachment 350174 [details, diff]
net-p2p/bitcoind/files/0.8.2-sys_leveldb.patch

This is a new version of the patch that adds boost_chrono to the list of imported libraries when linking the final executable. This was needed on one of my systems (with Boost 1.52) to avoid this linkage error:

/usr/include/boost/thread/pthread/condition_variable_fwd.hpp:177: error: undefined reference to 'boost::chrono::system_clock::now()'

[Comment 6 Luke-Jr 2013-06-05 07:54:25 UTC]

(In reply to Matt Whitlock from comment #5)
> Created attachment 350174 [details, diff] [details, diff]
> net-p2p/bitcoind/files/0.8.2-sys_leveldb.patch
> 
> This is a new version of the patch that adds boost_chrono to the list of
> imported libraries when linking the final executable. This was needed on one
> of my systems (with Boost 1.52) to avoid this linkage error:
> 
> /usr/include/boost/thread/pthread/condition_variable_fwd.hpp:177: error:
> undefined reference to 'boost::chrono::system_clock::now()'

This is unrelated to leveldb, so doesn't really belong in the same patch file.

It also breaks building with older (stable) boost versions.

[Comment 7 Matt Whitlock 2013-06-05 09:51:46 UTC]

--- bitcoind-0.8.1.ebuild	2013-06-01 15:31:26.000000000 +0000
+++ bitcoind-0.8.2.ebuild	2013-06-05 09:30:25.099274226 +0000
@@ -47,8 +47,12 @@
 }
 
 src_prepare() {
-	epatch "${FILESDIR}/0.8.0-sys_leveldb.patch"
+	epatch "${FILESDIR}/0.8.2-sys_leveldb.patch"
 	rm -r src/leveldb
+
+	if has_version '>=dev-libs/boost-1.52' ; then
+		sed -e '/-l db_cxx/i \   -l boost_chrono$(BOOST_LIB_SUFFIX) \\' -i src/makefile.unix
+	fi
 }
 
 src_compile() {
@@ -97,7 +101,7 @@
 	fowners bitcoin:bitcoin /var/lib/bitcoin/.bitcoin
 	dosym /etc/bitcoin/bitcoin.conf /var/lib/bitcoin/.bitcoin/bitcoin.conf
 
-	dodoc doc/README
+	dodoc doc/README.md
 
 	if use examples; then
 		docinto examples