Package: nginx-extras
Version: 1.6.0-1
Severity: normal

Dear nginx maintainers,

I would like you to seriously consider adopting the following patch.
Basically it consists on switching over ngx_lua module on nginx-extras
in favor of luaJIT as Lua interpreter.
LuaJIT is a totally lua-compatible implementation that aims for, between
others, boosting performance on intensive CPU operations (like
arithmetics) by enabling calling external C code, and ends up building
just-in-time code to be loaded into ngx_lua stack.

Independently of this, there is a way larger extension support on LuaJIT
than regular Lua, that includes, for example, FFI built-in support. This
extends Nginx through ngx_lua into a whole new universe of features that
regular Lua interpreter alone lacks.
If it provides any help, as while developing my own apps on top of
ngx_lua, I migrated with no changes at all from lua5.1 to luajit5.1
implementation, just to serve the purpose of adopting missing features
on external modules that depended on, for example, FFI support.

Mostly all third-party ngx_lua modules out there are using it and even
the major nginx-modules provider and OpenResty maintainer fully
explains[1] his decision on why switching over to luaJIT for building
ngx_lua.
This is the reason why I upgraded this bug report severity to "normal"
instead of an expected "wishlist" one, since I consider switching over
to luajit as Lua interpreter would extremely enhance user experience.


Thanks in advance.


Cheers,

Dererk


1. https://groups.google.com/forum/#!topic/openresty-en/Fd370Ly3Upg

-- 
BOFH excuse #342:
HTTPD Error 4004 : very old Intel cpu - insufficient processing power

diff -ruN nginx-1.6.0.orig/debian/changelog nginx-1.6.0/debian/changelog
--- nginx-1.6.0.orig/debian/changelog	2014-04-24 13:23:46.000000000 -0300
+++ nginx-1.6.0/debian/changelog	2014-07-24 03:50:55.018594460 -0300
@@ -1,3 +1,11 @@
+nginx (1.6.0-2) unstable; urgency=medium
+
+  * debian/modules/nginx-lua: 
+    + Switching over to luajit as lua interpreter (Closes: #).
+    + Reflect this on debian/control.
+
+ -- Ulises Vitulli <der...@debian.org>  Thu, 24 Jul 2014 03:48:25 -0300
+
 nginx (1.6.0-1) unstable; urgency=medium
 
   [ Christos Trochalakis ]
diff -ruN nginx-1.6.0.orig/debian/control nginx-1.6.0/debian/control
--- nginx-1.6.0.orig/debian/control	2014-04-24 13:23:46.000000000 -0300
+++ nginx-1.6.0/debian/control	2014-07-24 03:00:27.348014153 -0300
@@ -15,7 +15,7 @@
                libexpat-dev,
                libgd2-dev | libgd2-noxpm-dev,
                libgeoip-dev,
-               liblua5.1-dev,
+               libluajit-5.1-dev,
                libmhash-dev,
                libpam0g-dev,
                libpcre3-dev,
diff -ruN nginx-1.6.0.orig/debian/modules/nginx-lua/config nginx-1.6.0/debian/modules/nginx-lua/config
--- nginx-1.6.0.orig/debian/modules/nginx-lua/config	2014-04-24 13:23:46.000000000 -0300
+++ nginx-1.6.0/debian/modules/nginx-lua/config	2014-07-24 03:48:10.816586362 -0300
@@ -131,9 +131,9 @@
             ngx_feature="Lua library in /usr/"
             ngx_feature_path="/usr/include/lua5.1"
             if [ $NGX_RPATH = YES ]; then
-                ngx_feature_libs="-R/usr/lib -L/usr/lib -lm -llua5.1"
+                ngx_feature_libs="-R/usr/lib -L/usr/lib -lm -lluajit-5.1"
             else
-                ngx_feature_libs="-L/usr/lib -lm -llua5.1"
+                ngx_feature_libs="-L/usr/lib -lm -lluajit-5.1"
             fi
             . auto/feature
         fi

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to