Skip to content

Add Lua component (IEC-470)#658

Merged
suda-morris merged 7 commits into
espressif:masterfrom
georgik:feature/lua
May 11, 2026
Merged

Add Lua component (IEC-470)#658
suda-morris merged 7 commits into
espressif:masterfrom
georgik:feature/lua

Conversation

@georgik
Copy link
Copy Markdown
Contributor

@georgik georgik commented Jan 8, 2026

The Lua component is getting popular, so idea is to move it under Espressif namespace.

Formerly it was hosted here: https://github.com/georgik/esp-idf-component-lua

Component Registry: https://components.espressif.com/components/georgik/lua/

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Jan 8, 2026

CLA assistant check
All committers have signed the CLA.

@github-actions github-actions Bot changed the title Add Lua component Add Lua component (IEC-470) Jan 8, 2026
Comment thread lua/README.md Outdated
Comment thread lua/include/luaconf.h Outdated
Comment thread lua/README.md Outdated
Comment thread lua/include/luaconf.h Outdated
@georgik
Copy link
Copy Markdown
Contributor Author

georgik commented Jan 9, 2026

All builds are green, just one is failing in picolib with from ESP-IDF, I guess this is not specific to this component:

warning: times is not implemented and will always fail

Any idea how to resolve it?
https://github.com/espressif/idf-extra-components/actions/runs/20848120564/job/59896553185?pr=658#step:5:1673

@georgik
Copy link
Copy Markdown
Contributor Author

georgik commented Apr 29, 2026

@suda-morris I'm trying to pass tests, yet still one is failing.
It seems that in the particular version there is issue with: times is not implemented and will always fail
Build and Run Apps / Build Apps (latest, 3) (pull_request)

Should I add into .ignore_build_warnings.txt:
warning: times is not implemented and will always fail

or is there any other recommended approach, since warning appears in the final linking stage of lua_example for ESP32 target. The warning originates from system library (picolibc), not Lua component code. The times() POSIX function isn't implemented in ESP32's libc but gets linked anyway.

@georgik
Copy link
Copy Markdown
Contributor Author

georgik commented May 4, 2026

@suda-morris @Lapshin All checks seems to be green. Who could confirm the merge?

Copy link
Copy Markdown
Collaborator

@Lapshin Lapshin left a comment

Choose a reason for hiding this comment

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

LGTM!

@suda-morris suda-morris merged commit 4eccbc4 into espressif:master May 11, 2026
85 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.

5 participants