> > Fix #include lines to use "lua.h" instead of <lua.h> as per 
> > cmake-modules(7):
> 
> The actual issue in the FindLua doc is cautioning against `#include 
> <lua/lua.h>` because of the `lua/` _directory_ is not standardized. The 
> guidance to use "lua.h" seems quite odd, as quotes imply per-project files 
> rather than something from the system.

Yup, it does seem odd :smile:

> Once upon a time, the Lua interpreter sources were bundled in rpm, and for 
> _that_ the quotes were the right thing to do, plus there seems to be one 
> leftover from those days in rpmio/lposix.c. So I'd just fix that one case to 
> use `<lua.h>` instead and leave the rest alone.

Right, and good catch, that one leftover, I'll fix up the commit as suggested.



-- 
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/2469#issuecomment-1498623854
You are receiving this because you are subscribed to this thread.

Message ID: <rpm-software-management/rpm/pull/2469/c1498623...@github.com>
_______________________________________________
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint

Reply via email to