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

Documentation references non existing file: llext/shell_loader/README.rst #79318

Open
markus-becker-tridonic-com opened this issue Oct 2, 2024 · 1 comment · May be fixed by #79429
Open

Documentation references non existing file: llext/shell_loader/README.rst #79318

markus-becker-tridonic-com opened this issue Oct 2, 2024 · 1 comment · May be fixed by #79429
Assignees
@teburd
Labels
area: Documentation area: llext Linkable Loadable Extensions bug The issue is a bug, or the PR is fixing a bug priority: low Low impact/importance bug

Comments

@markus-becker-tridonic-com
Copy link
Contributor

Describe the bug

https://github.com/zephyrproject-rtos/zephyr/blob/main/samples/subsys/llext/shell_loader/README.rst references to a file tests/subsys/llext/hello_world/hello_world.c, which is not existent anymore. Possibly it is now samples/subsys/llext/modules/src/hello_world_ext.c .

To Reproduce

Impact

LLEXT shell loader sample cannot be used.

Logs and console output
N/A

Environment (please complete the following information):
N/A

Additional context
N/A
@markus-becker-tridonic-com markus-becker-tridonic-com added the bug The issue is a bug, or the PR is fixing a bug label Oct 2, 2024
@henrikbrixandersen henrikbrixandersen added the area: llext Linkable Loadable Extensions label Oct 3, 2024
@teburd
Copy link
Collaborator

teburd commented Oct 3, 2024

Thanks for reporting, I am working on this

@teburd teburd linked a pull request Oct 4, 2024 that will close this issue
Draft
@mmahadevan108 mmahadevan108 added the priority: low Low impact/importance bug label Oct 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@teburd teburd

Labels
area: Documentation area: llext Linkable Loadable Extensions bug The issue is a bug, or the PR is fixing a bug priority: low Low impact/importance bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

llext: Shell sample fixups teburd/zephyr
4 participants
@teburd @henrikbrixandersen @markus-becker-tridonic-com @mmahadevan108