-
-
Notifications
You must be signed in to change notification settings - Fork 497
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
AsyncTCP parameter name change #1951
Comments
Cannot reproduce... I cleaned all files in
And please keep in mind, the patch will NOT be applied to the But you are right, the new define |
Hi, Line 5 was apparently empty. OT |
@tbnobody thanks for your quick response. You're right, the patch is about @schneeer you're right it seems to be about white spaces in the patch file. My last choice is applying the patch manually, removing the patch file and make sure not cleaning |
Ja genau, so oder ähnlich war das bei mir auch, Ich habe nur leider wenig Ahnung und war froh, das ich das irgendwie hinbekommen habe. |
hmmm, jedenfalls wenn man die Leerzeichen wieder löscht ist der ursprüngliche Fehler |
What happened?
patch file "event_queue_size.patch" included in OpenDTU package fails to install patch with error:
error: corrupt patch at line 5
To Reproduce Bug
Library Manager: Installing mathieucarbou/Async TCP @ ^3.0.2
Unpacking 0% 10% 20% 30% 40% 50% 60% 70% 80%
Library Manager: Async [email protected] has been installed!
Expected Behavior
Expected behavior is no error message on such operation described above.
Install Method
Self-Compiled
What git-hash/version of OpenDTU?
g5b5a984
Relevant log/trace output
Anything else?
The new version 3.1.2 of AsyncTCP already has such patch installed, however the defined parameter is named slightly different.
OpenDTU uses "CONFIG_ASYNC_TCP_EVENT_QUEUE_SIZE" vs. "CONFIG_ASYNC_TCP_QUEUE_SIZE" used in AsyncTCP library.
It appears the event_queue_size.patch is no longer needed and the parameter name should be changed in platformio.ini file accordingly.
The text was updated successfully, but these errors were encountered: