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.

Sign up for GitHub

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

Jump to bottom

[asio]: Drop esp-asio patches in favor of sock-utils #717

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

[asio]: Drop esp-asio patches in favor of sock-utils #717

wants to merge 5 commits into from

Conversation

david-cermak
Copy link
Collaborator

@david-cermak david-cermak commented Dec 18, 2024 •
edited
Loading

  • switch to upstream
  • use socketpair() from sock-utils
  • use pipe-interrupter
  • stub pause() locally

TODO

  • archive esp-asio
  • move pause to sock_utils and define explicit dependency (^0.2 instead of *)
  • test everything
  • optimally -- upgrade asio to v1.32

Bump to v1.32

1.32.0

Features

  • Upgrade asio to 1.32 (b2674a0b)
  • Drop esp/asio patches in favor of sock-utils (ac633898)

Bug Fixes

  • Re-applie refs to common comps idf_component.yml (9fe44a45)
  • Reference common component from IDF (74fc228c)
  • Revert referencing protocol_examples_common from IDF (f9e0281a)
  • reference protocol_examples_common from IDF (09abb18b)
  • specify override_path in example manifest files (1d8923cf)

Updated

  • docs(asio): Updates asio docs (ce9337d3)

@david-cermak david-cermak self-assigned this Dec 18, 2024
@david-cermak david-cermak force-pushed the feat/asio_sync_upstream branch 2 times, most recently from 7cd3a5c to ace564b Compare December 18, 2024 11:42
@david-cermak david-cermak force-pushed the feat/asio_sync_upstream branch from ace564b to 12fdb51 Compare December 18, 2024 15:07
@david-cermak
feat(asio): Upgrade asio to 1.32
b2674a0
@david-cermak
bump(asio): 1.28.0~0 -> 1.32.0~0
2e87a7a 
1.32.0~0
Features
- Upgrade asio to 1.32 (b2674a0)
- Drop esp/asio patches in favor of sock-utils (ac63389)
Bug Fixes
- Re-applie refs to common comps idf_component.yml (9fe44a4)
- Reference common component from IDF (74fc228)
- Revert referencing protocol_examples_common from IDF (f9e0281)
- reference protocol_examples_common from IDF (09abb18)
- specify override_path in example manifest files (1d8923c)
Updated
- docs(asio): Updates asio docs (ce9337d)
@david-cermak
fix(asio): Make asio enable if_nametoindex to fix linking
48d2608
@david-cermak david-cermak mentioned this pull request Dec 18, 2024
Open
3 tasks
@david-cermak
Copy link
Collaborator Author

Checked for potential link issues on macOS (as reported here #614 (comment)) -- all issues are fixed in this PR (also selects LWIP_NETIF_API if asio is in the build).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@euripedesrocha euripedesrocha Awaiting requested review from euripedesrocha

At least 1 approving review is required to merge this pull request.

Assignees

@david-cermak david-cermak

Labels
asio
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@david-cermak