reitermarkus / esp32-hello

18 stars 6 forks source link

Joining efforts #2

Open ivmarkov opened 3 years ago

ivmarkov commented 3 years ago

Hi there. I'm also working on running Rust STD on top of ESP-IDF:

The thing is, when I did my work, over the course of a couple of weekends - I managed to miss the one pull request against the MabezDev Rust xtensa repo - yours. :)

Originally, I was a bit put off by the size of your changes until I realised that your patch-set is relatively small - just like mine. In fact, yours and my work are very much identical:

You have target_os = "freertos", mine is target_os = "none" and you do use some symbols from FreeRTOS, while I've managed to avoid that so far and sticked with the pthread API completely, but I've done 0 testing on mutexes and condvars so I might have missed something. For one, there is a bug (or misinterpretation) of the pthread TLS destructor API in the ESP-IDF pthread support that I hope they might fix if kindly asked - or else thread::join crashes when trying to free its TLS storage.

The bottom line is the following: I'll try to review/merge your changes into mine anyway, but if you are still pursuing STD on Rust, perhaps we can join efforts?

Or - if you've stumbled on a road-blocker (e.g. different from the sheer size of the generated executable which quickly eats up the ESP32 flash memory) - please let me know.

reitermarkus commented 3 years ago

Hi @ivmarkov, I have only seen this now. It would definitely make sense to join efforts on this. Did you make any more progress in the meantime?

ivmarkov commented 3 years ago

During November/December - yes. (You might want to take a look at https://github.com/ivmarkov/rust-esp32-std-hello which is the demo playground.)

In the past two months I was completely underwater with regular work activities, but I'm back at this now (over the weekends).

The good news is that the MabezDev fork is also back to live now, so maybe I need to again rebase my work on top of that fork.

How shall we tackle this?

reitermarkus commented 3 years ago

I think I will need to also rebase my fork again, since MabezDev's is now at 1.50. Once that's done I'd suggest you rebase your fork onto mine and send a PR so we can easily see the differences.

ivmarkov commented 3 years ago

Let me know once you rebase and I'll file my PR then.

reitermarkus commented 3 years ago

@ivmarkov, I'm pretty much done with my rebase. I will probably do some small refactorings still but it should be no problem to rebase again after these.