Skip to content

Conversation

@night1rider
Copy link
Contributor

Add documentation about using wolfSSH as an external module.

Copy link

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 comprehensive documentation for integrating wolfSSH as an external module in Zephyr. The documentation follows the established pattern used by other external modules (notably wolfSSL) and provides clear guidance for developers.

  • Introduces wolfSSH as a lightweight SSH library suitable for embedded systems
  • Provides complete integration instructions including west.yml configuration
  • Documents the dependency on wolfSSL and includes relevant external references

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.

Copy link
Contributor

@kartben kartben left a comment

Choose a reason for hiding this comment

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

minor request for changes - thank you for this!

Add documentation to external module section for adding wolfSSH to Zephyr

Signed-off-by: Zackery Backman <backman23@live.com>
@night1rider night1rider force-pushed the wolfSSH-documentation branch from f18b7ef to c8d1ca1 Compare December 2, 2025 21:40
@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 2, 2025

Now west recognizes ``wolfssh`` as a module, and will include its Kconfig and CMakeLists.txt in
the build system.

For more regarding the usage of wolfSSH with Zephyr, please refer to the `wolfSSH Zephyr Example
Copy link
Contributor

Choose a reason for hiding this comment

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

minor nit, "For more information regarding.."

@nashif nashif merged commit 2481516 into zephyrproject-rtos:main Dec 6, 2025
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants