lf-lang / lingo

Package manager and build tool for the Lingua Franca coordination language
BSD 2-Clause "Simplified" License
4 stars 1 forks source link

lf-pico-template updates #32

Closed gundralaa closed 1 year ago

gundralaa commented 1 year ago

Added minimal changes to allow for RP2040 target.

note: i've addressed comments in https://github.com/lf-lang/lingo/pull/39. please merge that into this branch first.

lhstrh commented 1 year ago

This looks good! Is it ready for review?

tanneberger commented 1 year ago

@gundralaa Can u finish this up ?

lhstrh commented 1 year ago

@gundralaa, there is a conflict with what's in main. Could you take a quick look? Thanks!

gundralaa commented 1 year ago

https://github.com/lf-lang/lingo/pull/41 fixed the conflict here

tanneberger commented 1 year ago

@gundralaa Did u test your changes can u produce binaries that can run on the pico ?

tanneberger commented 1 year ago

From the code side of things this looks very good!

gundralaa commented 1 year ago

@gundralaa Did u test your changes can u produce binaries that can run on the pico ?

Yes, it works with the current Lingo.toml in lf-pico-template. Binaries are generated in the target directory. https://github.com/lf-lang/lf-pico-template/blob/main/Lingo.toml