PaulStoffregen / Time

Time library for Arduino
http://playground.arduino.cc/code/time
1.26k stars 670 forks source link

Time lib incompatible with nRF52's adafruitadafruitnrf52 framework #149

Closed orrmany closed 4 years ago

orrmany commented 4 years ago

Description

DateStrings.cpp fails to compile

Steps To Reproduce Problem

Try to compile for for any nrF52 board

Hardware & Software

Board = any nRF52 board Shields / modules used: not relevant, since this is compile time error Arduino IDE version: n/a (using PlatformIO) Teensyduino version (if using Teensy): n/a Version info & package name (from Tools > Boards > Board Manager) not relevant, this is pure C++ problem Operating system & version: not relevant Any other software or hardware? not relavant

Arduino Sketch

// Change the code below by your sketch (please try to give the smallest code which demonstrates the problem)
#include <TimeLib.h>    // https://github.com/PaulStoffregen/Time

void setup() {
}

void loop() {
}

Errors or Incorrect Output

Executing task in folder SD-card-PoC: C:\Users\egbozie.platformio\penv\Scripts\platformio.exe run <

Processing adafruit_feather_nrf52840_s340 (platform: https://github.com/orrmany/platform-nordicnrf52.git#develop-s340; board: adafruit_feather_nrf52840_s340; framework: arduino)

Verbose mode can be enabled via -v, --verbose option CONFIGURATION: https://docs.platformio.org/page/boards/nordicnrf52/adafruit_feather_nrf52840_s340.html PLATFORM: Nordic nRF52 4.2.0 #3db8a48 > Adafruit Feather nRF52840 Express s340 HARDWARE: NRF52840 64MHz, 243KB RAM, 796KB Flash DEBUG: Current (jlink) External (jlink, stlink) PACKAGES:

orrmany commented 4 years ago

Sorry, it rather seems to be a framework problem

schmidie commented 4 years ago

@orrmany, did you find a way to get the TimeLib working on nRf52 ?

orrmany commented 4 years ago

Yes, by fixing the framework. See https://github.com/adafruit/Adafruit_nRF52_Arduino/issues/502

orrmany commented 4 years ago

See also: https://github.com/sandeepmistry/arduino-nRF5/pull/418

schmidie commented 4 years ago

:+1: cool, thanks