| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
| |
When used with the "fast" (optimistic concurrency) API, the engine library
emits a change notification local to a process after a change is initiated
from that process. Previously, it would emit this notification even if the
newly written value was the same as the previous value (according to that
process's view of the state). After this change, the local change
notification is not sent unless the new value is different from the
current value (as seen by that process).
|
|
|
|
|
|
|
| |
On FreeBSD using the default GNU BFD linker, the use of --version-script
fails unless the symbols `__progname` and `environ` are included. This
change includes them in the symbol map, which should allow the build to
succeed on FreeBSD.
|
| |
|
|
|
|
| |
This will allow it to be used for other binaries
|
|
|
|
| |
succeeded
|
| |
|
| |
|
|
|
|
| |
when appropriate
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
With recent changes made to GLib[1] invoking g_test_dbus_down()
will fail after a timeout if the GDBusConnection object for the
session bus leaks.
Note that this new test will fail as the current code will leak a
cached GDBusConnection. Fixes will be added in follow-up patches.
[1] https://gitlab.gnome.org/GNOME/glib/merge_requests/963
Signed-off-by: Andre Moreira Magalhaes <andre@endlessm.com>
|
|
|
|
|
|
|
| |
Fix linking with LLD by using weak symbols in gvdb, and update dconf
code to make it compatible with gvdb upstream changes.
Fixes https://gitlab.gnome.org/GNOME/dconf/issues/47
|
|
|
|
|
|
|
|
|
|
|
| |
When a bus address is unset the default value will be used. While the
session bus address is overwritten during tests, this not the case for
system bus.
Use empty invalid addresses as a safer default to avoid accidental
connections to the external buses.
Issue #51.
|
|\
| |
| |
| |
| | |
Tests: remove bug prone usage of dlsym and RTDL_NEXT
See merge request GNOME/dconf!37
|
| |
| |
| |
| | |
that need to be mocked out
|
| |
| |
| |
| |
| |
| |
| |
| | |
Due to the hacks used for large file support, wrapping pwrite is error
prone and can end up calling the wrong function. Currently, "pwrite" is
called instead of "pwrite64" on 32-bit ARM, causing the test to fail.
This commit attempts to determine the correct symbol to call from the
wrapper.
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
If load command attempts to change one of non-writable keys, the whole
operation fails with an error and no changes are made. Add an `-f`
option to the load command that skips non-writable keys and proceeds
with remaining changes.
Closes issue #1.
|
|/ |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
| |
Fixes issue #42.
|
|\
| |
| |
| |
| | |
Add further integration tests
See merge request GNOME/dconf!33
|
| | |
|
| | |
|
| | |
|
| | |
|
| |
| |
| |
| |
| |
| | |
Reduce the number of in-flight requests to one, so as to increase
chances of merging pending requests. Drop the in-flight queue since
it is no longer useful, replacing it with optional changeset.
|
|/
|
|
|
|
|
|
|
|
| |
Instead of queuing changes and sending them to a writer one by one,
coalesce them into a single changeset.
Coalescing changes requires a little bit more work on the client side,
see implementation of `dconf_changeset_change`, but it has chance to
substantially reduce the total amount of work necessary and avoid costly
disk writes.
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
| |
This reverts commit b799b08ec4e5978ab8db233a9da1532bf7918324.
|
|
|
|
| |
This reverts commit bf103302332603a637f1c33e603c93b82aa1dc0b.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The dconf-service internal static library depends on generated files,
and we need to make sure that those files are available by the time we
build the tests that depend on them, as Meson builds everything in
parallel. If we don't do that, in cases of massive parallelised
builders, we will end up with errors like:
```
ccache cc -Itests/tests@@writer@exe -Itests -I../tests -I. -I../ -Itests/../service -I../tests/../service -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include -fdiagnostics-color=always -pipe -D_FILE_OFFSET_BITS=64 -Wall -Winvalid-pch -O0 -g '-DSYSCONFDIR="/etc"' -fno-common -Wmissing-prototypes -Wwrite-strings -O2 -g -m64 -mtune=generic '-DSRCDIR="/ostbuild/source/dconf/tests"' -MD -MQ 'tests/tests@@writer@exe/writer.c.o' -MF 'tests/tests@@writer@exe/writer.c.o.d' -o 'tests/tests@@writer@exe/writer.c.o' -c ../tests/writer.c
../tests/writer.c:24:37: fatal error: service/dconf-generated.h: No such file or directory
```
Reviewed-by: nobody
Signed-off-by: Emmanuele Bassi <ebassi@gnome.org>
|
|
|
|
|
|
|
| |
Otherwise the tests fail now that we’re compiling with
G_LOG_USE_STRUCTURED.
Signed-off-by: Philip Withnall <withnall@endlessm.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Use g_test_subprocess() and g_test_trap_subprocess() instead. This
requires splitting up some of the unit tests, since subprocesses are
handled at a per-test level, and you can’t trivially fork multiple
different subprocesses from a single test (like you could with fork()).
While fork was safe to use on POSIX (and dconf only targets POSIX), the
fact that we had to redefine GLIB_VERSION_MIN_REQUIRED to hide the
deprecation errors was not doing wonders for the maintainability of the
tests.
Signed-off-by: Philip Withnall <withnall@endlessm.com>
|
|
|
|
| |
Signed-off-by: Philip Withnall <withnall@endlessm.com>
|
|
|
|
|
|
|
| |
This is useful for JHBuild environments and systems that don't want
to use /etc/dconf.
https://bugzilla.gnome.org/show_bug.cgi?id=739299
|
|
|
|
| |
Use g_assert_false instead of g_assert in unit tests
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
subscriptions to the same path
Remove accidental whitespace change
Simplify branching in watch_fast and unwatch_fast
Indentation fixes
Store the subscription counts directly in the hash table pointer instead of mallocing ints
Add documentation comments for new utility functions
|
|
|
|
|
|
| |
Having trailing commas in place means that if another source file or
parameter is added to the end of the list or function, there won't
be noise when adding the comma to the line above.
|
|
|
|
|
|
|
|
|
| |
The current meson build files make heavy use of meson's
`project_name` function. However this makes difficult for any
developer to find for given program/library/file names. The project
name is also never going to change.
Due to this reason these calls have been changed for `dconf` itself.
|
|
|
|
|
|
|
|
|
| |
meson is able to generate internal dependencies for handling built
libraries. These internal dependencies depend on other dependencies
as well, based on the includes exposed by their headers.
This have been fixed by using proper internal dependencies for these
libraries.
|
|
|
|
|
|
|
|
|
|
|
|
| |
I can’t work out why they’re failing, so disable them for the moment.
The immediate error they give is about not being able to start a bus
when $DISPLAY isn’t defined — but if you switch them to run under
GTestDBus, they fail more cryptically with a failure to find a bus
socket.
This can be reapproached later.
Signed-off-by: Philip Withnall <withnall@endlessm.com>
|
|
|
|
| |
Signed-off-by: Philip Withnall <withnall@endlessm.com>
|
|
|
|
|
|
|
| |
/usr/include/features.h:183:3: warning: #warning "_BSD_SOURCE and _SVID_SOURCE are deprecated, use _DEFAULT_SOURCE" [-Wcpp]
# warning "_BSD_SOURCE and _SVID_SOURCE are deprecated, use _DEFAULT_SOURCE"
Signed-off-by: Philip Withnall <withnall@endlessm.com>
|
|
|
|
| |
for the root path
|