-
Notifications
You must be signed in to change notification settings - Fork 2.1k
List of compiling issues on OS X #6473
Copy link
Copy link
Closed
Labels
OS: Mac OS XHost OS: This PR/issue concerns usage of RIOT with Mac OS X as a host systemHost OS: This PR/issue concerns usage of RIOT with Mac OS X as a host systemType: bugThe issue reports a bug / The PR fixes a bug (including spelling errors)The issue reports a bug / The PR fixes a bug (including spelling errors)
Description
I just ran
for i in $(ls); do if [[ -d "$i" ]]; then echo $i; make -C $i; fi; done;on the RIOT tests folder to know which things are free of error on OS X, and that gave results which can be useful
- tests_can fixed in sys/can: fix building errors on native OS X #7847.
/Users/facosta/git/RIOT-OS/RIOT/sys/can/conn/isotp.c:67:18: error: comparison of integers of different signs: 'int' and 'unsigned int'
[-Werror,-Wsign-compare]
/Users/facosta/git/RIOT-OS/RIOT/sys/shell/commands/sc_can.c:41:23: error: comparison of integers of different signs: 'int' and
'unsigned int' [-Werror,-Wsign-compare]
for (int i = 0; i < CAN_DLL_NUMOF; i++) {
~ ^ ~~~~~~~~~~~~~
/Users/facosta/git/RIOT-OS/RIOT/sys/shell/commands/sc_can.c:64:15: error: comparison of integers of different signs: 'int' and
'unsigned int' [-Werror,-Wsign-compare]
if (ifnum >= CAN_DLL_NUMOF) {
~~~~~ ^ ~~~~~~~~~~~~~
/Users/facosta/git/RIOT-OS/RIOT/sys/shell/commands/sc_can.c:94:15: error: comparison of integers of different signs: 'int' and
'unsigned int' [-Werror,-Wsign-compare]
assert(ifnum < CAN_DLL_NUMOF);
~~~~~ ^ ~~~~~~~~~~~~~
if (ifnum >= CAN_DLL_NUMOF) {
~~~~~ ^ ~~~~~~~~~~~~~
/Users/facosta/git/RIOT-OS/RIOT/core/include/assert.h:105:11: note: expanded from macro 'assert'
if (!(cond)) { \
^~~~
1 error generated.
3 errors generated.- driver_hd44780 Errors due to the spefici setting for Arduino. I propose to white list the boards which support Arduino pin mapping.
- gnrc_ipv6_nib Found a pontential corner case @miri64 :
/Users/facosta/git/RIOT-OS/RIOT/sys/net/gnrc/network_layer/ipv6/nib/_nib-internal.c:269:20: error: unused function '_get_l2addr_from_ipv6'
[-Werror,-Wunused-function]
static inline void _get_l2addr_from_ipv6(uint8_t *l2addr,
^
1 error generated.- gnrc_ipv6_nib_6ln Unused function error (another potential corner case):
/Users/facosta/git/RIOT-OS/RIOT/sys/net/gnrc/network_layer/ipv6/nib/nib.c:543:20: error: unused function '_is_reachable'
[-Werror,-Wunused-function]
static inline bool _is_reachable(_nib_onl_entry_t *entry)
^
1 error generated.- ssp buffer overflow detected. Fixed in tests/ssp: Fix on macOS while compiling w/ clang #7777
/Users/facosta/git/RIOT-OS/RIOT/tests/ssp/main.c:33:5: error: '__builtin___memset_chk' will always overflow destination buffer
[-Werror,-Wbuiltin-memcpy-chk-size]
memset(buf, 0, 32);
^~~~~~~~~~~~~~~~~~
/usr/include/secure/_string.h:77:3: note: expanded from macro 'memset'
__builtin___memset_chk (dest, val, len, __darwin_obsz0 (dest))
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1 error generated.- unittests Works! Fixed for release 2017.10
- saul Why it works? Does native provide support for SAUL? It works according to comment
Reactions are currently unavailable
Metadata
Metadata
Labels
OS: Mac OS XHost OS: This PR/issue concerns usage of RIOT with Mac OS X as a host systemHost OS: This PR/issue concerns usage of RIOT with Mac OS X as a host systemType: bugThe issue reports a bug / The PR fixes a bug (including spelling errors)The issue reports a bug / The PR fixes a bug (including spelling errors)