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

FreeBSD fixes #1232

Merged
merged 1 commit into from
Jun 6, 2017
Merged

FreeBSD fixes #1232

merged 1 commit into from
Jun 6, 2017

Conversation

johalun
Copy link
Contributor

@johalun johalun commented Jun 5, 2017

Increase _POSIX_SOURCE value for localtime_r access.
Increase _XOPEN_SOURCE value for lstat() and various printf() functions access.
Conditionally link to libcap (only on Linux).
Possibly some trailing whitespace fixes (automatic).

@ddevault
Copy link
Contributor

ddevault commented Jun 5, 2017

Please don't increase _XOPEN_SOURCE globally, but rather only in files where the newer functions are required.

@johalun
Copy link
Contributor Author

johalun commented Jun 6, 2017

After fixing 6-7 files manually I gave up and replace everywhere. I'll give it another shot.

Increase _POSIX_SOURCE value where needed.
Increase _XOPEN_SOURCE value where needed.
Conditionally link to libcap (only on Linux).
Possibly some trailing whitespace fixes (automatic).
@johalun
Copy link
Contributor Author

johalun commented Jun 6, 2017

Fixed.

@ddevault ddevault merged commit fd47a30 into swaywm:master Jun 6, 2017
@ddevault
Copy link
Contributor

ddevault commented Jun 6, 2017

Thanks!

@ddevault ddevault mentioned this pull request Jun 15, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants