From 944d5467e1655aac20a14325631df6daccaf5804 Mon Sep 17 00:00:00 2001 From: Jan Tojnar Date: Sun, 3 Mar 2019 01:13:46 +0100 Subject: [PATCH] Fix building on Nix ./configure.py tries to find dbus-python header in dbus-1 includedir obtained from pkg-config or from --dbus flag. Unfortunately, when supplied, it also uses the flag for locating dbus-1 headers. This fails on Nix, since every package is installed into its own immutable tree so we cannot use a single directory for both dbus-python and dbus-1. We can fix this by using pkg-config for finding dbus-python headers too. Additionally, the build system also tries to install the dbus support module to dbus-python tree. Often, it is possible to handle this in pkgconfig as well [1] but unfortunately, dbus-python does not export the moduledir in its pc file so I have decided to solve this with an extra configure flag. [1]: https://www.bassi.io/articles/2018/03/15/pkg-config-and-paths/ --- configure.py | 13 +++++++++++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/configure.py b/configure.py index c6663e4..65e7da7 100644 --- a/configure.py +++ b/configure.py @@ -905,6 +905,9 @@ class TargetConfiguration: if opts.pydbusincdir is not None: self.pydbus_inc_dir = opts.pydbusincdir + if opts.pydbusmoduledir is not None: + self.pydbus_module_dir = opts.pydbusmoduledir + if opts.pyuicinterpreter is not None: self.pyuic_interpreter = opts.pyuicinterpreter @@ -1191,6 +1194,11 @@ def create_optparser(target_config): metavar="DIR", help="the directory containing the dbus/dbus-python.h header is " "DIR [default: supplied by pkg-config]") + g.add_option("--dbus-moduledir", dest='pydbusmoduledir', type='string', + default=None, action='callback', callback=store_abspath, + metavar="DIR", + help="the directory where dbus support module will be installed to" + "DIR [default: obtained from dbus.mainloop python module]") p.add_option_group(g) # Installation. @@ -2277,7 +2285,7 @@ def check_dbus(target_config, verbose): inform("Checking to see if the dbus support module should be built...") - cmd = 'pkg-config --cflags-only-I --libs dbus-1' + cmd = 'pkg-config --cflags-only-I --libs dbus-1 dbus-python' if verbose: sys.stdout.write(cmd + "\n") @@ -2307,7 +2315,8 @@ def check_dbus(target_config, verbose): inform("The Python dbus module doesn't seem to be installed.") return - target_config.pydbus_module_dir = dbus.mainloop.__path__[0] + if target_config.pydbus_module_dir == '': + target_config.pydbus_module_dir = dbus.mainloop.__path__[0] # Try and find dbus-python.h. We don't use pkg-config because it is broken # for dbus-python (at least for versions up to and including v0.81.0). diff --git a/project.py b/project.py index fe9fbce..9ae1ca1 100644 --- a/project.py +++ b/project.py @@ -261,7 +261,7 @@ del find_qt dbus_lib_dirs = [] dbus_libs = [] - args = ['pkg-config', '--cflags-only-I', '--libs dbus-1'] + args = ['pkg-config', '--cflags-only-I', '--libs dbus-1', 'dbus-python'] for line in self.read_command_pipe(args, fatal=False): for flag in line.strip().split():