summaryrefslogtreecommitdiff
blob: 0da1b148a49c0babcc1d864a65492d3c865daf5c (plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
From b693feda0148eb421e1f59d468caceb6f3183a82 Mon Sep 17 00:00:00 2001
From: Mart Raudsepp <mart@leio.tech>
Date: Mon, 16 Mar 2020 23:43:12 +0200
Subject: [PATCH] build: Handle lua as needed by Gentoo

Gentoo currently uses unversioned lua:0 - lua.pc instad of lua5.1.pc,
/usr/bin/lua instead of /usr/bin/lua5.1
Additionally lua and luajit can be installed in parallel, but only one
should be chosen - so add explicit meson options to choose between them.
This is currently in a hacky un-upstreamable way: working correctly only
if both options are disabled, or only one is enabled and the other
disabled, but having logic issues if any is "auto" or both enabled; this
is because feature option is mainly used to handle explicit choice
between them more easily in a Gentoo-specific patch without having to
add various conditional branches thanks to the feature being disabled
acting as a full disabler, so the other option is automatically not
checked.
---
 meson.build       | 22 +++++++---------------
 meson_options.txt |  5 ++++-
 2 files changed, 11 insertions(+), 16 deletions(-)

diff --git a/meson.build b/meson.build
index 8322fea..49e5011 100644
--- a/meson.build
+++ b/meson.build
@@ -117,18 +117,12 @@ endif
 
 python2_dep = dependency('python2', version: python2_req, required: false)
 pygobject_dep = dependency('pygobject-3.0', version: pygobject_req, required: false)
-lua51_dep = dependency('lua51', version: lua_req, required: false)
-if not lua51_dep.found()
-  lua51_dep = dependency('lua-5.1', version: lua_req, required: false)
-endif
-luajit_dep = dependency('luajit', version: luajit_req, required: false)
+lua51_dep = dependency('lua', version: lua_req, required: get_option('lua51'))
+luajit_dep = dependency('luajit', version: luajit_req, required: get_option('luajit'))
 lua_lgi_found = false
 lua_lgi_ver = 'not found'
-lua51_prg = find_program('lua5.1', required: false)
-if not lua51_prg.found()
-  lua51_prg = find_program('lua51', required: false)
-endif
-luajit_prg = find_program('luajit', required: false)
+lua51_prg = find_program('lua', required: get_option('lua51'))
+luajit_prg = find_program('luajit', required: get_option('luajit'))
 xmllint_prg = find_program('xmllint', required: false)
 
 if (luajit_dep.found() and luajit_prg.found()) or (lua51_dep.found() and lua51_prg.found())
@@ -144,6 +138,8 @@ if (luajit_dep.found() and luajit_prg.found()) or (lua51_dep.found() and lua51_p
             lua51_lgi_dep = declare_dependency(version: lua_lgi_ver)
             lua_lgi_found = true
         endif
+    else
+        error('lua support requested but lua-lgi not found')
     endif
     message('lua-lgi version: ' + lua_lgi_ver)
 endif
@@ -222,11 +218,7 @@ if generate_gir and not introspection_dep.found()
   generate_gir = false
 endif
 
-build_lua51_loader = get_option('lua51')
-lua51_found = (luajit_dep.found() or lua51_dep.found()) and lua_lgi_found
-if build_lua51_loader and not lua51_found
-  build_lua51_loader = false
-endif
+build_lua51_loader = (luajit_dep.found() or lua51_dep.found()) and lua_lgi_found
 
 build_python2_loader = get_option('python2')
 python2_found = python2_dep.found() and pygobject_dep.found()
diff --git a/meson_options.txt b/meson_options.txt
index 18e1779..dc3e776 100644
--- a/meson_options.txt
+++ b/meson_options.txt
@@ -1,6 +1,9 @@
 option('lua51',
-       type: 'boolean', value: true,
+       type: 'feature', value: 'enabled',
        description: 'Enable Lua 5.1 support (requires lua-lgi)')
+option('luajit',
+       type: 'feature', value: 'disabled',
+       description: 'Use LuaJIT for Lua 5.1 support (requires lua-lgi)')
 
 option('python2',
        type: 'boolean', value: false,
-- 
2.20.1