Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Support for NetBSD, DragonFlyBSD and OpenBSD #5

Merged
merged 1 commit into from Nov 9, 2011

Conversation

Projects
None yet
3 participants
Contributor

Vagabond commented Apr 8, 2011

There were 3 things that were fixed in this patch:

The endian detection header didn't contain all the possible machine
architecture definitions for AMD64, see
http://predef.sourceforge.net/prearch.html#sec3

DragonFlyBSD wasn't checked for as an OS along with the other BSDs

OpenBSD was exposing BSD symbols not defined by POSIX so rebar.config
now passes -D_POSIX_C_SOURCE to ensure these are hidden.

The fix for OpenBSD is wrong, please see pull request #6.

Contributor

Vagabond commented Apr 13, 2011

Yes, you're correct, that is the right way to fix it.

Contributor

Vagabond commented Jul 1, 2011

Updated commit to fix things the right way. Tested working on all the BSDs.

Contributor

Vagabond commented Oct 5, 2011

Rebased on top of master.

Contributor

dizzyd commented Nov 9, 2011

+1

dizzyd added a commit that referenced this pull request Nov 9, 2011

Merge pull request #5 from basho/adt-bsd-support
Support for NetBSD, DragonFlyBSD and OpenBSD

@dizzyd dizzyd merged commit cd531f8 into master Nov 9, 2011

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment