Feature/meson support #522
Conversation
lcm-python/meson.build
Outdated
| if windows | ||
| lcm_python_lib = library('lcm-python', lcm_python_sources, | ||
| dependencies : lcm_static_lib_dep, | ||
| include_directories : include_directories(python_include), | ||
| install_dir : python_site + '/lcm', | ||
| name_suffix : '.pyd') | ||
| elif host_machine.system() == 'darwin' #macos | ||
| lcm_python_lib = library('lcm-python', lcm_python_sources, | ||
| dependencies : lcm_static_lib_dep, | ||
| include_directories : include_directories(python_include), | ||
| install_dir : python_site + '/lcm', | ||
| link_args : '-undefined dynamic_lookup -Wl,-no_fixup_chains') | ||
| else | ||
| lcm_python_lib = library('lcm-python', lcm_python_sources, | ||
| dependencies : lcm_static_lib_dep, | ||
| include_directories : include_directories(python_include), | ||
| install_dir : python_site + '/lcm') | ||
| endif |
There was a problem hiding this comment.
You'll want a Python extension module rather than library here
There was a problem hiding this comment.
Switched to extension_module and set up install: 7ebd789
Zinput
force-pushed
the
feature/meson_support
branch
from
c09c279
to
2918a0b
Compare
Zinput
force-pushed
the
feature/meson_support
branch
3 times, most recently
from
5a2e0f9
to
5be306d
Compare
Done: 5be306d |
There was a problem hiding this comment.
The way the Cmake support does this is it creates a python/lcm/ directory in the build directory which contains both the library and __init__.py. As it currently is, a user would have to import _lcm which isn't in line with the docs.
You might be able to recreate something similar here by removing these extension_module calls to an lcm subdir. Then you might be able to copy in the __init__.py by using a meson function like configure_file, although you might have to resort to one of the newer features like the filesystem module.
…onforming directory i.e. bin
fix python library installation and remove comment
…instead of unused option
…ring back hacky platlib dir for compatibility with ubuntu 22.04's ancient meson version
…atures are disabled
Zinput
force-pushed
the
feature/meson_support
branch
from
531fabc
to
daedc52
Compare
Add support for building LCM with Meson and add workflow tests for LCM built with Meson. Supports LCM core features. Meson support will make it easier to use LCM as a subproject in other Meson projects.
Does not support building Python, Java, and Lua modules from Meson for now (moved to new branch to simplify pull request).