NordicSemiconductor / ble-sdk-arduino

MIT License
249 stars 97 forks source link

patch to prevent compilation error, examples now compile using arduino 1... #4

Closed samthetechie closed 10 years ago

samthetechie commented 10 years ago

...:1.0.5+dfsg2-2 (which is the version in the ubuntu and debian repositories).

Note: this bug has already been documented in various places:

find . -name ".ino" -print | xargs subl find . -name ".cpp" -print | xargs subl

find: static hal_aci_data_t setup_msgs[NB_SETUP_MESSAGES] PROGMEM

replace: static const hal_aci_data_t setup_msgs[NB_SETUP_MESSAGES] PROGMEM

find: aci_state.aci_setup_info.setup_msgs = setup_msgs;

replace: aci_state.aci_setup_info.setup_msgs = (hal_aci_data_t*)setup_msgs;

Skjerve commented 10 years ago

We'll try to pull this into our next release.

samthetechie commented 10 years ago

Thanks for the update. Note: I made a similar code modification and pull request to a library from RedBearLab https://github.com/RedBearLab/nRF8001/pull/2#issuecomment-45693268 and they replied "Thanks, but this doesn't work on Mac OSX with Arduino IDE 1.0.5" I do not have an apple device to test on myself.

Skjerve commented 10 years ago

Just tested this on OS X, unfortunately it causes conflicts. Will add this to known issues.

samthetechie commented 10 years ago

On 15/07/14 09:58, Skjerve wrote:

Just tested this on OS X, unfortunately it causes conflicts. Will add this to known issues.


Reply to this email directly or view it on GitHub: https://github.com/NordicSemiconductor/ble-sdk-arduino/pull/4#issuecomment-49001404

Thanks for the feedback!

superchinois commented 9 years ago

thank you. It worked for me I use os x, arduino ide 1.5.8, yun, ble 2.1 and nRF8001 20140701