Skip to content

Fix building on FreeBSD#59

Merged
n1tehawk merged 1 commit intolinux-sunxi:masterfrom
evadot:master
Oct 5, 2016
Merged

Fix building on FreeBSD#59
n1tehawk merged 1 commit intolinux-sunxi:masterfrom
evadot:master

Conversation

@evadot
Copy link
Copy Markdown
Contributor

@evadot evadot commented Oct 1, 2016

Hello,

This patch fix building on FreeBSD.

@n1tehawk n1tehawk merged commit fc30490 into linux-sunxi:master Oct 5, 2016
n1tehawk added a commit to n1tehawk/sunxi-tools that referenced this pull request Nov 22, 2016
See pull requests linux-sunxi#59 and linux-sunxi#83.

It looks like various BSD derivates have adopted the more common naming
conventions for the endian conversion macros, which means that we would
attempt to redefine them without need (potentially even to 'historic'
names that no longer exist).

Try to avoid this by properly checking for existing functions first.

Signed-off-by: Bernhard Nortmann <bernhard.nortmann@web.de>
@n1tehawk
Copy link
Copy Markdown
Collaborator

This has resurfaced with Dragonfly BSD (see #83). @evadot could you check if the above patch with an alternative solution would work for you, too?

Regards, NiteHawk

n1tehawk added a commit to n1tehawk/sunxi-tools that referenced this pull request Nov 28, 2016
See pull requests linux-sunxi#59 and linux-sunxi#83.

It looks like various BSD derivates have adopted the more common naming
conventions for the endian conversion macros, which means that we would
attempt to redefine them without need (potentially even to 'historic'
names that no longer exist).

Try to avoid this by properly checking for existing functions first.

Signed-off-by: Bernhard Nortmann <bernhard.nortmann@web.de>
n1tehawk added a commit that referenced this pull request Nov 29, 2016
See pull requests #59 and #83.

It looks like various BSD derivates have adopted the more common naming
conventions for the endian conversion macros, which means that we would
attempt to redefine them without need (potentially even to 'historic'
names that no longer exist).

Try to avoid this by properly checking for existing functions first.

Signed-off-by: Bernhard Nortmann <bernhard.nortmann@web.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants