search

ARMmbed / ble-nrf51822

Nordic stack and drivers for the mbed BLE_API
Other
46 stars 51 forks source link

Update S110 detection macros, again #49

Closed jpbrucker closed 8 years ago

jpbrucker commented 8 years ago

The mbed SDK actually prefixes all labels from targets.py with "TARGET_". Update our detection macros accordingly.

Hi Rohit,

This is embarrassing. The mbed SDK still doesn't select the macros that allow to remove Central features, when building with a S110 SoftDevice. I noticed this while investigating another issue in the live IDE. I now have a simple test case to check that the SoftDevice in use matches the build options. I built it locally with workspace_tools/singletest.py and this patch, to ensure that my two patches will have the right behaviour once integrated into the live IDE. Hopefully, this should do the trick.

miklis commented 8 years ago

Automatic CI verification build not done, please verify manually.