build: Bump EDS dependency to 3.9.1

This EDS version fixes a Vala binding which was causing folks to
not build.

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

consider an individual as favorite if one of its persona is a favorite

We used to consider all the personas as having the "is-favourite" property
defined. So _update_single_valued_property was giving the priority to the
primary store (usually EDS) leading to the individual being considered as
unfavourite even if it has a Telepathy persona as favourite.

We now consider the "is-favourite" property being defined if it's set to TRUE
so any favourite persona will take presedence.

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

key-file: do not use deprecated loop_run_with_non_fatal_timeout()

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

Fix coding style in previous patch

TpfPersona: Manage the TpContact weak ref ourself instead of using GWeakRef

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

build: distcheck fails trying to remove a srcdir file

When we referenced $(srcdir)/libfolks_telepathy_la_vala.stamp, we were
telling automake that it was OK to delete it from the srcdir, which is
considered bad form. By referring to it from the builddir, we don't
implicitly suggest it can be deleted (as it is as a part of its creation
process).

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

docs: don't include the img/* content to avoid a build break

This ignores content in the img/ subdir to avoid build errors if it's empty
(as it currently is). This directory will store custom images, so if we add
any, we will need to re-add those files to the appropriate *_DATA
definitions.

build: don't depend on tp-zeitgeist.vapi for --disable-zeitgeist

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

Revert "docs: fix the path to the backend support library header files"

This reverts commit a945327bcbbab2af0c4871e9cbb70080ec71db8b.

See commit 311690817f40affee131cbd64870f9f5c4d32196 for details.

Revert "build: Don’t build Vala headers in a subdirectory unnecessarily"

This reverts commit 4c06ecc26bfde7397357ef4ae7ad83a5925db43d
because it caused build failures when building folks for the first time
without a system-installed copy of the development headers.

Bug 703709 — can't compile folks. fails at GISCAN TpLowlevel-0.6.gir

Add a missing GNU Make dependency for the GIR file.

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

docs: links cannot span multiple lines

docs: correctly delimit links in the tp-zeitgeist comments

docs: only use one copy of the tp-zeitgeist interface

We can't include both tp-zeitgeist and tp-zeitgeist-dummy because valadoc
will complain of re-definitions. Using the dummy version seems to work even
if --enable-zeitgeist is used, so this should cover us in all cases.

docs: fix the path to the backend support library header files

docs: Use the correct Vala version string for valadoc

It was failing to find the expected driver, previously, which broke the
build

build: Remove redundant variable from Makefile.am

telepathy: Fix build with --disable-zeitgeist

Vala doesn’t pass #if conditionals through to generated C code, so when
building folks from a tarball, the --[enable|disable]-zeitgeist option was
having no effect, and Zeitgeist was always being pulled into the build.

Fix this by separating the Zeitgeist code out into a small noinst shim
library and always generating C code for two versions of it: one with
Zeitgeist enabled, and one with it disabled. The correct one’s C code is
then compiled and linked into libfolks-telepathy.la when building from the
tarball.

This is a horrible, ugly fix, but the only better fix is to disable
distribution of generated C code. This was attempted, but thwarted by
automake’s Vala support.

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

telepathy: Move Zeitgeist code entirely into Tpf.PersonaStore

Ready to split it out into a shim library.

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

build: Makefile whitespace fixes

build: Reorder DIST_SUBDIRS to match SUBDIRS

So we don’t encounter problems during `make dist` due to recursing in
a different order from normal.

build: Don’t build Vala headers in a subdirectory unnecessarily

build: Remove generated C/H/vapi/stamp files from MAINTAINERCLEANFILES

As long as --vapi and -H/-h options are correctly passed to a target-specific
VALAFLAGS, automake will automatically add these generated files to the
maintainer-clean target.

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

build: Use target-specific VALAFLAGS rather than AM_VALAFLAGS

This allows automake to parse the flags for each target and generate
appropriate rules. For example, by explicitly specifying “--vapi foo.vapi” in
foo_VALAFLAGS (not just specifying “--library foo”), dependency and clean-up
rules for foo.vapi will be generated by automake. Accordingly, this commit
also adds missing --vapi flags.

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

build: Disable fatal warnings when making distcheck

It’s just too painful. They’re normally turned on during development
anyway.

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

build: Remove redundant entries from GITIGNOREFILES

Now that git.mk has been modified to add better Vala support, these
are unnecessary.

build: Modify git.mk to ignore generated C/H files and Vala .stamp files

These modifications are being submitted upstream.

build: Update git.mk from upstream

Upstream is now located at https://github.com/behdad/git.mk. The file hasn’t
been modified from upstream.

Do not install the TpLowlevel.{gir,typelib}

Adapted from a patch by Markus Lehtonen <markus.lehtonen@linux.intel.com>

Make the last commit change use the same formatting for Tp backend

Link the support library GIRs against libfolks

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

Add namespace and version to libfolks-tracker GIR

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

Add namespace and version to libfolks-libsocialweb GIR

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

Add namespace and version to libfolks-eds GIR

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

Add namespace and version to libfolks-telepathy GIR

Without this in place, JavaScript (and probably other introspected) code
calling Vala client code can end up mysteriously hitting this error:

CRITICAL **: g_irepository_find_by_name: assertion 'typelib != NULL' failed

because it fails to locate the proper typelib.

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

core: Add PresenceDetails::client-types and implement on Individual and Tpf.Persona

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

Post-release version bump

Release version 0.9.3

Added GIR bindings for the backend support libraries

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

Generate FolksTracker gir file

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

Generate FolksLibsocialweb gir file

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

Generate FolksEds gir file

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

Generate FolksTelepathy gir file

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

Clean up version and capitalization for TpLowlevel GIR

This is required to make it actually usable by FolksTelepathy's GIR

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

Split the API version definition so it can be used more extensively

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

Include /folks in PKG_CONFIG_PATH

This is required for the gobject-introspection tools to find the folks
pkgconfig files.

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

Include GIR version number in folks.vapi

The fact that this was missing was causing build warnings for all clients

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

Update NEWS

Partially revert commit c5ada29a75e9c5a1 for the telepathy tests

Fixes #686715

Signed-off-by: Seif Lotfy <seif@lotfy.com>

telepathy: Port to zeitgeist 2.0

Signed-off-by: Seif Lotfy <seif@lotfy.com>

eds: Clarify mapping between properties and vCard fields in EDS

docs: Clarify use of interfaces for property access on Folks.Persona

tests: Fix whitespace in Makefile

Revert "Temporarily work around valadoc crasher with eds docs"

This reverts commit f2e229d963bd6f10a8e9210688cc144e449e756c.

Revert "Temporarily work around valadoc crasher with Telepathy docs"

This reverts commit 0b3481831b43b9bc8dd9133bfe6d78f1a0103fe0.

Post-release version bump

Release version 0.9.2

Temporarily work around valadoc crasher with eds docs

Temporarily work around valadoc crasher with Telepathy docs

Cut excess line

Set release version of new API and update NEWS

Make Telepathy docs build defs stand-alone

Pass the --driver version to valadoc to minimize complications.

This ensures that we're building the library with the same version of Vala
that valadoc uses

Use correct @return instead of @returns (which breaks valadoc)

Bump libebook requirement to un-break GIR bindings.

E.BookClient.connect() had been annotated to return void so this should fix
the issue.

backends: eds: make the personna store backend more 'async'

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

Correctly check for existence of Zeitgeist.

Correct the terminology in Zeitgeist check failure

Remove autodependency on zeitgeist

build fix: include folks-generics in docs arguments

Add warnings (normally fatal) on use of archaic C features

We don't want these even in C code generated by Vala, and certainly not
in hand-written C code.

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=697354
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Reviewed-by: Philip Withnall <philip@tecnocode.co.uk>

Ensure that the C bits of TestCase are pre-declared in a header

This lets us verify that their Vala and C ABIs are the same: if they
are not, compilation will fail.

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=697354
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Reviewed-by: Philip Withnall <philip@tecnocode.co.uk>

Declare internal function folks_persona_store_set_is_user_set_default()

As explained in the comment, I can't find any better way to do this.
Implicit declarations are Bad, so we should be making them into errors
by default.

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=697354
Reviewed-by: Philip Withnall <philip@tecnocode.co.uk>
[added a comment as requested -smcv]
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>

Include ERROR_CFLAGS everywhere we compile C code

AM_CFLAGS was missing from a couple of foo_CFLAGS variables (it isn't
"inherited" automatically), so add that too.

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=697354
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Reviewed-by: Philip Withnall <philip@tecnocode.co.uk>

Re-brand ERROR_CFLAGS to C_ERROR_CFLAGS

These are flags that we only use when compiling hand-written C code,
mostly originating from telepathy-glib. We can't be that strict with
the C code generated by Vala, because it's full of questionable things
(which the Vala compiler verifies are OK, but the C compiler still warns
about).

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=697354
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Reviewed-by: Philip Withnall <philip@tecnocode.co.uk>

TpfPersonaStore: make _supported_fields, _supported_fields_ro private

Making them "internal" means we end up #include'ing small-set.h, which
we don't install, so nothing can be built against the installed
libfolks-telepathy. Nothing outside this file uses those members
directly anyway.

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=697542
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>

update Simplified Chinese (zh_CN) translation

Declare folks_small_set_copy properly

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=687161
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Reviewed-by: Philip Withnall <philip@tecnocode.co.uk>

tests: clear out various D-Bus-related variables

The actual terminal in gnome-terminal 3.8 is an activatable service,
so it has DBUS_STARTER_ADDRESS in its environment. telepathy-glib prefers
DBUS_STARTER_ADDRESS over DBUS_SESSION_BUS_ADDRESS, but GTestDBus
doesn't clear or set the former, only the latter; so we end up trying to
use the real bus when run under a recent gnome-terminal.

See also <https://bugs.freedesktop.org/show_bug.cgi?id=63119>.

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=697261
Reviewed-by: Philip Withnall <philip@tecnocode.co.uk>
[added FIXME comment about #697348]
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>

all tests: disable non-local GIO VFS backends, such as gvfs

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=697261
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Reviewed-by: Philip Withnall <philip@tecnocode.co.uk>

tracker tests: do not request starting all Tracker services

We don't care about tracker-miner-fs etc. for these tests; all we want
to do is stop any running Tracker stuff. D-Bus service activation will
ensure that we get the necessary services when they're needed, without
starting the unnecessary ones.

(This fixes the Tracker tests on a system without tracker-miner-fs.)

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=697261
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Reviewed-by: Philip Withnall <philip@tecnocode.co.uk>

Use SmallSet.copy() where it makes sense

It isn't more than fractionally faster in tests/eds/perf, but might help.

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=687161
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Reviewed-by: Philip Withnall <philip@tecnocode.co.uk>

Add fast-path iteration for SmallSet

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=687161
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Reviewed-by: Philip Withnall <philip@tecnocode.co.uk>

folks, backends: use "small sets" instead of hash sets most of the time

Notable exceptions are:

* sets of personas that are, or might be, the entire set from a backend
  (possible future refinement: use a SmallSet if there aren't many)

* sets of potentially many Individuals (likewise)

* the object cache (potentially pretty large)

* debug stuff (not relevant for performance)

* the set of IM addresses being matched (we want to keep this O(1),
  but it should be a GHashTable)

This speeds up tests/eds/perf by around 5%.

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=687161
Reviewed-by: Philip Withnall <philip@tecnocode.co.uk>
[squashed in responses to review -smcv]
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>

Folks.SmallSet: add and test

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=687161
Reviewed-by: Philip Withnall <philip@tecnocode.co.uk>
[squashed in responses to review -smcv]
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>

contactlist: copy .deps to the same place as .vapi if necessary

valac specifically expects to find them in the same place: it's not
enough to have them both in the search path.

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=697188
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Reviewed-by: Philip Withnall <philip@tecnocode.co.uk>

Compile tp-lowlevel.vapi before libfolks_telepathy_la_vala.stamp

The make rules generated by Automake are all done in terms of a "stamp"
file in the $(srcdir), so if we want to give it more dependencies,
we need to do the same. This fixes a clean out-of-tree build.

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=697188
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Reviewed-by: Philip Withnall <philip@tecnocode.co.uk>

Do not explicitly link libfolks-internal.la into things

libfolks.la embeds a copy of libfolks-internal.la, so every public
(or internal) symbol in libfolks-internal.la is available in
libfolks.la, even if linked with -Wl,--no-copy-dt-needed-entries.

If libfolks-internal.la defines a GObject type, then linking two copies
of it is actively harmful, because they can't both be registered with the
type system.

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=687161
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Reviewed-by: Philip Withnall <philip@tecnocode.co.uk>

backends: when adding a persona by details, do not assume HashSet

All we need is a Set, so we shouldn't need to cast this strictly.

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=687161
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Reviewed-by: Philip Withnall <philip@tecnocode.co.uk>

By default, do debug output iff at least one debug domain is enabled

Previously, debug output (and expensive checks that are enabled alongside
debug output) was always enabled unless explicitly disabled, but
Philip noted that this was not intentional.

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=687161
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Reviewed-by: Philip Withnall <philip@tecnocode.co.uk>

IndividualAggregator: assert and assume that _link_map[key][i] != null

The link map's type is non-nullable, and when we add individuals
to the link map we do so through a function that does not allow
a null argument, so we don't need to sprinkle (!) casts around.

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=687161
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Reviewed-by: Philip Withnall <philip@tecnocode.co.uk>

IndividualAggregator: use a GHashTable<string,GPtrArray> for the link map

The link map is a hot path, particularly when matching multiple copies
of the same contact. Speed this up.

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=687161
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Reviewed-by: Philip Withnall <philip@tecnocode.co.uk>

Added Basque language\nAdded 'eu' (Basque) to LINGUAS

Edsf.PersonaStore: defer contacts-complete like contacts-added

Otherwise, we signal quiescence while not actually quiescent: our
initial batch of contacts could still be coming in.

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=696659
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Reviewed-by: Philip Withnall <philip@tecnocode.co.uk>

Remove a remnant of the avatar coming from the environment

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=690830
Reviewed-by: Philip Withnall <philip@tecnocode.co.uk>

Set up tests' backend paths in Vala code

One less thing in TESTS_ENVIRONMENT.

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=690830
Reviewed-by: Philip Withnall <philip@tecnocode.co.uk>

NEWS: updated to include eds non-blocking change processing

eds: avoid blocking event processing

When there are many incoming D-Bus change notifications, processing of
other D-Bus messages can be delayed considerably. This commit is a
first step towards solving this by caching the change notifications
and processing them with lower priority in a glib idle callback.

Ordering of the changes relative to each other is preserved, so
semantically this is the same as immediate processing.

The worst-case scenario is when contacts get added one-by-one to EDS
address books while folks is running, because then each change
triggers one D-Bus signal. With 2000 contacts and four address books
on a fast laptop, the process hosting folks became unresponsive for
137s. The patch reduced that to 0.1s. The downside is an increase of
total test runtime of 4%.

In the scenario where data is already in EDS when folks starts, the
patch reduced response time from 2.3s to 0.2s.

See https://bugs.freedesktop.org/show_bug.cgi?id=60851#c6 for details.

EDS testing: allow EDS to add additional phone parameters

When EDS is compiled with libphonenumber support, it automatically
computes a normalized E.164 representation of each phone number and
returns it as an additional X-EVOLUTION-E164 parameter.

This broke the strict phone_fd.equals check in set-phones, because it
compared against a PhoneFieldDetails without that parameter. Now the
test only looks for a phone number with "1234" as value and checks for
the expected type after returning from the event loop.

testing: avoid false warning

When FOLKS_BACKEND_PATH is set to the build dirs during testing, the
"content type of '%s' appears to be '%s' which looks suspicious. Have
you installed shared-mime-info?" warning gets triggered repeatedly,
because these dirs contain all kinds of files (executables, compiler
dependency files, etc.) which are not expected in the normal install
location.

This is annoying when running a test with "make <test>.gdb" because
gdb stops on a glib trace point for each warning. Therefore disable
the warning when FOLKS_BACKEND_PATH is set. Extending the whitelist
has the drawback that it might not catch all files that can appear
there in the future.