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 <[email protected]> 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/[email protected]>
_______________________________________________ Rpm-maint mailing list [email protected] https://lists.rpm.org/mailman/listinfo/rpm-maint
