[Pkg-nginx-maintainers] Bug#941917: nginx: FTBFS on several architectures: luajit.h: No such file or directory

[Roland Rosenfeld]

On Di, 08 Okt 2019, Hideki Yamane wrote:

>  In debian/control, Build-Depends: libluajit-5.1-dev [i386 amd64
>  kfreebsd-i386 armel armhf powerpc powerpcspe mips mipsel] is wrong,
>  as https://buildd.debian.org/status/package.php?p=luajit&suite=sid
>  says below archs are built.

>  So, it should be libluajit-5.1-dev [amd64 arm64 armel armhf i386
>  mips64el mipsel ppc64el hurd-i386 kfreebsd-amd64 kfreebsd-i386
>  powerpc ppc64] and so on.

That should be correct, but it does not solve the problem for s390x,
where libluajit-5.1-dev still does not exist.

> And it seems that it should check whether its arch has
> libluajit-5.1-dev or not.

That's the root problem.
Looking at the salsa git repository of nignx, the following commit
seems to trigger the problem:

https://salsa.debian.org/nginx-team/nginx/commit/132704ab76aa72ce29d00e4acd50d3218693558b#f072d72ac821d4e63439739975074c7c5aed9bf7

commit 132704ab76aa72ce29d00e4acd50d3218693558b
Author: Christos Trochalakis <ctrochalakis at debian.org>
Date:   Mon Sep 9 17:32:06 2019 +0300

    http-lua: Upgrade to 0.10.15

[...]
--- a/debian/modules/http-lua/src/ngx_http_lua_common.h
+++ b/debian/modules/http-lua/src/ngx_http_lua_common.h
@@ -17,7 +17,7 @@
 #include <setjmp.h>
 #include <stdint.h>
 
-#include <lua.h>
+#include <luajit.h>
 #include <lualib.h>
 #include <lauxlib.h>
[...]

So here we may need a mechanism that checks whether luajit.h exists
and falls back to lua.h otherwise or stop building nginx on systems
where libluajit-5.1-dev does not exist.

From my point of view this should not be a blocker of the perl
transition, since nginx 1.16.1-1 with the above change was uploaded
2019-09-12 and never successfully built on arm64, mpis64el, ppc64el
and s390x while the perl migration started at 2019-10-05.

I expect the testing version 1.14.2-3 to build without problems with
perl 5.30.0-6 on these architectures...

Greetings
Roland