gconf: fix saving of settings when config folder doesnt exist

In some circumstances, gconf isn't able to save configurations
because ~/.config folder aka root_dir doesn't exist.

For example when saving settings using matchbox-appearance,
the following error is shown:

GConf Error: Configuration server couldn't be contacted: D-BUS error:
Can't overwrite existing read-only value: Value for
`/desktop/poky/interface/font_name' set in a read-only source at the
front of your configuration path

This issue was not seen before because ~/.config directory is shared
between several packages and one of those packages usually creates it
by the time gconf wants to use it.

This patch makes sure that gconf creates the .config directory if it
doesn't exist, along with the gconf directory inside it.

[YOCTO #12632]

(From OE-Core rev: 4d16fa05e47ccc8425ebb085c295d7d8dca6b2e6)

Signed-off-by: Alejandro Hernandez <alejandr@xilinx.com>
Signed-off-by: Jaewon Lee <jaewon.lee@xilinx.com>
Signed-off-by: Ross Burton <ross.burton@intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
This commit is contained in:
Jaewon Lee 2018-04-10 14:26:47 -07:00 committed by Richard Purdie
parent 1ab6bfd0ee
commit 7ec063a3ad
2 changed files with 29 additions and 0 deletions

View File

@ -0,0 +1,28 @@
Upstream-Status: Pending
In some circumstances, gconf isn't able to save configurations
because ~/.config folder aka root_dir doesn't exist.
This issue was not seen before because ~/.config directory is shared
between several packages and one of those packages usually creates it
by the time gconf wants to use it.
This patch makes sure that gconf creates the .config directory if it
doesn't exist, along with the gconf directory inside it.
Signed-off-by: Jaewon Lee <jaewon.lee@xilinx.com>
Signed-off-by: Alejandro Hernandez <alejandr@xilinx.com>
Index: GConf-3.2.6/backends/markup-backend.c
===================================================================
--- GConf-3.2.6.orig/backends/markup-backend.c
+++ GConf-3.2.6/backends/markup-backend.c
@@ -276,7 +276,7 @@ resolve_address (const char *address,
/* dir_mode without search bits */
file_mode = dir_mode & (~0111);
}
- else if (g_mkdir (root_dir, dir_mode) < 0)
+ else if (g_mkdir_with_parents (root_dir, dir_mode) < 0)
{
/* Error out even on EEXIST - shouldn't happen anyway */
gconf_set_error (err, GCONF_ERROR_FAILED,

View File

@ -11,6 +11,7 @@ inherit gnomebase gtk-doc gettext gobject-introspection gio-module-cache
SRC_URI = "${GNOME_MIRROR}/GConf/${@gnome_verdir("${PV}")}/GConf-${PV}.tar.xz;name=archive \
file://remove_plus_from_invalid_characters_list.patch \
file://unable-connect-dbus.patch \
file://create_config_directory.patch \
"
SRC_URI[archive.md5sum] = "2b16996d0e4b112856ee5c59130e822c"