[Xfce4-commits] <midori:master> Avoid pre- and suffix with extension default configs
Christian Dywan
noreply at xfce.org
Thu Oct 20 23:46:01 CEST 2011
Updating branch refs/heads/master
to 3f77b9f300a918a598536e22987128f8458ab054 (commit)
from b29ee13da66591fa3129015855f8957e30daa78a (commit)
commit 3f77b9f300a918a598536e22987128f8458ab054
Author: Christian Dywan <christian at twotoasts.de>
Date: Thu Oct 20 23:39:28 2011 +0200
Avoid pre- and suffix with extension default configs
So this mainly matters to package maintainers, as the
configuration files for extensions have pre- and suffix
according to library naming of the platform. Basically
this only ever worked on Linux and Windows.
Now default configs are always installed with plain names.
midori/midori-extension.c | 11 ++++++++++-
wscript | 14 ++------------
2 files changed, 12 insertions(+), 13 deletions(-)
diff --git a/midori/midori-extension.c b/midori/midori-extension.c
index 4eb83ef..634462c 100644
--- a/midori/midori-extension.c
+++ b/midori/midori-extension.c
@@ -294,7 +294,16 @@ midori_extension_activate_cb (MidoriExtension* extension,
if (error->code == G_FILE_ERROR_NOENT)
{
gchar* filename = g_object_get_data (G_OBJECT (extension), "filename");
- gchar* folder = g_strconcat ("extensions/", filename, NULL);
+ gchar* folder;
+ if (g_str_has_prefix (filename, "lib"))
+ filename = &filename[strlen ("len")];
+ if (g_str_has_suffix (filename, G_MODULE_SUFFIX))
+ filename = g_strndup (filename,
+ strlen (filename) - strlen ("." G_MODULE_SUFFIX));
+ else
+ filename = g_strdup (filename);
+ folder = g_strconcat ("extensions/", filename, NULL);
+ g_free (filename);
katze_assign (config_file,
sokoke_find_config_filename (folder, "config"));
g_free (folder);
diff --git a/wscript b/wscript
index d8abd15..36051a8 100644
--- a/wscript
+++ b/wscript
@@ -476,23 +476,13 @@ def build (bld):
bld.install_files ('${MDATADIR}/' + APPNAME + '/res', 'data/autosuggestcontrol.js')
bld.install_files ('${MDATADIR}/' + APPNAME + '/res', 'data/autosuggestcontrol.css')
- # FIXME: Determine the library naming for other platforms
- if bld.env['platform'] == 'win32':
+ if 1:
extensions = os.listdir ('data/extensions')
for extension in extensions:
- folder = 'lib' + extension + '.dll'
source = 'data/extensions/' + extension + '/config'
if os.path.exists (source):
bld.install_files ('${SYSCONFDIR}/xdg/' + APPNAME + \
- '/extensions/' + folder, source)
- elif Options.platform == 'linux':
- extensions = os.listdir ('data/extensions')
- for extension in extensions:
- folder = 'lib' + extension + '.so'
- source = 'data/extensions/' + extension + '/config'
- if os.path.exists (source):
- bld.install_files ('${SYSCONFDIR}/xdg/' + APPNAME + \
- '/extensions/' + folder, source)
+ '/extensions/' + extension, source)
if Options.commands['check'] or bld.env['tests']:
bld.add_subdirs ('tests')
More information about the Xfce4-commits
mailing list