Skip to content

Conversation

@ghisvail
Copy link
Contributor

According to the current build log, <sys/sysctl.h> get included when compiled on the gnu-hurd architecture although it should not be. This commit adds the necessary guard to prevent this.

@pavanky
Copy link
Member

pavanky commented Mar 28, 2016

I wonder where BSD is coming from..

@ghisvail
Copy link
Contributor Author

According to the list of compiler macros maintained here, it should probably be:

#if defined(__FreeBSD_kernel__)

instead of BSD (which is not considered an architecture but a version identifier), and would include both the FreeBSD kernel and GNU/kFreeBSD kernels. Perhaps this should be enough to exclude GNU/Hurd actually.

@pavanky pavanky added this to the 3.3.2 milestone Mar 29, 2016
@pavanky pavanky merged commit 8f8e54f into arrayfire:hotfix-3.3.2 Mar 29, 2016
@ghisvail ghisvail deleted the fix/gnu-hurd-support branch August 17, 2016 06:50
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