Skip to content

Commit

Permalink
Add support for cython_args
Browse files Browse the repository at this point in the history
cython_args was previoously ignored by Meson.
  • Loading branch information
tristan957 authored and nirbheek committed Mar 23, 2022
1 parent 2647f18 commit caa5220
Show file tree
Hide file tree
Showing 4 changed files with 39 additions and 0 deletions.
1 change: 1 addition & 0 deletions mesonbuild/backend/ninjabackend.py
Original file line number Diff line number Diff line change
Expand Up @@ -1608,6 +1608,7 @@ def generate_cython_transpile(self, target: build.BuildTarget) -> \
args += cython.get_option_compile_args(opt_proxy)
args += self.build.get_global_args(cython, target.for_machine)
args += self.build.get_project_args(cython, target.subproject, target.for_machine)
args += target.get_extra_args('cython')

ext = opt_proxy[OptionKey('language', machine=target.for_machine, lang='cython')].value

Expand Down
5 changes: 5 additions & 0 deletions test cases/cython/3 cython_args/cythonargs.pyx
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
def test():
IF VALUE:
return 1
ELSE:
return 0
30 changes: 30 additions & 0 deletions test cases/cython/3 cython_args/meson.build
Original file line number Diff line number Diff line change
@@ -0,0 +1,30 @@
project('cython_args', ['cython', 'c'])

pymod = import('python')
python = pymod.find_installation('python3')
python_dep = python.dependency()
if not python_dep.found()
error('MESON_SKIP_TEST: Python library not found.')
endif

mod = python.extension_module(
'cythonargs',
files('cythonargs.pyx'),
cython_args: [
'--compile-time-env',
'VALUE=1'
],
dependencies: [python_dep]
)

test(
'test',
python,
args: [
'test.py'
],
workdir: meson.current_source_dir(),
env: environment({
'PYTHONPATH': meson.current_build_dir(),
})
)
3 changes: 3 additions & 0 deletions test cases/cython/3 cython_args/test.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
import cythonargs

assert cythonargs.test() == 1

0 comments on commit caa5220

Please sign in to comment.