Skip to content

Porting mupnp for ESP32 with FreeRTOS#24

Merged
cybergarage merged 3 commits intomasterfrom
copilot/add-esp32-network-code
Apr 14, 2026
Merged

Porting mupnp for ESP32 with FreeRTOS#24
cybergarage merged 3 commits intomasterfrom
copilot/add-esp32-network-code

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 13, 2026

Changes

src/mupnp/net/interface_function.c

  • Added ESP_PLATFORM branch
  • Added mupnp_net_gethostinterfaces() implementation using esp_netif to enumerate host network interfaces
  • Populate IP address, netmask, and MAC address into mUpnpNetworkInterface
  • Skip loopback interfaces and interfaces with unset IP addresses

src/mupnp/net/socket.c

  • Added guards to exclude signal.h inclusion and SIGPIPE signal handling on ESP32

src/mupnp/util/thread.c

  • Added guards to disable processing that depends on signal.h, sigaction, and sigset_t on ESP32
  • pthread-based execution flow is preserved

Copilot AI and others added 3 commits April 13, 2026 11:42
Agent-Logs-Url: https://github.com/cybergarage/mupnp/sessions/8a0b8591-74d5-4453-ada1-e8fb2eff92c7

Co-authored-by: cybergarage <586639+cybergarage@users.noreply.github.com>
Agent-Logs-Url: https://github.com/cybergarage/mupnp/sessions/8a0b8591-74d5-4453-ada1-e8fb2eff92c7

Co-authored-by: cybergarage <586639+cybergarage@users.noreply.github.com>
@sonarqubecloud
Copy link
Copy Markdown

@cybergarage cybergarage changed the title docs: Confirm ESP32 pthread compatibility for mutex.c Porting mupnp for ESP32 with FreeRTOS Apr 13, 2026
Copilot AI requested a review from cybergarage April 13, 2026 13:35
Copilot stopped work on behalf of cybergarage due to an error April 13, 2026 13:35
@codecov-commenter
Copy link
Copy Markdown

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

@cybergarage cybergarage merged commit 077062b into master Apr 14, 2026
6 of 8 checks passed
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

Successfully merging this pull request may close these issues.

3 participants