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

unittests: native: fixes for OS X #3768

Merged
merged 3 commits into from
Oct 1, 2015

Conversation

thomaseichinger
Copy link
Member

Several fixes to get unittests compile on OS X

@@ -38,7 +38,7 @@

#if defined(MODULE_MSP430_COMMON)
# include "msp430_types.h"
#elif !defined(CPU_NATIVE)
#elif !defined(__linux__)
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Kijewski are you ok with this?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure.

@OlegHahm OlegHahm added Area: tests Area: tests and testing framework OS: Mac OS X Host OS: This PR/issue concerns usage of RIOT with Mac OS X as a host system labels Sep 3, 2015
@thomaseichinger
Copy link
Member Author

Fixes #2657.

@thomaseichinger thomaseichinger added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Sep 30, 2015
OS X clang complains "folded to constant array as an extension"
[-Wgnu-folding-constant]
Header <sys/types.h> needs to be included for OS X to use size_t.
@OlegHahm
Copy link
Member

OlegHahm commented Oct 1, 2015

ACK and go!

OlegHahm added a commit that referenced this pull request Oct 1, 2015
@OlegHahm OlegHahm merged commit 890d101 into RIOT-OS:master Oct 1, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: tests Area: tests and testing framework CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR OS: Mac OS X Host OS: This PR/issue concerns usage of RIOT with Mac OS X as a host system
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants