Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Drop-in replacement for this library #90

Open
karol-brejna-i opened this issue May 29, 2023 · 3 comments
Open

Drop-in replacement for this library #90

karol-brejna-i opened this issue May 29, 2023 · 3 comments

Comments

@karol-brejna-i
Copy link

karol-brejna-i commented May 29, 2023

I found this library very useful for my purposes. Combining OTA and remote debugging capabilities made things easier for my development efforts. The library is also popular among other Arduino/ESP developers.

Unfortunately, it seems like the library is not maintained anymore. The last commit was made in 2019. There are "hanging" pull requests and unattended issues.

The biggest problem with the library is that it simply doesn't compile in its current form.

I've gone through the issues and forks (there are almost 120 of them!). They are focusing mostly on the build problem. It is located here: https://github.com/karol-brejna-i/RemoteDebug.

Because I am actively using the library, I decided to fork it and make some fixes.

So if you are interested in including the functionality, you could use the fork I prepared. It is a drop-in replacement. You only need to change the dependency from joaolopesf/RemoteDebug@^3.0.5 to karol-brejna-i/RemoteDebug@^4.0.0.

The changes I made include:

  • Removed ArduinoWebsockets from the sources and used the one from the library manager
  • Included changes from the following PRs (not included in the original library ATTOW):
  • Removed unused code, cleaned up the sources
  • Make WebSockets (with the dedicated web app) work again
  • Provided some more documentation, examples

Of course, I'd be glad to contribute back to the original repo if the author is interested in that...

@karol-brejna-i
Copy link
Author

Also, I'd appreciate some feedback if the library works for your bords. I use only a small number of different ESP32 devices...

@mskressin
Copy link

Do you see any reason why your branch would not work or could be modified to work with the new Arduino Uno R4 WIFI board? I have not tried it yet, but since the Arduino Uno R4 WIFI uses the ESP32-S3 chip for WIFI, I thought I would ask since you know the code and it's dependencies.

@karol-brejna-i
Copy link
Author

Do you see any reason why your branch would not work or could be modified to work with the new Arduino Uno R4 WIFI board? I have not tried it yet, but since the Arduino Uno R4 WIFI uses the ESP32-S3 chip for WIFI, I thought I would ask since you know the code and it's dependencies.

@mskressin I haven't had any chance to work with the board you mentioned. It should work.
Please, try it out and let me know. If there are problems, we should be able to resolve them quickly.
If it works, let me know, too ;-)

I remember there were some problems with the arduino-esp32 library itself (some missing declarations for ESP32-S3), but it looks like they are solved already: espressif/arduino-esp32#8187 (comment).

Anyway, let's continue the discussion in the new repo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants