Skip to content
New issue

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

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Define feature macros only on Linux #1

Merged
merged 1 commit into from
Oct 5, 2018
Merged

Conversation

liweitianux
Copy link

This patch makes it working on BSD (tested on DragonFly BSD, should be
the same on FreeBSD). Fix issue: nsf#110

On Linux, such features macros (_POSIX_C_SOURCE, _XOPEN_SOURCE) are
required to expose more functions. However, on *BSD, almost everything
is exposed when no feature macros are defined, while defining feature
macros would restrict the functions to be exposed.

See more information at:
https://lists.freebsd.org/pipermail/freebsd-standards/2004-March/000474.html

This patch makes it working on BSD (tested on DragonFly BSD, should be
the same on FreeBSD).  Fix issue: nsf#110

On Linux, such features macros (_POSIX_C_SOURCE, _XOPEN_SOURCE) are
required to expose more functions.  However, on *BSD, almost everything
is exposed when no feature macros are defined, while defining feature
macros would restrict the functions to be exposed.

See more information at:
https://lists.freebsd.org/pipermail/freebsd-standards/2004-March/000474.html
liweitianux added a commit to liweitianux/ly-dm that referenced this pull request Oct 5, 2018
In preparation for the BSD support.

See also my relevant pull request to the termbox-next at:
nullgemm/termbox_next#1
@nullgemm nullgemm merged commit d356892 into nullgemm:master Oct 5, 2018
@nullgemm
Copy link
Owner

nullgemm commented Oct 5, 2018

Thank you for this :)

nullgemm pushed a commit to fairyglade/ly that referenced this pull request Oct 5, 2018
nullgemm pushed a commit that referenced this pull request Jul 27, 2020
Define feature macros only on Linux
nullgemm pushed a commit that referenced this pull request Jul 27, 2020
Define feature macros only on Linux
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