[Rpm-maint] [rpm-software-management/rpm] rpmio/lposix.hh: add LUALIB_API to luaopen_posix() definition (PR #3971)
Arturo Borrero Gonzalez
notifications at github.com
Tue Sep 30 08:57:59 UTC 2025
In file rpmio/lposix.cc the luaopen_posix() function is defined
with LUALIB_API, but in the header it doesn't have it.
This can produce linkage-time problems:
WARN /home/build/rpmio/lposix.cc:874:16: error: conflicting declaration of 'int luaopen_posix(lua_State*)' with 'C' linkage
WARN 874 | LUALIB_API int luaopen_posix (lua_State *L)
WARN | ^~~~~~~~~~~
WARN In file included from /home/build/rpmio/lposix.cc:38:
WARN /home/build/rpmio/lposix.hh:11:5: note: previous declaration with 'C++' linkage
WARN 11 | int luaopen_posix (lua_State *L);
WARN | ^~~~~~~~~~~
Signed-off-by: Arturo Borrero Gonzalez <arturo.borrero at chainguard.dev>
You can view, comment on, or merge this pull request online at:
https://github.com/rpm-software-management/rpm/pull/3971
-- Commit Summary --
* rpmio/lposix.hh: add LUALIB_API to luaopen_posix() definition
-- File Changes --
M rpmio/lposix.hh (2)
-- Patch Links --
https://github.com/rpm-software-management/rpm/pull/3971.patch
https://github.com/rpm-software-management/rpm/pull/3971.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/3971
You are receiving this because you are subscribed to this thread.
Message ID: <rpm-software-management/rpm/pull/3971 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rpm.org/pipermail/rpm-maint/attachments/20250930/27b6d0ba/attachment-0001.htm>
More information about the Rpm-maint
mailing list