More updates

gio/tests/gdbus-peer: make this work on non-Linux unixes

The test was using a socket in a temporary directory, but not actually
creating that temporary directory. This worked fine on Linux since it
actually ended up using an abstract socket instead, but failed on
unixes without abstract sockets.

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

Add a forgotten section

More doc additions

Fix a doc comment

Clean up gobject docs

Correct a doc typo

Remove no-longer existing api

GSocketAddress: create IPv4 addresses for IPv6 mapped-IPv4 addresses

IPv6-mapped-IPv4 addresses are annoying. Just translate them to real
IPv4 addresses.

based on a patch from Jonny Lamb
https://bugzilla.gnome.org/show_bug.cgi?id=646082

Add a 'since 2.30' index to gobject docs

Don't distribute MAINTAINERS

...it does not exist anymore.

First round of NEWS updates for 2.29.18

GDBusObjectManagerServer: Add locking

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

GDBusObjectManagerClient: Add locking

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

GDBusObjectProxy: Add locking

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

Don't call close on invalid fds

If an fd is -1, don't call close() on it, since that leads to
EBDAF and complaints from valgrind.

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

GDBusObjectSkeleton: Add locking

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

GDBusInterfaceSkeleton: Add locking

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

Print out URL to HTML coverage report after 'make lcov'

* For easy clicking

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

gdatetime: Tweak g_date_time_source_new() documentation

Spelling fixes

Spelling fixes in comments and docs, provided by
Kjartan Maraas in bug 657336.

Add Linux timerfd_create() backend for g_date_time_source_new()

This makes the source efficient on Linux.

Tested on Fedora 15 x86_64 + updates, kernel-2.6.40-4.fc15.x86_64
Also tested fallback code for unsupported flag TFD_TIMER_CANCEL_ON_SET
on kernel 2.6.38.

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

gdatetime: Add g_date_time_source_new()

Several different codebases in GNOME want to implement wall clocks.
While we could pretty easily share a private library, it's not a
substantial amount of code, and GLib already has a lot of the
necessary system-specific detection and handling infrastructure.

Note this initial implementation just wakes up once a second in the
cancel_on_set case; we'll add the Linux-specific handling in a
subsequent commit.

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

Updated Traditional Chinese translation(Hong Kong and Taiwan)

Updated Russian translation

Updated Norwegian bokmål translation

Updated galician translations

gio/tests/gtlsconsoleinteraction: Work around missing getpass() on win32

Loosely based on a patch by Sam Thursfield <ssssam@gmail.com>.
https://bugzilla.gnome.org/show_bug.cgi?id=656341

Updated Spanish translation

Updated Spanish translation

Fix symbol lists

Spell out file descriptors in a message

Translators don't necessarily know what 'fds' are.
Pointed out in bug 657454.

GDBusMessage: Use ngettext() where appropriate

Pointed out in bug 657452.

gsocketaddress: don't return ABSTRACT if the OS doesn't support it

FreeBSD apparently has non-0-length (but nameless) ANONYMOUS sockets.
Fix the heuristics here.

gunixcredentialsmessage: fix this on FreeBSD

pointed out by Henry Hu
https://bugzilla.gnome.org/show_bug.cgi?id=628904

GSocket: fix GIOCondition on timed-out socket operation

The docs for g_socket_set_timeout() claimed that if an async operation
timed out, the GIOCondition passed to the source callback would be
G_IO_IN or G_IO_OUT (thus prompting the caller to call
g_socket_receive/send and get a G_IO_ERROR_TIMED_OUT), but in fact it
ended up being 0, and gio/tests/socket.c was erroneously testing for
that instead of the correct value. Fix this.

Updated Brazilian Portuguese translation.

Fixed typo: s/ether/enter.

Updated Brazilian Portuguese translation.

giomodule: When loading GIO_EXTRA_MODULES skip duplicates

 * Load modules from paths listed in GIO_EXTRA_MODULES environment
   variable first.
 * Ignore duplicate modules based on module basename.
 * Add the concept of GIOModuleScope which allows other callers to
   skip duplicate loaded modules, or block specific modules based on
   basename.
 * Document behavior.

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

Updated Brazilian Portuguese translation.

Make GTlsInteraction virtual methods cancellable

 * Add cancellable argument to g_tls_interaction_ask_password
   and g_tls_interaction_ask_password_async.
 * This is API breakage, but this API has not yet been released
   in a stable release (and very unlikely used yet).
 * Since we're breaking unreleased API, expand amount of padding
   on GTlsInteractionClass because we're going to need it.

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

Add gdbus-codegen to glib-zip.in

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

Make gdbus-codegen 'relocatable' at runtime on Windows.

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

gdbus-codegen: Revert to using /usr/bin/env for Python

After a lot of discussion, we decided /usr/bin/env is the most
flexible for now.

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

Introduce the UNINSTALLED_GLIB_SRCDIR environment variable

This makes it possible to avoid setting PYTHONPATH in
Makefile.am files.

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

Introduce the UNINSTALLED_GLIB_BUILDDIR environment variable

This makes it possible to also use relative imports for
gdbus-codegen's config module.

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

Use relative imports for the gdbus/codegen package

But not yet for the config module (but is taken care
of in a followup patch).

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

Avoid using - (hyphen) in gdbus-codegen directory name

It's an invalid character in Python module names and prevents us from
being able to import it.

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

Use detected PYTHON shebang in gdbus-codegen

Change the hardcoded /usr/bin/python shebag from gdbus-codegen.in into
@PYTHON@. Is used in Makefile.am to use detected python binary.

$(AM_V_GEN) sed -e 's,@libdir\@,$(libdir),' -e 's,@PYTHON\@,$(PYTHON),'
$< > $@.tmp && mv $@.tmp $@

Signed-off-by: Ionut Biru <ibiru@archlinux.org>
https://bugzilla.gnome.org/show_bug.cgi?id=657274

Updated Russian translation

glib.doap: Fix download location

GCancellable: Small annotation additions

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

Updated Galician translations

Updated Spanish translation

gfileutils: fix docs/annotations for temp file methods

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

Add DOAP file and remove MAINTAINERS

Updated Norwegian bokmål translation

Updated Swedish translation

Bug 657206 — GInputStream leaked in g_file_icon_load_async()

Add a missing unref to g_file_icon_load_async().

Closes: bgo#657206

GIO: Add more missing GI annotations

 - g_io_extension_point_register()
 - g_io_extension_point_lookup()
 - g_io_extension_point_implement()

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

Updated Serbian translation

gdbus-codegen: update comment in code to reflect reality

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

gdbus-codegen: Rework C property getters

Rework property getters to use a vfunc so we can take the fast path
and avoid allocating memory for both the skeleton and the proxy
cases. This requires some special case because of how GVariant expects
you to free memory in some cases, see #657100. Add test cases for
this.

Document the _get_ functions as not being thread-safe and also
generate _dup_ C getters (which are thread-safe).

Mark all the generated _get_, _dup_ and _set_ as (skip) as non-C
languages should just use GObject properties and not the (socalled)
"C binding".

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

po: add missing gtls files to POTFILES.in

Pointed out by Daniel Mustieles
https://bugzilla.gnome.org/show_bug.cgi?id=657138

gdatetime: Don't include langinfo.h unconditionally

It doesn't exist on Windows.

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

Update VS property sheets

Stop the "installation" of gio/gtimezonemonitor.h as it has been removed
from GIO (commit 5b68b49b2072)

Update GLib Visual C++ Projects

Define USE_SYSTEM_PCRE for all configurations which uses the PCRE that
was already built and "installed" beforehand (i.e. the *_ExtPCRE
configurations) so that the compilation will not pick up the
GLib-bundled pcre.h when one wants to use the PCRE "installation" on
his/her system.

Update config.h.win32.in

Make the pre-configured config.h(.win32.in) for Windows more like the
config.h that would be produced during ./configure on Windows systems.

GTimeZoneMonitor: Revert addition of this class

The main rationale for adding it was to avoid having gnome-shell
mmap'ing /etc/localtime once a second.  However, we can just as easily
run inotify there, and given no one else was clamoring for a way to
detect when the time zone changes, I don't see a need for public API
here - at least not yet.

In the bigger picture, I just don't believe that the vast majority of
applications are going to go out of their way to instantiate and keep
around a random GTimeZoneMonitor class.  And if they do, it's has the
side effect that for other bits of code in the process, local GDateTime
instances may start varying again!

So, if code can't rely on local GDateTime instances being in a
consistent state anyways, let's just do that always.  The
documentation now says that this is the case.  Applications have
always been able to work in a consistent local time zone by
instantiating a zone and then using it for GDateTime constructors.

We fix the "gnome-shell stats /etc/localtime once a second" issue by
using timerfd (in glib) and inotify (in gnome-shell).

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

Update VS property sheets

-Added glib/ghmac.h to the list of files to copy during the "install" stage
-Cleaned up a bit (glib-2.0->glib-$(ApiVersion), where $(ApiVersion) is
 2.0)

gmain: Clarify that timeouts are in terms of monotonic time

Also note that monotonic time does not include time spent while
suspended (at least on Linux).

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

GDateTime: use nl_langinfo() when available

This makes g_date_time_format() react to LC_TIME, which is
what people expect.

Translators: this change means that the GDateTime strings
are only used when the C library does not already provide
suitable translated strings for these (month names, etc).

GDateTime: cosmetics

Shuffle things around a bit, to move locale-dependent
things together.

GDateTime: cosmetics

Don't hide the recursion in g_date_time_format() behind
a macro, make it explicit.

Add a quick utility to test datetime formatting

GDateTime: don't use separate strings for upper/lowercase am/pm

We can just as well change the case ourselves.

g_date_time_format: support %T

strftime supports it, and there is no reason not to do the same.

[gi] Add signal annotations

[gi] Add annotations to g_quark functions

Add missing GClosure annotations

Add missing (allow-none) annotations to g_value_set_string functions

Merge branch 'concurrent-cancellable'

gio: GCancellable can be used concurrently

 * Update documentation to note that GCancellable can be used
   concurrently by multiple operations.
 * Add documentation to g_cancellable_reset that behavior is
   undefined if called from within cancelled handler.
 * Add test for multiple concurrent operations using the same
   cancellable.

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

update translation for Punjabi

Added UG translation

Use actual uint64 values to compare uint64 variants

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

Add (allow-none) annotation for GValue setters.

Assorted g_value_set_xxx() and g_value_take_xxx() functions actually
allow NULL in source parameter.

Fixup max decomposition len guarantee

Unicode Technical Committee agreed to limit decomposition length to
18 in both cases.  Reflect that.

void functions should not return.  Fixes bug #656675.

post-release bump

glib 2.29.16

distcheck fixes

More NEWS updates

Updated Esperanto translation

Esperanto weekdays and month days are written in lowercase (as in
French).

Add a stress-test for GDBusProxy in threads with no default main context

Destroying a GDBusProxy in a thread used to race with NameOwnerChanged
being delivered to the main context's thread (GNOME #651133).

Also, g_dbus_proxy_call_sync in a thread would race with NameOwnerChanged
being delivered to the main context's thread and rewriting the name_owner
(GNOME #656039).

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=656039
Bug-NB: NB#259760
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Signed-off-by: David Zeuthen <davidz@redhat.com>

GDBusProxy: hold properties_lock while using any mutable property

This changes the meaning of "properties_lock" from "lock for D-Bus
properties" to "lock for GObject properties".

The most common problem, and the only one I've reproduced in a regression
test, is name_owner, which can be updated by the thread that owns
the GDBusProxy's main context (i.e. the thread-default main context of
the thread that constructed it) at the same time that a blocking call
is made. When a GDBusProxy is constructed in a thread-pool thread for
short-term use, the main context will typically be the global default
main context (which is actively running in the main thread!), making
this extremely problematic.

The interface info is perhaps a theoretical concern - one thread could
conceivably set it at the same time that another thread uses it, but only
in relatively pathological situations. The current API for this does have
the problem that it returns a borrowed ref, but interface info is
hopefully permanent anyway.

The default timeout is probably only a theoretical concern - it's just an
int, so writes are indivisible, and there's no worry about whether
something has been freed - but to be safe, let's hold the lock for that
too.

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=656039
Bug-NB: NB#259760

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

GDBusProxy: factor out async_init_data_set_name_owner

This removes the need for async_init_get_name_owner_cb to cope with being
called without a real GAsyncResult, and will simplify the addition of
correct thread-locking.

In async_init_data_set_name_owner, use the name_owner parameter instead
of the corresponding member of GDBusProxyPrivate, partly to reduce
pointer-chasing but mainly to avoid needing to hold the lock.

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

g_dbus_proxy_get_property: use accessors for all mutable state

These ought to have thread-locking, and having it in the accessor seems
better than duplicating it here.

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

GDBusProxy: if a well-known name is not owned, don't GetAll from the dbus-daemon

If you run:

    ( cd gio/tests && G_DBUS_DEBUG=all ./gdbus-proxy-well-known-name )

you can see that in the case where the name com.example.TestService isn't
owned yet, the GDBusProxy calls GetAll() with no destination, resulting
in an error reply from the peer (the dbus-daemon itself). That's clearly
not right!

However, if priv->name is NULL, that indicates the special case where we
really do want to talk directly to a peer, instead of via the bus daemon
(most likely to be used on peer-to-peer connections); in that special
case, do call GetAll().

Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Signed-off-by: David Zeuthen <davidz@redhat.com>