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.

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

Missing symbols with the current linux build #35

Closed
ttyyls opened this issue Sep 1, 2024 · 2 comments · Fixed by #36
Closed

Missing symbols with the current linux build #35

ttyyls opened this issue Sep 1, 2024 · 2 comments · Fixed by #36

Comments

@ttyyls
Copy link

ttyyls commented Sep 1, 2024

Hi,

Description

luasystem build is missing a couple of symbols. which makes lua-busted fail to load with the following error:

lua: error loading module 'system.core' from file '/usr/lib/lua/5.4/system/core.so':
	Error relocating /usr/lib/lua/5.4/system/core.so: bitflags_open: symbol not found
stack traceback:
	[C]: in ?
	[C]: in function 'require'
	/usr/share/lua/5.4/system.lua:7: in main chunk
	[C]: in function 'require'
	/usr/share/lua/5.4/busted/core.lua:6: in main chunk
	[C]: in function 'require'
	/usr/share/lua/5.4/busted/runner.lua:18: in function 'busted.runner'
	/usr/bin/busted:3: in main chunk
	[C]: in ?

Environment

Chimera Linux
Lua 5.4
LuaRocks N/A
Clang version 18.1.8

Steps to reproduce

install and run busted on chimera-linux

Attempts to resolve

The following downstream patch fixes the issue

diff --git a/src/Makefile b/src/Makefile
index b4ed16f..6425a4a 100644
--- a/src/Makefile
+++ b/src/Makefile
@@ -217,7 +217,7 @@ LUALIB= $(LUALIB_$(PLAT))
 #------
 # Objects
 #
-OBJS=core.$(O) compat.$(O) time.$(O) environment.$(O) random.$(O) term.$(O)
+OBJS=bitflags.$(O) compat.$(O) core.$(O) environment.$(O) random.$(O) term.$(O) time.$(O) wcwidth.$(O)
 
 #------
 # Targets

linked pr chimera-linux/cports#2796

It'd be great if this fix or something equivalent were to be incorporated upstream, instead of having to maintain it downstream.

@Tieske
Copy link
Member

Tieske commented Sep 3, 2024

released in v0.4.4

@ttyyls
Copy link
Author

ttyyls commented Sep 3, 2024

Thanks!

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

Successfully merging a pull request may close this issue.

2 participants