commit ec2a7cc710988176e3f4a68c7dd8c81538037550 Author: Matthias Clasen Date: Wed Aug 17 12:21:13 2016 -0400 2.49.5 NEWS | 41 +++++++++++++++++++++++++++++++++++++++++ configure.ac | 2 +- 2 files changed, 42 insertions(+), 1 deletion(-) commit b4326bc348cf57274e8c97e9a2663a71958d22e8 Author: Philip Withnall Date: Wed Aug 17 12:41:26 2016 +0200 gmessages: Drop unnecessary stdio.h include This was needed for an earlier version of the structured logging work, but not the latest. glib/gmessages.h | 1 - 1 file changed, 1 deletion(-) commit 15b315b472184cbeb842256defb3d4d7d0dbd4fd Author: Philip Withnall Date: Tue Aug 16 17:46:30 2016 +0200 gdbus-codegen: Allow '@since: UNRELEASED' in documentation comments Previously, this would not work, as it would result in comparing the order of a string and an integer. Make it work, and make 'UNRELEASED' compare higher than other versions so it's always treated as the latest version. 'UNRELEASED' is commonly used by maintainers to highlight new API while it's being prototyped, until they know which version it will actually be released in. At the time of release, they replace all 'UNRELEASED' strings in git with the new version number. An example of this usage is here: https://gitlab.com/walbottle/walbottle/commit/d380ac6a2a4a3f4004b805b755c2e6cd041180dc#9208ee267cb05db1afd3a5c323d71e51db489447_7619_7656 https://bugzilla.gnome.org/show_bug.cgi?id=769995 gio/gdbus-2.0/codegen/utils.py | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) commit 6d1178b2d923963588bc3430a1de49eb9fc8bc89 Author: Allison Lortie Date: Tue Aug 9 12:46:30 2016 +0200 gregex: loosen behaviour testing Circa 8.38, upstream PCRE (intentionally?) changed behaviour with respect to whether options set with expressions like "(?i)" at the top-level were reported via the pcre_fullinfo() API as having been requested during compilation. GLib contained a test that verified that these options were indeed reported as if they had been provided as flags on the API. Remove that check, and document the no-longer-deterministic behaviour. https://bugzilla.gnome.org/show_bug.cgi?id=767240 glib/gregex.c | 4 ++++ glib/tests/regex.c | 13 ++++++------- 2 files changed, 10 insertions(+), 7 deletions(-) commit 037719c27c541e5e529c7f0dc8a14792a26303c9 Author: Simon McVittie Date: Sun Aug 14 15:36:17 2016 +0100 gdbus-example-unix-fd-client: avoid strftime %c specifier gcc 6 warns (fatally, by default) that %c only uses a 2-digit year in some locales. The precise format does not seem to be important for this sample code, so use ISO 8601 instead of suppressing the warning with a pragma. Signed-off-by: Simon McVittie Reviewed-by: Colin Walters Bug: https://bugzilla.gnome.org/show_bug.cgi?id=768453 gio/tests/gdbus-example-unix-fd-client.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) commit 634990a37403e0c8fb642adcd1130e19dbcf546f Author: Jordi Mas Date: Sun Aug 14 17:29:30 2016 +0200 Update Catalan translation po/ca.po | 43 ++++++++++++++++--------------------------- 1 file changed, 16 insertions(+), 27 deletions(-) commit 5707c91a56fbf436d727dd700426296cd651aeaf Author: Philip Withnall Date: Sat Aug 13 10:31:26 2016 +0200 win_iconv: Fix some file handler leaks on exit I realise this is a contradiction in terms, but it keeps code analysis tools happy. As spotted by cppcheck, which could not attend GUADEC, but sends everyone its best wishes anyway. glib/win_iconv.c | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) commit f1eeb7cf8c14b30e6267ed6aa663c0f617f55693 Author: Philip Withnall Date: Sat Aug 13 10:29:31 2016 +0200 tests: Fix GSList test to not dereference off the end of an array This looks like a typo or copypasta error. As spotted by cppcheck, which is the code analysis tool GLib deserves, but not the one it needs right now. glib/tests/slist.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) commit ba874a8580a1ff4af921ffcf42512101b1e7ec28 Author: Philip Withnall Date: Sat Aug 13 10:28:24 2016 +0200 tests: Add an assertion to guard against unexpected flags This should not change the behaviour of the test, unless the test's behaviour is changed in future. As spotted by cppcheck, which has impeccable taste and a flair for the unexpected. glib/tests/markup-parse.c | 2 ++ 1 file changed, 2 insertions(+) commit c868d9879c7b3ae118283849bde3f160bc55cc69 Author: Philip Withnall Date: Sat Aug 13 10:27:54 2016 +0200 glib-mirroring-tab: Fix string format placeholder to be unsigned Because the argument is unsigned. As spotted by cppcheck, which never sleeps. glib/glib-mirroring-tab/gen-mirroring-tab.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) commit 03b43f3f6a985a31391d993fe317295a9fe1200d Author: Philip Withnall Date: Sat Aug 13 10:27:28 2016 +0200 docs: Fix early termination of some documentation comments As spotted by cppcheck, which has an eye for these things. glib/docs.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) commit 5cdf0efaabb612908a08105b3a1826380c785234 Author: Philip Withnall Date: Sat Aug 13 10:24:23 2016 +0200 tests: Fix a typo on a dereferencing assignment Otherwise the assignment is pointless. Spotted by cppcheck. gio/tests/gdbus-peer.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) commit a42b6eab66bdcb4f551bd58cf61e956ae3c8fd7d Author: Philip Withnall Date: Sat Aug 13 10:24:04 2016 +0200 gresource-tool: Clarify precedence of operations As suggested by cppcheck, which knows more than me. gio/gresource-tool.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) commit a5044a8e78eeab79172a28991c4676e19b2bdbfa Author: Philip Withnall Date: Sun Aug 7 11:24:41 2016 +0100 build: Fix SystemTap build to disable semaphores as before At some point, upstream SystemTap changed from using a STAP_HAS_SEMAPHORES preprocessor variable for this, to using _SDT_HAS_SEMAPHORES instead. We need to update our build system to disable that as well. The original discussion about use of semaphores is here: https://bugzilla.gnome.org/show_bug.cgi?id=606044 This was breaking the build with -flto enabled, either because -flto doesn’t work with semaphores. https://bugzilla.gnome.org/show_bug.cgi?id=768198 gio/Makefile.am | 5 ++++- glib/Makefile.am | 5 ++++- gobject/Makefile.am | 5 ++++- 3 files changed, 12 insertions(+), 3 deletions(-) commit 0e3f968a2e9d35b4c087128c9819f07ef9e4ce99 Author: Philip Withnall Date: Fri Aug 12 15:58:39 2016 +0200 gio: Fix use of Docbook in documentation comment This is literally so last year. gio/ginetsocketaddress.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) commit a079de930e9a2311ec7fb48e65db81bada3b5dfa Author: Philip Withnall Date: Fri Aug 12 11:53:19 2016 +0200 gmessages: Add doc cross-refs about enabling structured logging Ensure that all the old log handler documentation includes cross-references to a new section about how to enable structured logging (tl;dr: #define G_LOG_USE_STRUCTURED). https://bugzilla.gnome.org/show_bug.cgi?id=769785 glib/gmessages.c | 82 +++++++++++++++++++++++++++++++++++++++++++++++++------- 1 file changed, 72 insertions(+), 10 deletions(-) commit f7825f98e76cfee53e430d3afb424994c1acab6a Author: Philip Withnall Date: Fri Aug 12 11:05:07 2016 +0200 gmessages: Drop one of the documentation sections It was short, unhelpful, and easy to confuse with the longer and more informative documentation sections. https://bugzilla.gnome.org/show_bug.cgi?id=769785 glib/gmessages.c | 15 +-------------- 1 file changed, 1 insertion(+), 14 deletions(-) commit 599bb6eaf3e3f7a42742b036c5e2c4e6e009229c Author: Jordi Mas Date: Fri Aug 12 04:41:31 2016 +0200 Update Catalan translation po/ca.po | 1362 +++++++++++++++++++++++++++++++++++++++++++++++--------------- 1 file changed, 1042 insertions(+), 320 deletions(-) commit 3b20acecc6b45c93f544ca0901cc2d5482303e0b Author: Dušan Kazik Date: Mon Aug 8 11:52:47 2016 +0000 Updated Slovak translation po/sk.po | 1312 +++++++++++++++++++++++++++++++++++++++++++++++--------------- 1 file changed, 1000 insertions(+), 312 deletions(-) commit ad2da25693caf3ebf9e9126822e0580ca005a43b Author: Yosef Or Boczko Date: Sun Aug 7 13:42:32 2016 +0300 Updated Hebrew translation po/he.po | 340 ++++++++++++++++++++++++++++++++++----------------------------- 1 file changed, 181 insertions(+), 159 deletions(-) commit 371078b6a3af9985ae9efc7e146d718d6cd1cfff Author: Philip Withnall Date: Wed Aug 3 22:11:57 2016 +0100 gmessages: Expand documentation on log domains and G_MESSAGES_DEBUG Give some example log domains, and recommend that G_MESSAGES_DEBUG is used universally as the way to enable debug output (rather than having a separate environment variable per library). https://bugzilla.gnome.org/show_bug.cgi?id=682794 glib/gmessages.c | 39 +++++++++++++++++++++++++++++++++++---- 1 file changed, 35 insertions(+), 4 deletions(-) commit 0e132b8ac5b6d77e970850e86749fa7f38293cd7 Author: Philip Withnall Date: Wed Aug 3 21:55:53 2016 +0100 gmessages: Document g_test_expect_message() doesn’t like structured logs It’s effectively deprecated if G_LOG_USE_STRUCTURED is defined, or if the structured logging API is used directly. See the documentation for rationale. https://bugzilla.gnome.org/show_bug.cgi?id=769486 glib/gmessages.c | 46 ++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 46 insertions(+) commit 4e66036b82e17364dc94c711e6cb2076676bd727 Author: Philip Withnall Date: Wed Aug 3 21:28:28 2016 +0100 Revert "Make g_test_expect_message work for structured logs" This reverts commit df02e8f47cac4d2c550491c9de633233218c7415. We are going to make g_test_expect_message() only work for the old logging API. https://bugzilla.gnome.org/show_bug.cgi?id=769486 glib/gmessages.c | 98 +++++++++++++++++--------------------------------------- 1 file changed, 30 insertions(+), 68 deletions(-) commit 2a5d9f88984a4c708665b3b5e37151ad8d2bfda2 Author: Philip Withnall Date: Wed Aug 3 21:28:06 2016 +0100 Revert "Add a test for expected messages with structured logging" This reverts commit bc40c7a05cbd0e7c9b97ce86cccb6a4b78c2e56a. We are going to make g_test_expect_message() only work for the old logging API. https://bugzilla.gnome.org/show_bug.cgi?id=769486 glib/tests/testing.c | 27 --------------------------- 1 file changed, 27 deletions(-) commit 5c3205f2fb0f8bf4bbce7e1e532e834ad36da633 Author: Philip Withnall Date: Thu Aug 4 10:35:27 2016 +0100 gmessages: Don’t require is_journald() call before writer_journald() g_log_writer_is_journald() works out whether the process’ stderr or stdout are redirected to journald. While the default writer function uses this in conjunction with g_log_writer_journald(), that does not always have to be the case — other writer functions might want to always write to the journal, and never write to stderr (for example). Consequently, automatically open the journal socket in writer_journald(), rather than is_journald(). https://bugzilla.gnome.org/show_bug.cgi?id=769507 glib/gmessages.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) commit 5ff8579527e75a9077c7152d4878d282123ddb45 Author: Emmanuele Bassi Date: Thu Aug 4 09:33:25 2016 +0100 Fix debug builds under MSVC The C spec leaves conditional evaluation inside a macro expansion as undefined behaviour. This means we cannot use constructs like: GOBJECT_IF_DEBUG(OBJECTS, { ... #ifdef BLAH ... #endif ...}); Because compilers are entirely justified to ignore the conditional, or, like in the case of MSVC, error out. https://bugzilla.gnome.org/show_bug.cgi?id=769504 gobject/gobject.c | 19 ++++++++++++++----- 1 file changed, 14 insertions(+), 5 deletions(-) commit 64f6d14d0c486da821ad14e7a35ec26ec65ea507 Author: Aurimas Černius Date: Wed Aug 3 22:07:51 2016 +0300 Updated Lithuanian translation po/lt.po | 394 +++++++++++++++++++++++++++++++-------------------------------- 1 file changed, 192 insertions(+), 202 deletions(-) commit 04bbf9df377bac2d2dd920c528756b1fd32da547 Author: Philip Withnall Date: Wed Aug 3 18:46:29 2016 +0100 gdatagrambased: Clarify that connection-oriented ≠ stream-based GDatagramBased allows connection-oriented and connection-less sockets, but does not allow stream-based sockets (because it’s datagram-based). So it supports SCTP and UDP, but not TCP. Clarify that in the documentation, and people sometimes confuse connection-oriented with stream-based, due to the prevalence of TCP. gio/gdatagrambased.c | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) commit 555c5a63767e3ca988639bf69932a1fc8d017c1b Author: Colin Walters Date: Wed Aug 3 11:34:43 2016 -0400 gvariant: Fix typo in previous commit glib/gvariant.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) commit 1be86b3d185cfc64bd355c1afd958b1757a1761a Author: Yury Usishchev Date: Thu Jul 28 00:41:38 2016 +0300 glib/gvariant.c: Fix NULL pointer check in is_valid_heap_iter https://bugzilla.gnome.org/show_bug.cgi?id=769245 glib/gvariant.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) commit bc40c7a05cbd0e7c9b97ce86cccb6a4b78c2e56a Author: Matthias Clasen Date: Tue Aug 2 23:15:27 2016 -0400 Add a test for expected messages with structured logging This tests the fix in the previous commit. glib/tests/testing.c | 27 +++++++++++++++++++++++++++ 1 file changed, 27 insertions(+) commit df02e8f47cac4d2c550491c9de633233218c7415 Author: Matthias Clasen Date: Tue Aug 2 23:14:28 2016 -0400 Make g_test_expect_message work for structured logs This is used in GTK+ tests, and GTK+ has switched to G_LOG_USE_STRUCTURED, so we need this to keep working. glib/gmessages.c | 98 +++++++++++++++++++++++++++++++++++++++----------------- 1 file changed, 68 insertions(+), 30 deletions(-) commit 1b711d9b4addb9c488879deb3916f7d6d5048953 Author: Daniel Mustieles Date: Tue Aug 2 16:46:35 2016 +0200 Updated Spanish translation po/es.po | 128 ++++++++++++++++++++++++++++----------------------------------- 1 file changed, 56 insertions(+), 72 deletions(-) commit fbf72a31b13159cef980f17e711dd7600c08176e Author: Mario Sanchez Prada Date: Thu Jul 28 11:56:39 2016 -0600 Don't enable libmount by default for now This is causing trouble with flatpaks because the org.gnome.Platform runtime does not bundle libmount, while the org.gnome.Sdk does it. As this probably requires a change in the freedesktop.org Yocto base, we disable this support by default for now as a temporary measure until it can be properly reviewed by someone who knows those bits better, probably Alex Larsson. https://bugzilla.gnome.org/show_bug.cgi?id=769284 configure.ac | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) commit ed75b1623d4a51b16dc52a2044681b6d93e694d0 Author: Mario Sanchez Prada Date: Wed Jul 27 13:50:54 2016 -0600 gunixmounts: Ensure that libmount's context gets freed on early returns Otherwise, it will be leaked in case the relevant table (mtab, fstab) could not be retrieved for some reason. https://bugzilla.gnome.org/show_bug.cgi?id=769238 gio/gunixmounts.c | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) commit b28c76b0d82b22ccde08999c2bc70dff0283e053 Author: Mario Sanchez Prada Date: Wed Jul 27 13:50:26 2016 -0600 gunixmounts: Do not leak libmount tables in _g_get_unix_mounts() Use mnt_context_get_mtab instead of using mnt_context_get_table(), since that's the recommended way of accessing mtab/mountinfo information, and also because that way the (struct libmnt_table *) will get automatically deallocated when calling mnt_free_context() https://bugzilla.gnome.org/show_bug.cgi?id=769238 gio/gunixmounts.c | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) commit 7cf037ec1614878d859fb9050311d182aae6e8f5 Author: Chun-wei Fan Date: Wed Jul 27 16:37:09 2016 +0800 gmessages.h: Use G_STRFUNC instead of __func__ We have G_STRFUNC which takes care of __func__, which may not be available depending on compiler, so we ought to make use of it. https://bugzilla.gnome.org/show_bug.cgi?id=744456 glib/gmessages.h | 26 +++++++++++++------------- 1 file changed, 13 insertions(+), 13 deletions(-) commit 48317b199b44fd711c8eb54b102045a2c89695c4 Author: Chun-wei Fan Date: Wed Jul 27 16:06:21 2016 +0800 gio/gappinfo.c: Include gtask.h This avoids warning/error C4013 (aka implicit declaration of ...) as we need APIs from gtask.h. gio/gappinfo.c | 1 + 1 file changed, 1 insertion(+) commit 6d6b97aa59c3b97e4e584618ea23c04205e35811 Author: Chun-wei Fan Date: Tue Jul 26 12:32:42 2016 +0800 gmessages.c: Fix build on non-Linux journald is a part of systemd which is Linux-only at this time, so only compile the journald items on Linux only, and just return FALSE for g_log_writer_is_journald() and G_LOG_WRITER_UNHANDLED for g_log_writer_journald() on non-Linux. https://bugzilla.gnome.org/show_bug.cgi?id=744456 glib/gmessages.c | 15 +++++++++++++++ 1 file changed, 15 insertions(+) commit c1e8f705dd3043e6d001d49db00fe6ea12959613 Author: Matthias Clasen Date: Sat Jul 16 11:45:44 2016 -0400 Add async variant of g_app_info_launch_default_for_uri This is useful in the portalized case, when the portal may present an app chooser dialog to the user. https://bugzilla.gnome.org/show_bug.cgi?id=768752 docs/reference/gio/gio-sections.txt | 2 + gio/gappinfo.c | 256 +++++++++++++++++++++++++++++------- gio/gappinfo.h | 11 ++ 3 files changed, 219 insertions(+), 50 deletions(-) commit f885c4dd0de984ec5dd4a498cc7819070129f9b5 Author: Mario Sanchez Prada Date: Fri May 20 16:59:02 2016 +0100 Monitor /proc/self/mountinfo when using libmount https://bugzilla.gnome.org/show_bug.cgi?id=522053 gio/gunixmounts.c | 32 ++++++++++++++++++++++++++++---- 1 file changed, 28 insertions(+), 4 deletions(-) commit 97f799b53a46c9c440e40640417c8d083d8ec3ad Author: Mario Sanchez Prada Date: Fri May 20 15:58:56 2016 +0100 Refactor common code into create_unix_mount_point () https://bugzilla.gnome.org/show_bug.cgi?id=522053 gio/gunixmounts.c | 127 +++++++++++++++++++++++++++++++----------------------- 1 file changed, 72 insertions(+), 55 deletions(-) commit 44f4309e3b77ca130f62c66b2df08a8b7461c65f Author: Mario Sanchez Prada Date: Tue Jul 19 12:05:02 2016 +0100 Implemented _g_get_unix_mount_points() based on libmount https://bugzilla.gnome.org/show_bug.cgi?id=522053 gio/gunixmounts.c | 116 ++++++++++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 116 insertions(+) commit 2051ee767875d9b5c064583d9600a5f83c0f1bca Author: Mario Sanchez Prada Date: Tue Jul 19 13:31:09 2016 +0100 Refactor common code into create_unix_mount_entry () https://bugzilla.gnome.org/show_bug.cgi?id=522053 gio/gunixmounts.c | 93 +++++++++++++++++++++++-------------------------------- 1 file changed, 39 insertions(+), 54 deletions(-) commit 030594777adbe56fbb54c05bd0b1486b1a3d8c64 Author: Mario Sanchez Prada Date: Tue Jul 19 12:04:27 2016 +0100 Implemented _g_get_unix_mounts() based on libmount https://bugzilla.gnome.org/show_bug.cgi?id=522053 gio/gunixmounts.c | 107 ++++++++++++++++++++++++++++++++++++++++++++++++------ 1 file changed, 95 insertions(+), 12 deletions(-) commit 8d5cf2df102117607f65714a9c49a579ebedaa9b Author: Mario Sanchez Prada Date: Tue Jul 19 11:51:55 2016 +0100 Use libmount to find the path of the fstab file https://bugzilla.gnome.org/show_bug.cgi?id=522053 gio/gunixmounts.c | 7 +++++++ 1 file changed, 7 insertions(+) commit 4f9cddaeb8fc1b236ca49f08c1efa3fc9e75ecbf Author: Mario Sanchez Prada Date: Tue May 17 20:14:20 2016 +0100 Added autotools support for libmount Check whether libmount is available at configuration time and provide an option to explicitly enable or disable it, similar to libelf. https://bugzilla.gnome.org/show_bug.cgi?id=522053 configure.ac | 24 ++++++++++++++++++++++++ gio/Makefile.am | 5 +++++ 2 files changed, 29 insertions(+) commit 496c52ec7908701f1d9506114776c645df0a5c4b Author: Matthias Clasen Date: Mon Jul 25 12:31:52 2016 -0400 Avoid htole64 altogether GLib ships its own api for this, lets use it. glib/gmessages.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) commit ad285d9bd25419b5a4672cd4a6be77896b642992 Author: Matthias Clasen Date: Mon Jul 25 10:25:40 2016 -0400 Handle EINTR when sending logs to the journal Ray pointed out that we might well get interrupted here, and should not loose logs due to that. glib/gmessages.c | 13 +++++++++++-- 1 file changed, 11 insertions(+), 2 deletions(-) commit 34cb9c7de1ceaa854660f18544e4048cbdac4bf6 Author: Matthias Clasen Date: Mon Jul 25 10:15:08 2016 -0400 Explicitly include endian.h htole64 is defined here. https://bugzilla.gnome.org/show_bug.cgi?id=769139 glib/gmessages.c | 1 + 1 file changed, 1 insertion(+) commit a2d5d1f119ade1cba22c3c3802baa9afb3cb0ec0 Author: Matthias Clasen Date: Mon Jul 25 09:45:37 2016 -0400 Use SOCK_CLOEXEC if available Ray pointed out that we can avoid the race here. glib/gmessages.c | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) commit a5f209bc656177a25586c461631fc5ea5459449f Author: Thiago Macieira Date: Fri Jul 22 21:18:08 2016 -0700 Fall back for overflow-checked arithmetic with Intel compiler The Intel compiler does not have intrinsics like `__builtin_uadd_overflow`. https://bugzilla.gnome.org/show_bug.cgi?id=769104 glib/gtypes.h | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) commit df457c91109dc4b996053a6a517ae6f94233d781 Author: Matthias Clasen Date: Fri Jul 22 22:58:42 2016 -0400 Update logging tests We now treat the PRIORITY field like other string fields, and set its length to -1. Adapt the tests for this. glib/tests/logging.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) commit 6a07885a98b89f9f1bb627c5996ec3f6e74b5bb5 Author: Matthias Clasen Date: Fri Jul 22 22:56:26 2016 -0400 Drop libsystemd dependency Talk to the journal ourselves using sendmsg() instead of linking against libsystemd for sd_journal_sendv(). At the same time, we can also avoid excessive copying. The motivation for dropping the dependency is that we can then use structured logging e.g. in a flatpak sandbox where libsystemd may not be present in the runtime. The code here is inspired by similar code in libvirt. configure.ac | 15 ----- glib/gmessages.c | 195 +++++++++++++++++++++++++++++++++++++++++++------------ 2 files changed, 154 insertions(+), 56 deletions(-) commit ca775518d800408d1bf330fdf7f9578437130d19 Author: Matthias Clasen Date: Fri Jul 22 15:11:30 2016 -0400 Add more structured logging tests In particular, add a test to ensure that passing NULL as log_domain has no negative consequences. glib/tests/logging.c | 184 +++++++++++++++++++++++++++++++++------------------ 1 file changed, 118 insertions(+), 66 deletions(-) commit e7ee56dd1d6dd40cd330233d19783974d1aff57d Author: Matthias Clasen Date: Fri Jul 22 15:10:36 2016 -0400 Make g_log_set_writer_func work more than once It is fine to document that you shall only call this once, but for tests, it is important that we can call it multiple times. glib/gmessages.c | 1 - 1 file changed, 1 deletion(-) commit 97a38dc5865e554bd3567dea37f4cd33a5f50537 Author: Matthias Clasen Date: Fri Jul 22 15:09:31 2016 -0400 Fix a small typo glib/gtestutils.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) commit 38317cf74688ca32b565bf0c01ea84cb5d417fa2 Author: Emilio Pozuelo Monfort Date: Fri Jul 22 17:49:54 2016 +0200 Use a uint64 to unpack a 64 bit value https://bugzilla.gnome.org/show_bug.cgi?id=769089 gio/tests/gsettings.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) commit f414cddc1f765d06bbac9ddde0930f4567ccc70b Author: Ray Strode Date: Fri Jul 22 11:01:39 2016 -0400 gmessages: support NULL log domain It's possible that a project may not define the G_LOG_DOMAIN but still use g_log functions. In such cases, we now crash. This commit fixes that. https://bugzilla.gnome.org/show_bug.cgi?id=769087 glib/gmessages.c | 33 +++++++++++++++++++++++++-------- 1 file changed, 25 insertions(+), 8 deletions(-) commit f38845c11651405216d3caafdc9a5018d80b86b0 Author: Matthias Clasen Date: Thu Jul 21 14:52:46 2016 -0400 Expand g_log_structured docs a bit Mention two possible pitfalls that were pointed out by Ray Strode. glib/gmessages.c | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-) commit d5efa64539f71dc705826e2c0dc4ecb8354b9a77 Author: Matthias Clasen Date: Thu Jul 21 11:52:51 2016 -0400 Deal with lack of O_CLOEXEC Some platforms don't have it. We fall back to fcntl() in other places, so do it here as well. https://bugzilla.gnome.org/show_bug.cgi?id=769042 gio/gdocumentportal.c | 9 +++++++++ 1 file changed, 9 insertions(+) commit ad669500a60b87847ff16dd00c824748d6efb5af Author: Alberts Muktupāvels Date: Thu Jul 21 16:11:53 2016 +0300 gobject: add g_autoptr support for GTypeModule https://bugzilla.gnome.org/show_bug.cgi?id=769033 gobject/gtypemodule.h | 2 ++ 1 file changed, 2 insertions(+) commit d5a16fbd2d5132b3674817a45596a40a8d7bb71d Author: Krzesimir Nowak Date: Thu Jul 21 12:24:38 2016 +0200 gvariant: Avoid anonymous struct and union members C++ does not like them for various reasons. https://bugzilla.gnome.org/show_bug.cgi?id=766370 glib/gvariant.c | 12 ++++++------ glib/gvariant.h | 8 ++++---- 2 files changed, 10 insertions(+), 10 deletions(-) commit b9934f16b6b5ef44d3848185ae0b67459e4efbda Author: Daniel P. Berrange Date: Thu Jul 21 10:21:52 2016 +0100 gchecksum: annotate when G_CHECKSUM_SHA512 was introduced Currently the docs for GChecksumType are simpy annotated with 'Since 2.16' which is when GChecksumType was first introduced. No mention is made of the fact that the G_CHECKSUM_SHA512 constant was only added much later in 2.36. Signed-off-by: Daniel P. Berrange https://bugzilla.gnome.org/show_bug.cgi?id=769027 glib/gchecksum.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) commit d3111779481a7a3917338c86b6cb8d26b06f5a33 Author: Víctor Manuel Jáquez Leal Date: Thu Jul 21 11:46:55 2016 +0200 gmessage: Suppress string format literal warning https://bugzilla.gnome.org/show_bug.cgi?id=769029 glib/gmessages.c | 5 +++++ 1 file changed, 5 insertions(+)