Closed proffalken closed 5 years ago
Hmm, I'm afraid we introduced it in #241.
Can you confirm it works with avr/pgmspace.h
?
Yup, hard-coding it to that value works fine.
OK that's regression from #243.
Can you file a PR to fix this? We need a compiler directive again, but this time taking into account this platform.
Sure, I'll get to it ASAP.
OK, I'm now thoroughly confused - I went to fork/patch/PR and found out that the logic was already there. I then check the Arduino libraries manager and saw that there was an update, and on updating it pulled in the code and compiled successfully.
It looks like the relevant commit is https://github.com/TheThingsNetwork/arduino-device-lib/commit/b967fc4a773873bed344ac89994de31242a744c3 but I've no idea why my libraries weren't showing as out of date in the IDE until now.
Anyway, it's working with the latest libraries in the IDE, so I'll "Close - Won't Fix" this now, and try not to look too embarrassed at the conference next week! :joy:
Happy that it's solved, no problem at all! See you next week.
I'm trying to compile the basic SODAQ example from the LoRaWAN Academy, but I'm hitting an issue when trying to compile.
The code is simple:
However the error I'm seeing is:
I thought it might be related to #241 , however the patch appears to be present in the header file, so I'm not sure what's causing this!