Revert "Add private _g_bus_get_singleton_if_exists() function"

This reverts commit f8a8e9039889c090fb3323e195730b9e460396e4.

Revert "Add GTestDBus object"

This reverts commit 1b5f70b5b035019ba28da6a5db6eff8122e17ae7.

GMenuModel: Don't leak GDBusConnection in test

See https://bugzilla.gnome.org/show_bug.cgi?id=672985#c89

Signed-off-by: David Zeuthen <davidz@redhat.com>

Add GTestDBus object

This is a helper to write unit tests using a private dbus-daemon.

session_bus_up/down() are now just wrappers around a GTestDBus singleton.

https://bugzilla.gnome.org/show_bug.cgi?id=672985

Add private _g_bus_get_singleton_if_exists() function

This is used by g_test_dbus_down() to ensure the GDBusConnection
gets disposed, but not create one if the singleton already got
disposed.

https://bugzilla.gnome.org/show_bug.cgi?id=672985

Add a private copy of gio/tests/gdbus-tests.c,h to gio/

https://bugzilla.gnome.org/show_bug.cgi?id=672985

Updated Norwegian bokmål translation

Do not detect GNUstep as Cocoa

GNUstep also installs Foundation/Foundation.h

https://bugzilla.gnome.org/show_bug.cgi?id=674172

Signed-off-by: William Hua <william@attente.ca>

Use Cocoa header for Cocoa test instead of Foundation.

gdbus-codegen: Don't leak stuff in tests

Signed-off-by: David Zeuthen <davidz@redhat.com>

GDBusObjectProxy: Don't leak connection

Signed-off-by: David Zeuthen <davidz@redhat.com>

GDBusObjectManagerClient: Don't leak object proxy when handling D-Bus signal

It's g_object_unref(), not g_object_ref(). Ugh.

Therefore, use g_clear_object().

Signed-off-by: David Zeuthen <davidz@redhat.com>

Updated POTFILES.in

gio: use GPollable* to implement fallback read_async/write_async

If a GInputStream does not provide a read_async() implementation, but
does implement GPollableInputStream, then instead of doing
read-synchronously-in-a-thread, just use
g_pollable_input_stream_read_nonblocking() and
g_pollable_input_stream_create_source() to implement an async read in
the same thread. Similarly for GOutputStream.

Remove a bunch of existing read_async()/write_async() implementations
that are basically equivalent to the new fallback method.

https://bugzilla.gnome.org/show_bug.cgi?id=673997

gio: implement GPollableInput/OutputStream in more stream types

Implement GPollableInputStream in GMemoryInputStream and
GConverterInputStream, and likewise implement GPollableOutputStream in
the corresponding output streams.

https://bugzilla.gnome.org/show_bug.cgi?id=673997

gpollableutils: utility functions for pollable stream implementations

Move g_pollable_source_new() here from gpollableinputstream.c, add
g_pollable_source_new_full(), and add some new methods to do either
blocking or nonblocking reads depending on a boolean argument.

https://bugzilla.gnome.org/show_bug.cgi?id=673997

gio: minor GPollableInputStream / GPollableOutputStream fixes

Make g_pollable_input_stream_read() and
g_pollable_output_stream_write() look a little bit more like the
non-pollable versions in terms of error handling, etc. Also, use the
read_fn and write_fn virtual methods directly rather than calling
g_input_stream_read()/g_output_stream_write(), to avoid problems with
re-entrancy involving the "pending" flag.

Also belatedly add single-include guards to the header files.

https://bugzilla.gnome.org/show_bug.cgi?id=673997

GConverterInputStream: fix an infinite loop when fill_buffer returns an error

The loop was using a GConverterResult variable where it meant to use a
gssize, and since GConverterResult was ending up as an unsigned type,
this meant the (res < 0) check always failed.

avoid warning in gutils.h when using gcc with -Wconversion

Old solution to avoid warning with gcc flag -Wconversion does not
work with gcc >=4.3 since the behaviour for that flag has been
changed, ref. http://gcc.gnu.org/wiki/NewWconversion. Now cast for
__builtin_clzl is required, which itself is declared to return int.

https://bugzilla.gnome.org/show_bug.cgi?id=619026

Updated Telugu Translation

Updated Bulgarian translation

Updated Galician translations

gdbus-codegen: Avoid warnings in generated code

Initialising a "gchar *" struct member from a string literal may produce
a warning; add an explicit cast to fix that.

Bug #664275.

resources: compiler: Fix resources on big endian architectures

Resources are always little endian, so the gvdb is byteswapped. When looking
up the value, it would return a new byteswapped variant, making the data
returned from do_lookup() invalid once that variant is unref'd. Since
byteswapping doesn't matter for the "ay" data anyway, just use
gvdb_table_get_raw_value() instead and only byteswap the length and flag
values.

https://bugzilla.gnome.org/show_bug.cgi?id=673409

[l10n] Updated German translation

Updated Slovenian translation

Updated Polish translation

tests: Fix race conditions in mainloop/invoke test

1) The test was using GCond incorrectly (it always needs a
   state variable)
2) The state assertion was racing with the thread; just delete it.
   All we're really trying to test here is that the invoke runs by the
   time the thread is gone, and the function has an assertion that
   it runs in the correct thread.

https://bugzilla.gnome.org/show_bug.cgi?id=674213

tests/mainloop: Add a test for recursive child sources

https://bugzilla.gnome.org/show_bug.cgi?id=669260

tests/mainloop: add a test for child sources

https://bugzilla.gnome.org/show_bug.cgi?id=669260

gmain: block child sources when blocking the parent

When blocking a source that has child sources, we need to consider the
children blocked as well. Otherwise they will still trigger repeatedly
in an inner loop started from the parent source's callback.

https://bugzilla.gnome.org/show_bug.cgi?id=669260

Only build gmenumodel test on unix

It depends on gdbus-sessionbus.c which only builds on unix.

tests/fileutils: Fix Windows build

Updated HINDI translation

tests/date: force English locale running the GDateTime tests

The parsing test needs to make some assumption about the locale
representation of the string to be parsed, so we need to explicitly
override the locale here.

Updated Spanish translation

Update Czech translation

Add support for MX, TXT, NS and SOA records to GResolver

 * Add resolver functions for looking up DNS records of
   various types. Currently implemented: MX, TXT, SOA, SRV, NS
 * Return records as GVariant tuples.
 * Make the GSrvTarget lookups a wrapper over this new
   functionality.
 * Rework the resolver test so that it has support for
   looking up MX, NS, SOA, TXT records, and uses GOptionContext

https://bugzilla.gnome.org/show_bug.cgi?id=672944

Update Czech translation

update Simplified Chinese (zh_CN) translation

configure: Fix typo in ELF check

Signed-off-by: Colin Walters <walters@verbum.org>

build: When cross compiling, don't require host binaries if tests are not enabled

These binaries are now only used by the test suite.  glib-genmarshal
*used* to be required to generate marshallers, but isn't anymore now
that we use libffi (via g_cclosure_marshal_generic).

https://bugzilla.gnome.org/show_bug.cgi?id=667806

build: Add --disable-modular-tests build option

This patch solves two problems:

First, it allows builders to optionally cut the circular dependency
between dbus and glib by disabling the modular tests (just like how
the tests can be disabled in dbus).

Second, the tests are entirely pointless to build if cross-compiling.

It also moves us slightly closer to the long term future we want where
the tests are a separate ./configure invocation and run against the
INSTALLED glib, not the one in the source tree. This would allow us to
run the tests constantly, not just when glib is built.

https://bugzilla.gnome.org/show_bug.cgi?id=667806

Updated Russian translation

fileinfo: document the correct type for trash::orig-path

The correct type for this attribute, as set by GVfs, is
G_FILE_ATTRIBUTE_TYPE_BYTE_STRING (which is the correct type for file
paths anyway).

https://bugzilla.gnome.org/show_bug.cgi?id=674074

GDBusAuth: Handle when no there is no auth observer present

I obviously fucked up when adding the ::allow-mechanism signal, sorry.

Signed-off-by: David Zeuthen <davidz@redhat.com>

Check that auth methods work and interoperate with libdbus-1

See https://bugzilla.gnome.org/show_bug.cgi?id=673943

Signed-off-by: David Zeuthen <davidz@redhat.com>

GSocketControlMessage: Don't warn about unknown messages

If we do this, the tests added in bug 673943 will cause warnings.

Signed-off-by: David Zeuthen <davidz@redhat.com>

GDBusAuthObserver: Add a way to control what authentication mechanisms to use

This is related to https://bugzilla.gnome.org/show_bug.cgi?id=673943
but also useful in a lot of other contexts.

GDesktopAppInfo: add an accessor for StartupWMClass

Components using GIO to do window to application matching can
use that field to retrieve potential candidates.

https://bugzilla.gnome.org/show_bug.cgi?id=673659

Add version macros for 2.34

This marks the start of the new development cycle, and opens the
window for API additions.

https://bugzilla.gnome.org/show_bug.cgi?id=673659

Bump version

2.32.1

Fix distcheck

I forgot to dist a test script, and forgot to look for it
in srcdir instead of builddir.

[l10n] Updated German translation

tests/mainloop: fix a race condition

Rather than depending on the machine's speed/load, just interlock
between the two threads properly.

tests/mainloop: use g_assert_cmpint(), for better error messages

Make the markup-parse test independent of the locale

I added a setlocale call, because we need it for Unicode to
come out right; but I forgot to fix the locale, so we now
fail when comparing error messages to the expected (English)
result. Correct this by setting LANG explicitly to en_US.utf-8.

https://bugzilla.gnome.org/show_bug.cgi?id=669285

Updated British English translation

GResource docs: fix typo

GSettings docs: clarify what is a good path

Add an explicit note to the docs about choosing paths based on domain
names, not ones like "/apps/", "/desktop/" or "/system/".

glib-compile-schemas: warn about bad dconf paths

For quite some time the recommended usage of GSettings and dconf has
been to use paths like /org/gnome/example/.  Use of /apps/ has spilled
over from GConf and is continuing to make its way into a number of
applications as they port.

glib-compile-schemas will now warn about these types of paths being
used.  This generates a lot of noise, but hopefully it will reduce the
number of ported applications making this mistake.

Fall back to SO_PEERCRED if credentials passing fails

Turns out libdbus doesn't send struct ucred credentials on linux, but
just relies on the SO_PEERCRED support. However, gdbus does send, and
expect to recieve a ucred credential. So, when libdbus talks to a
gdbus server the authentication fails to send the credentials.

We fix this by falling back to g_socket_get_credentials() if we don't
get any credential messages.

Detect "empty" socket credentials on Linux

Linux uses struct ucred to pass over socket credentials. Historically
this has always worked in recievemsg, if SO_PASSCRED was set on the socket,
even if the remote side didn't pass any credits. But this change broke that:

http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commitdiff;h=16e5726269611b71c930054ffe9b858c1cea88eb;hp=a9e9fd7182332d0cf5f3e601df3e71dd431b70d7

However, it doesn't actually fail getting the credentials, it just returns
an "empty" one, as initialized by cred_to_ucred() at:

http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=blob;f=net/core/sock.c;h=b29ab61b029cf7f76fda992ecfcb8dcaa06b0483;#l756

So, we detect this and fail the credentials reading.

This actually happened in real life with gdbus acting as a server, as
gdbus expected an ucred but libdbus didn't send one.

Be more precise about Carbon v. Cocoa.

[l10n] Update Japanese translation

gio-2.0.pc: Avoid full path to executables

Instead of using full path in glib_compile_schemas and
glib_compile_resources variables, rely on having the executables in
PATH.

This fixes the cross-compiling case where we cannot execute the binaries
for target system. Instead of executing the target system's binaries, we
need to use the native versions installed on the build host. The easiest
way to find the native executables is to just pick them up from PATH.

In addition, this brings gio-2.0.pc in line with glib-2.0.pc -- the
latter has historically only listed the executable name and not the full
path.

https://bugzilla.gnome.org/show_bug.cgi?id=673911

Don't use fast enumeration in old version of Mac OS.

Signed-off-by: William Hua <william@attente.ca>

Link with Foundation framework on OSX.

Signed-off-by: William Hua <william@attente.ca>

Updated French translation

Updated HINDI translation

Updated Norwegian bokmål translation

Updates

[l10n]Updated Catalan (Valencian) translation

[l10n] Fixes on Catalan translation

gdbus: test case for 673612

When presented with an array of empty arrays of 8-byte-aligned types,
GDBus would incorrectly apply the 8-byte alignment when reading back.

https://bugzilla.gnome.org/show_bug.cgi?id=673612

Signed-off-by: David Zeuthen <davidz@redhat.com>

GDBusMessage: do not align for grandchildren of empty arrays.

D-Bus arrays are serialized as follows:

1. align to a 4-byte boundary (for the length)
2. uint32: the length of the serialized body in bytes
3. padding for the alignment of the body type (not included in the length)
4. the body.

Note that 3. is a no-op unless the body type is an 8-byte aligned type
(uint64, int64, double, struct, dict_entry), since you are always on a
4-byte boundary from aligning and writing the length.

So, an empty aax (that is, an array containing zero arrays of int64)
is serialized as follows:

1. align to a 4-byte boundary
2. length of the contents of this (empty) array, in bytes (0)
3. align to a 4-byte boundary (the child array's alignment requirement)
4. there is no body.

But previously, GDBus would recurse in step three to align not just for
the type of the child array, but for the nonexistent child array's
contents. This only affects the algorithm when the grandchild type has
8-byte alignment and the reader happened to not already be on an 8-byte
boundary, in which case 4 bytes were spuriously skipped.

https://bugzilla.gnome.org/show_bug.cgi?id=673612

Signed-off-by: David Zeuthen <davidz@redhat.com>

GDBusMessage: print more debug info about alignment

Signed-off-by: David Zeuthen <davidz@redhat.com>

Updated Hebrew translation.

Updated Serbian translation

gclosure: Support return values of GVariants

https://bugzilla.gnome.org/show_bug.cgi?id=673803

tests: always use four digit years for date parsing tests

Or some system with different locale settings might get confused whether
a two digit year is to be parsed with regard to the current century or
as an absolute year.

Updated Lithuanian translation

Updated POTFILES.skip

Updated Galician translations

binding: Improve test coverage

spawn: Improve test coverage

Add a test that excercises the script execution code.
Unfortunately, much of this code only runs in the forked
child, and therefore its execution does not get caught
by gcov.

mainloop: Improve test coverage

settings: Improve test coverage

gdbusserver: Improve test coverage

icon: Improve test coverage

contenttype: Improve test coverage

Excercise more parts of the guessing machinery.
These tests rely on the mime database being present.

networkaddress: Improve test coverage

networkaddress: Fix userinfo parsing in uris

The code that is checking the userinfo part was accidentally
given a pointer to the end of the userinfo, so it was not
checking the right portion of the string at all.

utils: Improve test coverage

Add a test for some of our gettext variants.

fileutils: Improve test coverage

Add tests for corner cases of g_path_get_basename, and
for the errno-to-file-error conversion.

sort: Improve test coverage

Test the code paths that sort 'large' items.

date: Improve test coverage

Add tests that exercise g_date_order, g_date_compare, g_date_clamp,
and more of the date parsing functions.

convert: Improve test coverage

Add a test that excercises the 'no conversion' code path.
This uncovered that we don't treat errno properly in this path,
and as a consequence, the returned error code is unreliable.

collate: Improve test coverage

Add some strings that exercise the non-ASCII parts of our
collation support.