DragonFly BSD patches for bitcoind

The following content was written by Venkatesh Srinivas on July 12, 2011, 12:44:13 AM in the thread DragonFly BSD patches for bitcoind. All content is owned by the author of the bitcointalk.org post. (original)

---

Hi,

I have a small series of patches that allow bitcoind to build on DragonFly BSD; what would be an acceptable way to submit these patches upstream? It looks like I should make a github fork of the bitcoin/bitcoin repository, push the patches on the master branch of the clone, and submit a pull request.

Some patches are specific to DragonFly BSD; others correct minor problems that are more generic; should I submit them as one patch with one pull request? Multiple patches with one pull request? Multiple patches with multiple pull requests?

Thanks,
— vs

---

The following content was written by ghotir on July 14, 2011, 09:37:36 AM in the thread DragonFly BSD patches for bitcoind. All content is owned by the author of the bitcointalk.org post. (original)

---

I’d be interested in those patches for my DragonFlyBSD install.

---

The following content was written by Vladimir on July 14, 2011, 09:42:23 AM in the thread DragonFly BSD patches for bitcoind. All content is owned by the author of the bitcointalk.org post. (original)

---

you could simply make/change freebsd port

---

The following content was written by ghotir on July 14, 2011, 01:02:37 PM in the thread DragonFly BSD patches for bitcoind. All content is owned by the author of the bitcointalk.org post. (original)

---

DragonFlyBSD uses the NetBSD pkgsrc tree, so if the OP has already done some ground work, why reinvent the wheel?
On that note- OP should contact the pkgsrc-wip project about getting it submitted: https://lists.sourceforge.net/lists/listinfo/pkgsrc-wip-discuss

---

The following content was written by marino on July 14, 2011, 02:56:36 PM in the thread DragonFly BSD patches for bitcoind. All content is owned by the author of the bitcointalk.org post. (original)

---

Vladimir,

I think vsrinivas is trying update the code base such that the change only has to be made once.  Your suggestion is to add the patches to the pkgsrc package (rather than the FreeBSD port) and leave the code base alone, and risk that the patches eventually stop working. 

If it’s not possible to fix the code base, fine, your suggestion would work as a last resort, but it’s better to attempt to fix it properly.

---

The following content was written by Venkatesh Srinivas on July 14, 2011, 05:25:33 PM in the thread DragonFly BSD patches for bitcoind. All content is owned by the author of the bitcointalk.org post. (original)

---

Some of the patches are more general than just DragonFly support:

1) The bitcoind make target in makefile.unix defines USE_UPNP to 0; however net.cpp among others uses ‘#ifdef’ as the test for miniupnp, which means the upnp headers are always required.

2) main.cpp has:
-char pchMessageStart[4] = { 0xf9, 0xbe, 0xb4, 0xd9 };

g++ 4.4.5 on DragonFly disapproves; the constants are being narrowed from int -> char inside an array initializer. I think the error is correct, though harsh.

3) g++ on DragonFly wants -std=c++0x or -std=gnu++0x; I agree with it — typed enumerations are used in the source. I don’t know why the same g++ version on Linux doesn’t require it, though.

4) in db.cpp, make_tuple is meant to resolve to boost::make_tuple; at least in our configuration, it resolves to std::make_tuple, which is not correct. I don’t know enough to say what correct behavior is when a template is defined in more than one namespace (std and boost) and both are ‘using namespace ‘-ed. But qualifying it helps.

DragonFly specific stuff:

1) Currently net.cpp, there is an assumption that all BSD systems support SO_NOSIGPIPE; DragonFly doesn’t. The test should probably be for SO_NOSIGPIPE.

---

The following content was written by ghotir on July 20, 2011, 11:31:09 AM in the thread DragonFly BSD patches for bitcoind. All content is owned by the author of the bitcointalk.org post. (original)

---

I knew about the SO_NOSIGPIPE issue ( supported in FreeBSD >5 only. Being that DragonFly is based on the 4.x series i can understand why it’s not; but also read somewhere that Open and Net  don’t either )
The db.cpp boost issue has  been driving me absolutely nuts (usually I can figure such things out with a little poking around make files, etc.).
Anyway, if it’s working for you – compiled, all that, I’d be willing to be a 2nd to apply the patches and build on my dragonflybsd system. 

---

The following content was written by Gavin Andresen on August 07, 2011, 03:08:34 PM in the thread DragonFly BSD patches for bitcoind. All content is owned by the author of the bitcointalk.org post. (original)

---

Two pull requests seem appropriate: one for the generic issues (talk with TheBlueMatt about the upnp #define, I believe it is working as designed), and one for DragonFly-specific stuff.

Frankly, DragonFlyBSD-specific stuff is unlikely to get pulled; there just aren’t enough DragonFly-BSD systems to justify the work of maintaining support for it (according to bsdstats.org it isn’t a very popular BSD variant).

---

The following content was written by JoelKatz on August 07, 2011, 03:36:46 PM in the thread DragonFly BSD patches for bitcoind. All content is owned by the author of the bitcointalk.org post. (original)

---

I believe this is intentional. USE_UPNP=0 means that UPNP is not used by default but can be enabled. It’s a three-way switch.

Really? That’s kind of a nutty error. Are you sure you aren’t using non-standard warning flags? GCC will issue crazy warnings if you enable all possible warnings, even “unsigned char j[10], k[10]; j[ i ]^=k[ i ];” will trigger an int->char warning.

---

The following content was written by wumpus on August 07, 2011, 03:47:12 PM in the thread DragonFly BSD patches for bitcoind. All content is owned by the author of the bitcointalk.org post. (original)

---

AFAIK the problem is not the narrowing from int to char (which is perfectly allowed if it fits). The problem is that plain “char” has undefined signedness in C (signed in most compilers these days). So values larger than >0x80 throw an overflow warning. The correct thing would be to use “unsigned char” or uint8_t.

---

The following content was written by Venkatesh Srinivas on August 07, 2011, 04:06:25 PM in the thread DragonFly BSD patches for bitcoind. All content is owned by the author of the bitcointalk.org post. (original)

---

>> char pchMessageStart[4] = { 0xf9, 0xbe, 0xb4, 0xd9 };

>> g++ 4.4.5 on DragonFly disapproves; the constants are being narrowed from int -> char inside an array initializer.

>AFAIK the problem is not the narrowing from int to char (which is perfectly allowed if it fits). The problem is that plain “char” has undefined signedness in C (signed in most compilers these days). So values larger than >0x80 throw an overflow warning. The correct thing would be to use “unsigned char” or uint8_t.

You’re right; I misunderstood the error message:

main.cpp:1773: error: narrowing conversion of ‘249’ from ‘int’ to ‘char’ inside { }

unsigned char would be correct.

It also turns out that the SO_NOSIGPIPE issue affects NetBSD and OpenBSD, both of which don’t have SO_NOSIGPIPE, but do define BSD. So none of the issues are DragonFly specific.

---

The following content was written by Venkatesh Srinivas on August 11, 2011, 05:20:04 AM in the thread DragonFly BSD patches for bitcoind. All content is owned by the author of the bitcointalk.org post. (original)

---

Okay, looks like all of the patches have been merged into bitcoind!

Thanks,

---

The following content was written by Venkatesh Srinivas on August 11, 2011, 04:23:11 PM in the thread DragonFly BSD patches for bitcoind. All content is owned by the author of the bitcointalk.org post. (original)

---

To build bitcoind on DragonFly BSD (or NetBSD), you can now grab a copy of the git tree for bitcoin. From pkgsrc, you’ll need to install Berkeley DB (db4); I used version 4.8, but 4.6 (also in pkgsrc, as db46) should work. You’ll also need boost; I installed: boost-1.46.1, boost-libs-1.46.1, and boost-headers-1.46.

You’ll need to make a few changes to makefile.unix before building:

To LIBS, I added -L/usr/pkg/lib ; replace with wherever your pkgsrc libraries are located. In LIBS, I also replaced -ldb_cxx with -ldb4_cxx.
To DEFS, I added -std=gnu++0x and -I/usr/pkg/include and -I/usr/pkg/include/db4 ; these changes don’t really belong in DEFS, but its an easy place to add them.

After that, use GNU make with makefile.unix; bitcoind works and is able to download the blockchain and make transactions. This has been tested on DragonFly 2.10 / i386; the same steps should apply to NetBSD, but I haven’t had a chance to try it.

Thanks!

---

The following content was written by phma on August 19, 2011, 12:37:23 AM in the thread DragonFly BSD patches for bitcoind. All content is owned by the author of the bitcointalk.org post. (original)

---

I’m getting an error; there are two files called serialize.h, and apparently it’s getting the wrong one. The error is:
There are /usr/include/sys/serialize.h and bitcoin-0.3.24/src/src/serialize.h. I’m using DragonFly 2.11. I modified makefile.unix as follows:
There’s also a db5 package. I tried it and got an error, switched to db46, and still got an error.

---

The following content was written by Venkatesh Srinivas on August 19, 2011, 06:06:55 AM in the thread DragonFly BSD patches for bitcoind. All content is owned by the author of the bitcointalk.org post. (original)

---

Let me just post my Makefile here:

---

The following content was written by phma on August 26, 2011, 03:58:39 AM in the thread DragonFly BSD patches for bitcoind. All content is owned by the author of the bitcointalk.org post. (original)

---

I had to remove crypter from the Makefile (there’s no such file in the source) and change “db4” to “db46”. I still get an error relating to serialize:

---

The following content was written by Venkatesh Srinivas on August 26, 2011, 04:55:13 AM in the thread DragonFly BSD patches for bitcoind. All content is owned by the author of the bitcointalk.org post. (original)

---

Which version of bitcoin are you using? crypter is present in git master.

Also, what Berkeley DB package do you have? /usr/pkg/include/db4 was provided by db4-4.8.30 from pkgsrc.

---

The following content was written by phma on August 26, 2011, 08:16:47 PM in the thread DragonFly BSD patches for bitcoind. All content is owned by the author of the bitcointalk.org post. (original)

---

db46-4.6.21
bitcoin-0.3.24
From the build-unix file:
You need Berkeley DB 4.7.  Don’t use 4.8, the database/log0000* files
are incompatible.

How hard would it be to make the build process use cmake or the like so that this sort of mess wouldn’t happen?

---

The following content was written by Venkatesh Srinivas on August 26, 2011, 09:05:18 PM in the thread DragonFly BSD patches for bitcoind. All content is owned by the author of the bitcointalk.org post. (original)

---

Incompatible with what? If you download the blockchain from within the client and don’t need to share the db with precompiled clients, 4.8 works fine.

---

The following content was written by phma on August 29, 2011, 03:23:16 PM in the thread DragonFly BSD patches for bitcoind. All content is owned by the author of the bitcointalk.org post. (original)

---

I don’t know what it’s incompatible with. I quoted the file.

I installed db 4.8 and changed the “46” in the makefile to “4”. I’m still getting serialize errors. There are two it could be getting:
/usr/include/sys/serialize.h
/usr/local/src/bitcoin-0.3.24/src/src/serialize.h

---