Fix %luarocks_* macros.

The `%lua_version`, `%lua_version_nodots`, and `%lua_noarchdir`
macros have been refactored to use the RPM internal Lua
interpreter blocks (`%{lua:...}`) instead of relying solely on
shell command execution (`%()`).

This change offers several benefits:
1. Consistency: Standardizes the approach, as other macros like
   `%lua_version_default` already use RPM Lua blocks.
2. Robustness: Improves path handling by explicitly trimming
   whitespace/newlines from the `pkgconf` output in
   `%lua_noarchdir`.
3. Clarity: Simplifies the logic for extracting the version
   number and calculating the no-dots version.

The `%lua_noarchdir` logic was also improved to correctly handle
a failure of the `pkgconf` command (e.g., when the package isn't
installed) by checking for `f == nil`.

A comment was also added to `%lua_provides` to clarify the
required definition of `%mod_name`.

.gitattributes

@@ -21,3 +21,4 @@
 *.xz filter=lfs diff=lfs merge=lfs -text
 *.zip filter=lfs diff=lfs merge=lfs -text
 *.zst filter=lfs diff=lfs merge=lfs -text
+*.changes merge=merge-changes

.gitignore

@@ -1 +1,9 @@
 .osc
+_scmsync.obsinfo
+_buildconfig-*
+_buildinfo-*.xml
+lua-macros-*-build/
+*.obscpio
+*.osc
+_build.*
+.pbuild

Makefile

@@ -0,0 +1,18 @@
+all:
+	mkdir -p build && cd build/ \
+	&& cmake .. \
+		'-GUnix Makefiles' \
+		-DCMAKE_INSTALL_PREFIX:PATH=/usr \
+		-DCMAKE_BUILD_TYPE=RelWithDebInfo '-DCMAKE_C_FLAGS=-O2 -Wall -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=3 -fstack-protector-strong -funwind-tables -fasynchronous-unwind-tables -fstack-clash-protection -Werror=return-type -flto=auto -g' \
+		'-DCMAKE_EXE_LINKER_FLAGS=-flto=auto -Wl,--as-needed -Wl,--no-undefined -Wl,-z,now' \
+		'-DCMAKE_MODULE_LINKER_FLAGS=-flto=auto -Wl,--as-needed' \
+		'-DCMAKE_SHARED_LINKER_FLAGS=-flto=auto -Wl,--as-needed -Wl,--no-undefined -Wl,-z,now' \
+		-DCMAKE_VERBOSE_MAKEFILE:BOOL=ON \
+		-DBUILD_SHARED_LIBS:BOOL=ON \
+		-DCMAKE_COLOR_MAKEFILE:BOOL=OFF \
+		-DCMAKE_POLICY_VERSION_MINIMUM=3.5 \
+		-DLUA_INCLUDE_DIR:PATH=$(shell pkgconf --variable=includedir lua) \
+	&& make
+
+clean:
+	rm -rf build CMakeCache.txt

lua-macros.changes

@@ -1,3 +1,38 @@
+-------------------------------------------------------------------
+Tue Oct 14 23:16:09 UTC 2025 - Matej Cepl <mcepl@cepl.eu>
+
+- Fix %luarocks_* macros.
+
+-------------------------------------------------------------------
+Mon Oct 13 20:24:01 UTC 2025 - Matej Cepl <mcepl@cepl.eu>
+
+- Fix %%lua_provides.
+
+-------------------------------------------------------------------
+Mon Oct 13 17:20:42 UTC 2025 - Matej Cepl <mcepl@cepl.eu>
+
+- Quote %%lua_archdir macro.
+
+-------------------------------------------------------------------
+Sat Oct 11 16:45:06 UTC 2025 - Matej Cepl <mcepl@cepl.eu>
+
+- Rework Lua version and path macros using RPM Lua blocks.
+
+-------------------------------------------------------------------
+Fri Oct 10 10:56:01 UTC 2025 - Matej Cepl <mcepl@cepl.eu>
+
+- Make %{lua_noarchdir} work even without lua-devel installed.
+
+-------------------------------------------------------------------
+Thu Oct  9 22:44:04 UTC 2025 - Matej Cepl <mcepl@cepl.eu>
+
+- Make definition of variables dynamic based on pkgconf output.
+
+-------------------------------------------------------------------
+Tue Aug  5 14:05:41 UTC 2025 - Matej Cepl <mcepl@cepl.eu>
+
+- On Factory BR lua-interpreter.
+
 -------------------------------------------------------------------
 Fri Nov 25 19:15:41 UTC 2022 - Michal Suchanek <msuchanek@suse.com>
 

lua-macros.spec

@@ -27,10 +27,13 @@ License:        MIT
 Group:          Development/Languages/Other
 URL:            https://www.lua.org
 Source0:        macros.lua
-%if 0%{?suse_version} >= 1500
+%if 0%{?suse_version} > 1600
+Requires:       lua-interpreter
+%elif 0%{?suse_version} >= 1500
 # on SLE 12 lua is lua5.2 unconditionally, avoid
 Requires:       lua
 %endif
+Requires:       pkgconfig
 BuildArch:      noarch
 
 %description
@@ -38,9 +41,6 @@ RPM macros for lua packaging
 
 %prep
 cp -p %{SOURCE0} .
-%if 0%{?rhel} || 0%{?fedora}
-sed -i -e '/includedir/s!/lua!&-!' macros.lua
-%endif
 
 %build
 :

macros.lua

@@ -1,18 +1,48 @@
 # RPM macros for Lua
 
 # The major.minor version of Lua
-%lua_version %(lua -e 'print(_VERSION)' | cut -d ' ' -f 2)
-%lua_version_nodots %(lua -e 'print((string.gsub("%{lua_version}", "%.", "")))')
-
+%lua_version %{lua:
+local f = io.popen("lua -e 'print(_VERSION)'")
+local output = f:read("*a")
+f:close()
+local ver = output:match("Lua (%d%.%d)")
+print(ver)
+}
+%lua_version_nodots %{lua:
+  local ver = rpm.expand("%{lua_version}")
+  local nodots = ver:gsub("%.", "")
+  print(nodots)
+}
 # compiled modules should go here
-%lua_archdir %{_libdir}/lua/%{lua_version}
-
+%lua_archdir %{lua:
+  local handle = io.popen("pkgconf --variable=INSTALL_CMOD lua 2>/dev/null")
+  local result = handle:read("*a")
+  handle:close()
+  result = result:gsub("^%s*(.-)%s*$", "%1")
+  print(result)
+}
 # pure Lua modules should go here
-%lua_noarchdir %{_datadir}/lua/%{lua_version}
-
+%lua_noarchdir %{lua:
+local output = ""
+local f = io.popen("pkgconf --variable=INSTALL_LMOD lua 2>/dev/null")
+if f ~= nil then
+    output = f:read("*a"):gsub("^%s*(.-)%s*$", "%1")
+    f:close()
+end
+if output:len() > 0 then
+    print(output)
+    return
+end
+print(rpm.expand("%{_datadir}/lua/%{lua_version}"))
+}
 # lua includes folder
-%lua_incdir %{_includedir}/lua%{lua_version}
-
+%lua_incdir %{lua:
+  local handle = io.popen("pkgconf --variable=includedir lua 2>/dev/null")
+  local result = handle:read("*a")
+  handle:close()
+  result = result:gsub("^%s*(.-)%s*$", "%1")
+  print(result)
+}
 %lua_version_default %{lua:
 local result = 5.4
 local ver = rpm.expand("%{?suse_version}")
@@ -20,45 +50,54 @@ if #ver > 0 then
     ver = tonumber(ver)
     if ver < 1500 then
         result = 5.1
-    elseif ver == 1500 then
+    elseif ver < 1600 then
         result = 5.3
     end
 end
 print(result)
 }
-%lua_version_default_nodots %(lua -e 'print((string.gsub("%{lua_version_default}", "%.", "")))')
-
-%ifluadefault \
-%if %{lua_version_nodots} == %{lua_version_default_nodots} \
-%{nil}
-
+# Define a conditional that evaluates true when building for the default Lua version
+%lua_version_default_nodots %{lua:
+  local ver = rpm.expand("%{lua_version_default}")
+  local nodots = ver:gsub("%.", "")
+  print(nodots)
+}
 # Lua default version
+# This REQUIRES macro %{mod_name} to be defined.
 # -e: Exclude lua prefix
 # -n: Specify name
-%lua_provides(en:) \
-%ifluadefault \
-%if 0%{?-n:1} \
-Provides: %{-n*} = %{version}-%{release} \
-Obsoletes: %{-n*} < %{version}-%{release} \
-%else \
-%if 0%{?-e:1} \
-Provides: %{mod_name} = %{version}-%{release} \
-Obsoletes: %{mod_name} < %{version}-%{release} \
-%else \
-Provides: lua-%{mod_name} = %{version}-%{release} \
-Obsoletes: lua-%{mod_name} < %{version}-%{release} \
-%endif \
-%endif \
-%endif \
-%{nil}
+%lua_provides(en:) %{lua:
+local mod_name = rpm.expand("%{?mod_name}")
+if mod_name == "" then
+    print("-- Error: %{mod_name} is not defined!")
+    return
+end
+print([[
+%if "%{lua_version_nodots}" == "%{lua_version_default_nodots}"
+%if 0%{?-n:1}
+Provides: %{-n*} = %{version}-%{release}
+Obsoletes: %{-n*} < %{version}-%{release}
+%else
+%if 0%{?-e:1}
+Provides: %{mod_name} = %{version}-%{release}
+Obsoletes: %{mod_name} < %{version}-%{release}
+%else
+Provides: lua-%{mod_name} = %{version}-%{release}
+Obsoletes: lua-%{mod_name} < %{version}-%{release}
+%endif
+%endif
+%endif
+]])
+}
 
 # LuaRocks
 %luarocks_build \
-luarocks --lua-version "%{lua_version}" \\\
-make --pack-binary-rock --deps-mode none
-
-%luarocks_install \
-luarocks --lua-version="%{lua_version}" --tree="%{buildroot}%{_prefix}" \\\
-install --deps-mode=none --no-manifest
+luarocks --lua-version "%{lua_version}" make --deps-mode none --pack-binary-rock
 
 %luarocks_treedir %{_prefix}/lib/luarocks/rocks-%{lua_version}
+
+%luarocks_install \
+  /bin/sh -c 'luarocks --lua-version="%{lua_version}" --tree="%{buildroot}%{_prefix}" install --deps-mode=none --no-manifest "$@" && \
+  source_dir="%{buildroot}%{luarocks_treedir}/%{mod_name}/%{rock_version}" && \
+  [ -d "${source_dir}" ] && [ "$(ls -A "${source_dir}" | wc -l)" -gt 0 ] && \
+  mv -v "${source_dir}" __rocktree' --