diff options
author | Ralf Habacker <ralf.habacker@freenet.de> | 2015-09-20 13:47:23 +0200 |
---|---|---|
committer | Ralf Habacker <ralf.habacker@freenet.de> | 2015-09-20 13:48:28 +0200 |
commit | 36d864e4697bc6e1eeffb32092bd78b108362ab5 (patch) | |
tree | 2a1be313765cf5d74a783f03173aadb192cc8045 | |
parent | 0a6310fe16d56cfcf1d3eea46b71a18ad5fe02a1 (diff) |
Make Windows dbus-daemon look for the config file we install
The canonical location for bus setup changed from
${sysconfdir}/dbus-1 to ${datadir}/dbus-1 (or their CMake
equivalents) in version 1.9.18.
Also stop trying to use bus/session.conf from the build tree,
which will not work if our ${prefix} contains an older
${sysconfdir}/dbus-1/session.conf.
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=92028
Reviewed-by: Ralf Habacker <ralf.habacker@freenet.de>
-rw-r--r-- | dbus/dbus-sysdeps-util-win.c | 22 | ||||
-rw-r--r-- | dbus/dbus-sysdeps-win.c | 65 | ||||
-rw-r--r-- | dbus/dbus-sysdeps-win.h | 4 |
3 files changed, 22 insertions, 69 deletions
diff --git a/dbus/dbus-sysdeps-util-win.c b/dbus/dbus-sysdeps-util-win.c index e83c5397..57c353e9 100644 --- a/dbus/dbus-sysdeps-util-win.c +++ b/dbus/dbus-sysdeps-util-win.c @@ -1647,6 +1647,28 @@ _dbus_get_standard_system_servicedirs (DBusList **dirs) return TRUE; } +static dbus_bool_t +_dbus_get_config_file_name (DBusString *str, + const char *basename) +{ + DBusString tmp; + + if (!_dbus_string_append (str, _dbus_windows_get_datadir ())) + return FALSE; + + _dbus_string_init_const (&tmp, "dbus-1"); + + if (!_dbus_concat_dir_and_file (str, &tmp)) + return FALSE; + + _dbus_string_init_const (&tmp, basename); + + if (!_dbus_concat_dir_and_file (str, &tmp)) + return FALSE; + + return TRUE; +} + /** * Append the absolute path of the system.conf file * (there is no system bus on Windows so this can just diff --git a/dbus/dbus-sysdeps-win.c b/dbus/dbus-sysdeps-win.c index b3b459f8..69644b00 100644 --- a/dbus/dbus-sysdeps-win.c +++ b/dbus/dbus-sysdeps-win.c @@ -3330,71 +3330,6 @@ _dbus_get_install_root(char *prefix, int len) return TRUE; } -/** - find config file either from installation or build root according to - the following path layout - install-root/ - bin/dbus-daemon[d].exe - etc/<config-file>.conf *or* etc/dbus-1/<config-file>.conf - (the former above is what dbus4win uses, the latter above is - what a "normal" Unix-style "make install" uses) - - build-root/ - bin/dbus-daemon[d].exe - bus/<config-file>.conf -*/ -dbus_bool_t -_dbus_get_config_file_name(DBusString *config_file, char *s) -{ - char path[MAX_PATH*2]; - int path_size = sizeof(path); - - if (!_dbus_get_install_root(path,path_size)) - return FALSE; - - if(strlen(s) + 4 + strlen(path) > sizeof(path)-2) - return FALSE; - strcat(path,"etc\\"); - strcat(path,s); - if (_dbus_file_exists(path)) - { - // find path from executable - if (!_dbus_string_append (config_file, path)) - return FALSE; - } - else - { - if (!_dbus_get_install_root(path,path_size)) - return FALSE; - if(strlen(s) + 11 + strlen(path) > sizeof(path)-2) - return FALSE; - strcat(path,"etc\\dbus-1\\"); - strcat(path,s); - - if (_dbus_file_exists(path)) - { - if (!_dbus_string_append (config_file, path)) - return FALSE; - } - else - { - if (!_dbus_get_install_root(path,path_size)) - return FALSE; - if(strlen(s) + 4 + strlen(path) > sizeof(path)-2) - return FALSE; - strcat(path,"bus\\"); - strcat(path,s); - - if (_dbus_file_exists(path)) - { - if (!_dbus_string_append (config_file, path)) - return FALSE; - } - } - } - return TRUE; -} - /* See comment in dbus-sysdeps-unix.c */ dbus_bool_t _dbus_lookup_session_address (dbus_bool_t *supported, diff --git a/dbus/dbus-sysdeps-win.h b/dbus/dbus-sysdeps-win.h index a5367705..e9b30d7a 100644 --- a/dbus/dbus-sysdeps-win.h +++ b/dbus/dbus-sysdeps-win.h @@ -85,10 +85,6 @@ _dbus_win_sid_to_name_and_domain (dbus_uid_t uid, dbus_bool_t _dbus_file_exists (const char *filename); DBUS_PRIVATE_EXPORT -dbus_bool_t _dbus_get_config_file_name(DBusString *config_file, - char *s); - -DBUS_PRIVATE_EXPORT dbus_bool_t _dbus_get_install_root(char *prefix, int len); void _dbus_threads_windows_init_global (void); |