commit ec2a7cc710988176e3f4a68c7dd8c81538037550
Author: Matthias Clasen <mclasen@redhat.com>
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 <philip.withnall@collabora.co.uk>
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 <philip.withnall@collabora.co.uk>
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 <desrt@desrt.ca>
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 <smcv@debian.org>
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 <smcv@debian.org>
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 <jmas@softcatala.org>
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 <philip.withnall@collabora.co.uk>
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 <philip.withnall@collabora.co.uk>
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 <philip.withnall@collabora.co.uk>
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 <philip.withnall@collabora.co.uk>
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 <philip.withnall@collabora.co.uk>
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 <philip.withnall@collabora.co.uk>
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 <philip.withnall@collabora.co.uk>
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 <philip.withnall@collabora.co.uk>
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 <philip.withnall@collabora.co.uk>
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 <philip.withnall@collabora.co.uk>
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 <philip.withnall@collabora.co.uk>
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 <jmas@softcatala.org>
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 <prescott66@gmail.com>
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 <yoseforb@src.gnome.org>
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 <philip.withnall@collabora.co.uk>
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 <philip.withnall@collabora.co.uk>
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 <philip.withnall@collabora.co.uk>
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 <philip.withnall@collabora.co.uk>
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 <philip.withnall@collabora.co.uk>
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 <ebassi@gnome.org>
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 <aurisc4@gmail.com>
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 <philip.withnall@collabora.co.uk>
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 <walters@verbum.org>
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 <yury.usischev@phystech.edu>
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 <mclasen@redhat.com>
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 <mclasen@redhat.com>
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 <daniel.mustieles@gmail.com>
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 <mario@endlessm.com>
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 <mario@endlessm.com>
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 <mario@endlessm.com>
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 <fanchunwei@src.gnome.org>
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 <fanchunwei@src.gnome.org>
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 <fanchunwei@src.gnome.org>
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 <mclasen@redhat.com>
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 <mario@endlessm.com>
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 <mario@endlessm.com>
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 <mario@endlessm.com>
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 <mario@endlessm.com>
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 <mario@endlessm.com>
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 <mario@endlessm.com>
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 <mario@endlessm.com>
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 <mclasen@redhat.com>
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 <mclasen@redhat.com>
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 <mclasen@redhat.com>
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 <mclasen@redhat.com>
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 <thiago.macieira@intel.com>
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 <mclasen@redhat.com>
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 <mclasen@redhat.com>
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 <mclasen@redhat.com>
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 <mclasen@redhat.com>
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 <mclasen@redhat.com>
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 <pochu27@gmail.com>
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 <rstrode@redhat.com>
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 <mclasen@redhat.com>
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 <mclasen@redhat.com>
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 <alberts.muktupavels@gmail.com>
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 <qdlacz@gmail.com>
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 <berrange@redhat.com>
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 <berrange@redhat.com>
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 <vjaquez@igalia.com>
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(+)