inspect: Fix crash when Utils.output_filestream is uninitialized.

Bug 682572 — Build failures due to accessing static members

Fix some more build failures due to accessing static members using an instance
variable. Also fix an instance of calling an async function with an implicit
‘.begin()’.

Hopefully that’s the last of them.

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

Updated Polish translation

Updated Indonesian translation

Bug 664072 — Folks should only use assert*() for critical, program-terminating errors

Turn various assert()s into return[_val]_if_fail()s.

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

Updated Galician translations

Note fixes for bgo#681420 in NEWS.

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

Access static class members through class; use explicit .begin()

This covers the libsocialweb backend tests.

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

Access static class members through class, not instance.

This covers the libsocialweb backend.

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

docs: Use correct Valadoc syntax for monospaced content

The syntax is ‘``monospaced text``’ rather than ‘`monospaced text`’, which
we were consistently getting wrong. Valadoc now produces the correct markup
for monospaced text, but it doesn’t appear differently from other text at
the moment due to Valadoc not having any CSS for it. That will change soon.

Similarly, Valadoc currently produces the wrong markup for monospaced text
with the gtkdoc doclet, but that will soon be fixed.

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

Add gir and .vapi files to DISTCLEANFILES if not building a release

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

eds: Add create_source_registry to eds tests.

tracker tests fixed to match last vala release.

Vala change:
1. Warn when accessing static members with an instance reference.
2. Deprecate implicit .begin for async methods.

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

tracker backend fixed to match last vala release.

Vala change:
1. Warn when accessing static members with an instance reference.
2. Deprecate implicit .begin for async methods.

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

docs: Add replacement symbols to ‘Deprecated’ annotations

This improves the documentation (and valac error messages) a little.

Added Marathi Translation

Update Punjabi Translation

eds: Add create_source_registry to eds-backend to fix build.

Prefer earlier alphabetical uid persona when two personas from the same persona store.
Fixes bug https://bugzilla.gnome.org/show_bug.cgi?id=681346

Updated gujarati file

Updated gujarati file

Updated gujarati file

docs: Various small fixes and expansions of documentation

 • Various links fixed to point to class documentation rather than its
   constructor documentation (this is a quirk of Valadoc; if using
   “{@link ClassName}” inside a method of ClassName, the link will point to
   ClassName’s constructor because symbols are resolved relatively and the
   class’ constructor is called “ClassName”).
 • Various bits of documentation expanded (mostly trivially) to shut gtk-doc
   up about missing long descriptions.
 • Some Vala code attributes moved around so the documentation comments are
   correctly associated with the code. (This shouldn’t change the behaviour
   of the attributes themselves.)
 • Some trivial constructors added to classes in order to give them
   documentation.
 • Constructor for Utils added and deprecated immediately. Utils should
   become a nested namespace on the next API break, since it will only ever
   contain static methods, and thus doesn’t need to be instantiable (which
   folks will make it at the moment).

docs: Remove protected symbols from the documentation

Folks isn’t really extensible outside its own source tree, so why clutter
up the documentation (and its build process/logs) with protected symbols?

eds: Minor API fixes in the EDS backend

These APIs, introduced in folks 0.7.1 (so this is an unstable API break,
but that’s OK), weren’t quite right in C. The SourceRegistry constructor
shouldn’t be exposed publicly as part of folks, and the PersonaStore
constructor had a stray extra parameter.

tests fixed to match last vala release.

Vala change:
1. Warn when accessing static members with an instance reference.
2. Deprecate implicit .begin for async methods.

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

Fixes to build with last vala release.

Vala change:
1. Warn when accessing static members with an instance reference.
2. Deprecate implicit .begin for async methods.

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

telepathy backend fixed to match last vala release.

Vala change:
1. Warn when accessing static members with an instance reference.
2. Deprecate implicit .begin for async methods.

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

key-file backend fixed to match last vala release.

Vala change:
1. Warn when accessing static members with an instance reference.

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

eds backend fixed to match last vala release.

Vala change:
1. Warn when accessing static members with an instance reference.

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

Added Latvian translation

LINGUAS updated and new hindi translation added

Assamese translation completed

Added Assamese to LINGUAS

configure: -Werror= make no sense for "no" and "extra"

gcc error out when -Werror=extra -Werror=no-*
are used. And it is the most sensible thing to do.

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

Post-release version bump

build: Add a missing file to MAINTAINERCLEANFILES

build: Remove -Werror from CFLAGS

Replace it with a more targetted set of -Werror=[flag] flags, copied from
libsoup: http://git.gnome.org/browse/libsoup/tree/configure.ac?id=f5902fce98ae0314f0d9ca6e544895548c94a456#n339

Updated Galician translations

eds: Remove unnecessary valadoc taglet

The [Deprecated] code attribute serves the same purpose.

docs: Add “wiki” pages to the documentation

These, aptly enough, contain information moved from the wiki on
freedesktop.org about folks concepts, debugging and a glossary.

Release version 0.7.3

build: Add a missing file to the tests Makefile

Updated Russian translation

Updated German translation

Updated gujarati file  & Added LINGUAS file for gujarati

Revert "Change libgee dependency from 0.6 to 0.8."

This reverts commit 5b5360bff7a61ef17173cbe35cc32712ca0fc329.

Revert "Port folks to libgee 0.8."

This reverts commit e069bbe59dc54eb3595d8ee5c96aea4a36cff9aa.

Bug 660236 — Paging for long text in folks-inspect

Use $PAGER to allow paging of the output of really long commands in
folks-inspect.

Based on work by Raúl Gutierrez Segales and Jeremy Whiting.

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

Remove extra calls to set this._prepare_pending to false.

Removed locks in trf-persona-store that are not needed.
Minor style fixes based on review of previous changes.

Remove additional locks around this._is_prepared.

Cut unnecessary locking in Tracker backend tests.

Closes: bgo#652637 - Don't hold locks across async calls

Cut unnecessary Swf.PersonaStore._is_prepared locking.

Helps: bgo#652637 - Don't hold locks across async calls

Cut unnecessary Kf.PersonaStore._is_prepared locking.

Helps: bgo#652637 - Don't hold locks across async calls

Cut unnecessary Tpf.PersonaStore._is_prepared locking.

Helps: bgo#652637 - Don't hold locks across async calls

Cut unnecessary Trf.PersonaStore._is_prepared locking.

Helps: bgo#652637 - Don't hold locks across async calls

Prevent extra "concurrent" Aggregator prepare() calls from one thread.

Helps: bgo#652637 - Don't hold locks across async calls

Prevent extra "concurrent" PersonaStore.prepare() calls from one thread.

The lock() calls which were in place only prevent concurrent calls from
different threads; however, the critical section could still be reached
from two async calls in the same thread a la:

backend.prepare.begin ((s,r) => {...});
backend.prepare.begin ((s,r) => {...});

The bodies of the handlers would not literally execute concurrently, but
would be interleaved if each were sufficiently long. Our fix simply
ignores any calls which happen while one is still working.

(yield backend.prepare(); yield backend.prepare(); was already safe, of
course)

Helps: bgo#652637 - Don't hold locks across async calls

Add test cases for mutexing for async calls from the same thread.

The standard Vala lock() call is a recursive lock, so it only prevents
concurrent access between different threads. But we rarely run in
distinct threads, so this test uses a more-complete solution that we can
depend upon (since the test will fail if its assumptions ever change in
Vala).

Helps: bgo#652637 - Don't hold locks across async calls

Fix Tracker structure locking for user-initiated Persona adds.

Helps: bgo#652637 - Don't hold locks across async calls

Updated Norwegian bokmål translation

eds: Make some PersonaStore methods synchronous

There was absolutely no point in them being asynchronous (they didn’t yield
at all, and didn’t run any blocking operations), so were just generating
excess C code and slowing everything down.

This cuts 1200 lines from edsf-persona-store.c.

eds: Hide the “Starred in Android” group from the set of groups

For Google address books, it’s used to determine whether a contact is a
favourite, so needn’t be shown in the set of groups.

Note that this requires commit 161faab40fa65e813ac98042ef822298dc58cfde
from EDS in order to behave properly.

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

Added Belarusian translation.

core: Fix documentation on GroupDetails.groups

It was out of date and didn’t match the data type of GroupDetails.groups
any more.

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

telepathy: Preserve Tpf.Persona avatars from the cache for online contacts

This is a fairly hacky way of fixing the problem in bug #660128: maintaining
a map of IIDs to avatar files for all personas in the Telepathy backend, and
referring to it in case Telepathy doesn’t know the current avatar for a
TpContact. This can happen if that contact is currently offline while we’re
online, for example. This allows use of the avatars cached from last time
contacts were online, even though those contacts are now offline.

A more comprehensive fix, which I would have implemented if I hadn’t just got
my degree classification and weren’t just heading out to celebrate, would
involve rearchitecting the caching of the Telepathy backend so that the
Tpf.Personas weren’t destroyed and re-created whenever the backend went
offline/online; instead, they would have their properties updated from the
cache/online contact list. This would eliminate spurious personas-changed
signals, unify the code implementing the different properties (currently it’s
split between the properties and separate implementations in
Tpf.Persona.from_cache()), and allow property values to persist from offline
to online without the need for hacks like this.

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

telepathy: Allow for updated Tpf.Personas properties to update the cache

Currently, the cache is only written when going offline, which means that any
properties of Tpf.Personas which are updated at runtime aren’t written to the
cache unless the client explicitly goes offline before being closed. This
doesn’t often happen, meaning the property updates are lost and the cache
becomes stale.

This commit keeps track of the clean/dirty state of the cache and writes it
out when PersonaStore.flush() is called. It also adds a new method,
IndividualAggregator.unprepare(), which ensures all the persona stores handled
by the aggregator are flushed. Clients should call this method before closing
their main loop.

(Calling flush() in the finalise function of the PersonaStore doesn’t work
because Tpf.PersonaStores are often never finalised due to the implementation
of Tpf.PersonaStore.dup_for_account(). Furthermore, calling flush() in the
finalise function of the IndividualAggregator doesn’t work because the client
will typically quit the main loop immediately afterwards, which will cancel
the asynchronous flush call.)

New API:
 • IndividualAggregator.unprepare()

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

core: Don’t keep references to PersonaStores in each Individual

Pointless and it makes debugging reference counting bugs a real pain.

telepathy: Change warning about CM not implementing contact list to debug message.
fixes: 675223

Updated Serbian translation

Added Serbian translation

Update NEWS file adding recently fixed bug.

eds: Trust google address books FULL since authorization is required to load.

folks: Don't downgrade persona store trust level.

tracker: Fix tracker unit tests to wait until persona store is prepared.

tracker: Throw an error if personas are added before store is prepared.

tests: Add execute-test.sh to do test output capturing.
tests: Call execute-test.sh from each of with-session-bus scripts.

Send eds server debug output into /dev/null along with regular output.
Extend sleep time to 2 seconds.

Updated Spanish translation

Added Lithuanian translation

Updated Traditional Chinese translation(Hong Kong and Taiwan)

telepathy: Tidy up group change notification handling in Tpf.Persona

eds: Add an Edsf.Persona.in_google_personal_group property

This is set whenever the persona is in the “My Contacts” group in Google
Contacts, which is normally exposed by EDS as being in the “Personal”
category. This system group can’t be reliably detected normally as “Personal”
is translated by EDS. This caused problems for gnome-contacts, hence the
new API.

The new API requires EDS 3.5.3 at runtime, and will always be false
otherwise.

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

Remove gconf soft dependency used for eds unit tests.
Fixes: bgo 674611

Add a sleep for one second after starting the eds servers.
Fixes inconsistent eds unit test failures.  All tests pass here now.

telepathy: Set _prepare_pending to true at beginning of prepare.
Fixes folks init unit test failure.

Merge branch 'cleantestoutput'
Reviewed-by: Philip Withnal
Closes: bgo 657456 https://bugzilla.gnome.org/show_bug.cgi?id=657456

Bug 679618 — Crash when unsetting all groups

Rework the group changing code in Tpf.Persona so that:
 1. change_groups() no longer changes the set of groups while iterating
    over it; and
 2. batch Telepathy calls are made where possible, rather than one call per
    group change.

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

Make test output use color for PASS/FAIL.
Don't show test output unless CHECK_VERBOSE is set.
Fixes: bgo 657456

Updated Norwegian bokmål translation

New French translation

Updated POTFILES.in and POTFILES.skip

key-file: Add anti-linking support

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

eds: Add anti-linking support

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

core: Add core anti-linking support

This adds the core of the anti-linking support, based around a new
AntiLinkable interface. This will be implemented by Persona subclasses which
can store anti-linking information (in the form of a set of Persona UIDs
which the given Persona should never be linked to).

This approach allows anti-linking information to be stored with the personas
(presumably in the primary persona store) and thus it should be network
transparent. i.e. Using folks on two different computers with a Google
Contacts address book as primary should cause the anti-linking data to be
shared.

This also includes the necessary IndividualAggregator changes.

Sadly, no unit tests are included.

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

core: Replace the linking step in IA.ensure_individual_property_writeable()

It’s a waste of time to create a new Persona to ensure the property is
writeable, then create _another_ new Persona to link it to the Individual.
Why not just create the writeable Persona so that it’s automatically linked
to the Individual?

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

core: Split details table code out of IA.link_personas()

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

aggregator: Convert the link map to use a MultiMap

When it's possible to have personas sharing IM addresses but not being
linked (due to anti-links), the link map's assumption that one IM address
(or other linkable property) maps to one individual will be invalid. This
commit converts the link map to be a HashMultiMap, effectively changing it
from a mapping of string → Individual to a mapping of
string → Set<Individual>.

Helps: bgo#629537

Change urls to links in doc comments, fixes doc creation.