[Bug 226878] www/elinks: lua support does not work

[bugzilla-noreply at freebsd.org]

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=226878

            Bug ID: 226878
           Summary: www/elinks: lua support does not work
           Product: Ports & Packages
           Version: Latest
          Hardware: Any
                OS: Any
            Status: New
          Severity: Affects Some People
          Priority: ---
         Component: Individual Port(s)
          Assignee: beat at FreeBSD.org
          Reporter: z7dr6ut7gs at snkmail.com
          Assignee: beat at FreeBSD.org
             Flags: maintainer-feedback?(beat at FreeBSD.org)

I suspect that elinks lua support is quite old and won't work with lua later
than 5.0 (which is gone from ports).

It builds, but configure fails, so lua is turned off.
'make configure' with the LUA option on produces:

===================
 .
 .
checking for Lua... no
 .
 .
===================


And in config.log...

===================
 .
 .
configure:17581: checking for Lua
configure:17640: cc -o conftest -O2 -pipe  -I/usr/local/include/nspr
-I/usr/local/include/js-1.7 -fstack-protector -isystem /usr/local/include
-fno-strict-aliasing -Wall  -I/usr/local/include/lua51 -isystem
/usr/local/include    -L/usr/local/lib -lnspr4 -pthread -lpthread -lm
-Wl,-rpath,/usr/local/lib -fstack-protector -rdynamic conftest.c -llua -llualib
-lm -ljs -lexecinfo -L/usr/local/lib -lbz2 -lexpat >&5
conftest.c:145:17: warning: implicit declaration of function 'luaL_newstate' is
invalid in C99 [-Wimplicit-function-declaration]
        lua_State *L = lua_open();
                       ^
/usr/local/include/lua51/lua.h:287:20: note: expanded from macro 'lua_open'
#define lua_open()      luaL_newstate()
                        ^
conftest.c:145:13: warning: incompatible integer to pointer conversion
initializing 'lua_State *' (aka 'struct lua_State *') with an expression of
type 'int' [-Wint-conversion]
        lua_State *L = lua_open();
                   ^   ~~~~~~~~~~
2 warnings generated.
/usr/bin/ld: cannot find -llua
 .
 .
================


And in config.h:

/* Define if you want: Lua support */
/* #undef CONFIG_LUA */


Just fixing the compilation args used for the configure test to work with lua51
(for example) is not enough.  The resulting test program seg faults (lua_open
produces something luaopen_base() does not like).


Unless someone wants to do surgery to update elinks to work with newer lua (and
submit upstream ideally), I suggest we just remove the option and pass
--without-lua to configure.

-- 
You are receiving this mail because:
You are the assignee for the bug.