bsiever / microbit-pxt-timeanddate

Software Based Real Time Clock (Time & Date) for the Micro:Bit
https://bsiever.github.io/microbit-pxt-timeanddate/
MIT License
4 stars 1 forks source link

Compatibility with micro:bit v2 #11

Closed microbit-sam closed 3 years ago

microbit-sam commented 3 years ago

Hi @bsiever ,

As you might have seen from microsoft/pxt-microbit#3506 this extension is currently not compiling against the latest version of Micro:bit CODAL. This means it will not work with micro:bit V2.

Software for micro:bit V2 boards is built using CODAL rather than the DAL, and no longer includes the mbed layer. We’ve implemented the most frequently used mbed classes to support easy porting of existing extensions, but there may still be corner cases with this compatibility layer.

As your extension is currently not building, it will be disabled for micro:bit V2. People attempting to use your extension with a micro:bit V2 board will get error 929. This extension will only be re-enabled on micro:bit V2 once it is fixed.

There are details on how to ensure extensions work on both version of the micro:bit here (lancaster-university/codal-microbit-v2#31).

Looking at your extension source I believe that your extension falls under this category:

  1. Use of mbed APIs that are not included in the compatibility layer, or microbit-dal APIs that need different constructors
/home/build/prjclone/pxtapp/microbit-pxt-timeanddate/timeanddate.cpp:61:28: error: 'us_ticker_read' was not declared in this scope
bsiever commented 3 years ago

Thanks @microbit-sam --- I believe my current fix/release will take care of the problem (I hope). Could you please confirm and, if resolved, close the issue here? (And if there are still problems could you post a follow up with some guidance?)

Thanks!

bsiever commented 3 years ago

I believe this is good-to-go (seems to compile in Beta anyway)