Skip to content

Conversation

ajsb85
Copy link

@ajsb85 ajsb85 commented Oct 15, 2025

No description provided.

Updated instructions for compiling the example on Linux to include the correct path for the executable.
@Copilot Copilot AI review requested due to automatic review settings October 15, 2025 22:34
@CLAassistant
Copy link

CLAassistant commented Oct 15, 2025

CLA assistant check
All committers have signed the CLA.

Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds installation instructions for the libbsd library dependency required when building ESP-IDF websocket client examples for the Linux target. The change addresses a missing prerequisite that prevents successful compilation.

  • Added package installation commands for major Linux distributions (Debian/Ubuntu, Fedora/RHEL, Arch, Alpine)
  • Corrected the executable path from ./websocket.elf to ./build/websocket.elf

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@github-actions github-actions bot changed the title ESP-IDF linux target expects libbsd headers ESP-IDF linux target expects libbsd headers (IDFGH-16636) Oct 15, 2025
@espressif-bot espressif-bot added the Status: Opened Issue is new label Oct 15, 2025
Copy link
Collaborator

@david-cermak david-cermak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for fixing the docs!

@david-cermak
Copy link
Collaborator

@ajsb85 Please amend the commit message per https://github.com/espressif/esp-protocols/actions/runs/18544522420/job/52886371480?pr=911

maybe something like: fix(websocket): Update linux build docs on required libs

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

Labels

Status: Opened Issue is new

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants