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.

Add bugfix and dependency change to NEWS.

Merge branch libgeetest which upgrades our libgee dependency.
Fixes bug https://bugzilla.gnome.org/show_bug.cgi?id=673918

Port folks to libgee 0.8.
Added hash_static and equal_static to AbstractFieldDetails.
Use AbstractFieldDetails hash_static and equal_static where needed.

backend-store: never translate warning messages

Most of them were not translated.

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

backend-store: output a debug message if mime type of potential backends looks weird

Will make my life much easier next time I'm wondering why nothing is working.

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

Arabic translation

Change libgee dependency from 0.6 to 0.8.

Port away from tp_connection_new since it has been deprecated.
Synced some sources from telepathy-glib/tests/lib/.

Get libexecdir from libedata-book-1.2.pc.

Bump released bit back to 0.
Change dependency in NEWS from 3.5.4 to 3.5.3.1

Fix eds dependency to 3.5.3.1.
Bump release numbers for 0.7.2.2 release.

Change UNRELEASED to 0.7.2.

Updated Vietnamese translation

Post 0.7.2.1 release version bump.
Added 0.7.2 to 0.7.3 section to NEWS.

Add Profiling api to NEWS file.
Bump soname numbers in configure.ac for Folks 0.7.2.1 release.

Bump nano version and revert released bit.
Add 0.7.3 section to the top of NEWS.

Use async creation of Source Registry in eds test backend.

Start eds source-registry and addressbook-factory ourselves.

Add test.source and other.source to EXTRA_DIST.

Add Bugfix to NEWS.

Add folks-internal to libsocialweb backend library vapi.
Fixes https://bugzilla.gnome.org/show_bug.cgi?id=679020

Add folks-internal to make docs dependencies for profiling symbols.

Default to xz tarballs for distribution.

Remove individual-zeitgeist from make check targets.

Folks 0.7.2 release.

tracker: Unmark a method as asynchronous

Private method, didn’t need to be asynchronous.

eds: Share a single E.SourceRegistry between all Edsf.PersonaStores

Turns out a single E.SourceRegistry takes about 200ms to construct, due to
involving D-Bus round trips, so sharing the existing SourceRegistry from the
Eds.Backend to all its Edsf.PersonaStores is a big win.

aggregator: Don’t prematurely reach quiescence

If one of the backends loads very fast, but doesn’t signal the addition of
any persona stores for a while, the IndividualAggregator will prematurely
reach quiescence. We now force quiescence to be reached at least after all
the backends have finished loading.

core: Add a profiling API and implement some profiling points

The API outputs messages in the format accepted by Federico’s
plot-timeline.py (http://people.gnome.org/~federico/news-2006-03.html#timeline-tools)
so graphs can easily be generated by running:

    strace -ttt -f -o /tmp/logfile folks-inspect
    python plot-timeline.py -o graph.png /tmp/logfile

So far, the BackendStore, IndividualAggregator and Backend and PersonaStore
classes for all the backends have been annotated.

Add bug fix to NEWS file.

Fix style issues as noted in https://bugzilla.gnome.org/show_bug.cgi?id=678811.
Use _addressbook_name to ref the test source.
Remove unused _source_name.

Remove GConf stop from eds test harness, since it's not needed.

Add mainloop to eds test backend tear_down to wait for addressbook removal.

Fix link-personas-diff-stores test.
Need to use both test and other addressbooks.

Use ESource display_name instead of uid.

Use async addressbook remove method, since sync method hangs currently.

Fix link-personas-diff-stores unit test.
Remove obsolete EDS TestBackend address_uri.
Allow choosing other or test source in set_up method.

Create another other.source for eds unit tests.

Port test eds backend to eds api changes.
Add test.source keyfile for unit tests.

Re-enable eds backend unit tests.

core: Replace symbols with spaces when matching names in PotentialMatch

This allows, for example, “Alice Badger” and “alice.badger” to have a HIGH
match. This is a common situation when trying to pair up IM personas with
other personas.

This also includes a test case.

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

eds: Fix memory leaks in EDS backend

These were unavoidably caused by a problem with the EDS Vala bindings. This
will be fixed in EDS 3.5.4, so our dependency has been bumped.

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

core: Fix indexing for unichars in PotentialMatch’s Jaro distance function

The indexing was getting out of sync between the two strings if one contained
a different number of non-ASCII characters to the other (as the indexing was
done in terms of bytes, rather than characters). This re-works the Jaro
distance function to operate on unichar arrays, and index in terms of
characters. This also means that exact matches now work over stripped non-ASCII
characters, which they didn’t before.

This adds a new test case. Yay.

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

core: Tidy up MatchResult usage in PotentialMatch a little

There’s no need for the result to be an instance variable.

eds: Adjust to e-d-s API break

Address books are now referenced by UID instead of URI, and the API
has been updated to reflect this.

Update Punjabi Translation

Updated Galician translations

eds: Fix warnings about empty postal addresses

eds: Revert checks for empty properties

These are in the wrong place and duplicate checks further down those code
paths.

Updated Greek translation

Added Greek translation

Updated Spanish translation

telepathy: Remove unused variable