Bump to 0.9.6

Change-Id: I9ee797d5c131ac8aef7482b6ac8082476073299c
Signed-off-by: Philippe Coval <philippe.coval@open.eurogiciel.org>

minimal build

Disable most backends by default. Building tests depends on telepathy
and is not needed, so disable it always.

This commit uses bcond_with to disable the features by default.

folks-uninstalled.pc.in: does not depend on telepathy

The dependency on telepathy-glib is wrong (does not show up in
installed version either) and causes build failures when compiling
without telepathy.

Exclude TpLowlevel typelib file

Which would make the package uninstallable.

Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>

changes: spec: adapt and workaround build issue

spec: adapt and workaround build issue

changes: Bump to 0.9.3

Bump to 0.9.3

Fix packaging: Update module_version value

changes: set zeitgeist-2.0 build dependency

spec: set zeitgeist-2.0 build dependency

changes: add glib2-tools required dependency

add glib2-tools required dependency

changes: Bump to 0.9.1

add libfolks ofono backend

Bump to 0.9.1

changes: Workaround : Add --disable-fatal-warning

Cleanup specfile

Workaround : Add --disable-fatal-warning

Add --disable-fatal-warning in configure option to fix the build issue with evolution-data-server

Fix: replace glib2 macro

Clean up spec file

Enabled evolution-data-server backend again

Fix: lang package dependency

remove evolution-data-server backend

Bump to 0.8.0

add: autogen

Inital packaging on 0.8.0

Release version 0.9.6

docs: Fix links in documentation comments

This prevents valadoc throwing more of a hissy fit than usual.

Updated Polish translation

tests: Add missing valgrind.mk include to test Makefile.ams

tests: Remove unused variables from Makefile.ams

eds: Remove unused -include in Makefile.am

build: Add folks-internal.la to $(backend_libadd)

build: Drop GLib dependency to ≥ 2.38.2 (from 2.39.0)

The GLib patches needed for the BlueZ backend to work properly have been
backported to GLib 2.38, and will appear in the GLib 2.38.2 release.

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

bluez: Prevent blocked Bluetooth devices from becoming persona stores

If the user has explicitly blocked a Bluetooth phone from their laptop, they
don’t want to try downloading contacts from it.

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

docs: Update NEWS with recent changes

ofono: Throw a useful error if prepare() fails for a missing D-Bus service

Previously the error was completely unhelpful.

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

core: Bless DBusError.SERVICE_UNKNOWN for D-Bus service problems on prepare()

As folks acquires more D-Bus-enabled backends, we need a common error code
for when the service can’t be started. In the Backend.prepare() documentation,
bless DBusError.SERVICE_UNKNOWN as this error code.

Also fix the BlueZ backend to emit it instead of a custom error code.

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

Updated Indonesian translation

Updated Lithuanian translation

Updated Serbian translation

Updated Brazilian Portuguese translation

Updated Brazilian Portuguese translation proofread by Enrico Nicoletto

Updated Slovenian translation

Updated Galician translations

Updated Galician translations

Updated Polish translation

bluez: Add a Bluetooth Phonebook Access Profile backend using BlueZ 5

This pulls contacts out of a paired Bluetooth device and dumps them in
folks.

No test cases are included.

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

This bumps the Vala and GLib dependencies of folks, needed for the following
two fixes.
 • https://bugzilla.gnome.org/show_bug.cgi?id=710643
 • https://bugzilla.gnome.org/show_bug.cgi?id=710726

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

build: Fix PKG_CONFIG_PATH for calls to g-ir-scanner

Ensure that g-ir-scanner’s PKG_CONFIG_PATH includes the build
directory containing the generated -uninstalled.pc files for folks,
so that the backends can link against it properly for being scanned.

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

backends: Fix paths in *-uninstalled.pc.in pkg-config files

Copy–paste errors.

build: Factor common automakery out of backends’ Makefile.ams

This makes the backends’ build process so much more consistent, and reduces
a huge amount of copy–paste automakery.

backends: Remove BackendFactory classes

They were constructed, made one method call in their constructor, and then
sat there for the lifetime of the process, using a little bit of memory each,
and not doing anything else.

Instead, just add the new Backend to the BackendStore directly from the
module initialiser. This cuts out a reasonable amount of generated C, and
slightly reduces memory overheads. Yay!

tests: Fix missing backslash in EDS tests Makefile.am

build: Enable colourised output from GCC

If GCC supports it, enable the -fdiagnostics-color=auto option, which adds
colour to warnings and errors outputted by GCC. Enable this for both generated
and non-generated C code.

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

build: Disable some GCC warnings for generated C code

The GCC warning spew which results from Vala’s non-perfect generated C
code is distracting and can mask genuine warnings from valac or GCC.

This patch adds some #pragmas in warnings.h which disable as many of these
warnings as possible. Unfortunately some of the warnings are for const
qualifier discards and passing parameters of the wrong type, which (as far
as I know) can’t be disabled.

In any case, this patch eliminates GCC warnings for several C files, which is
better than nothing.

Note that warnings.h should only be included in build targets which compile
only Vala, and don’t compile any non-generated C code. Non-generated C code
should always be compiled with all warnings enabled, to catch legitimate
errors. See the comment at the top of warnings.h.

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

build: Whitespace cleanup in various Makefile.am files

telepathy: Remove redundant entries from a CPPFLAGS variable

They’re already in AM_CPPFLAGS, which is substituted into that CPPFLAGS
variable.

core: Remove an excess parameter from a printf()-format function call

core: Add [PrintfFormat] attributes to some internal debug methods

core: Eliminate a shadowed variable

Vala is fine with this but GCC complains. Let’s keep GCC happy.

test: bump the timeout for the Tracker favourite-updates test

This may be a little excessive but smaller amounts occasionally caused a
failure on timeout.

In theory, this shouldn't matter too much, since the test exits early once
conditions are satisfied.

Fix phone normalization

This make the code actually match the algorithm described on
http://blog.barisione.org/2010-06/handling-phone-numbers/

It also use a StringBuilder instead of doing tones of string copy.

backends: Work around a Vala const-array null-termination bug

Bug #682698 means that Vala does not null-terminate const arrays, yet it
expects them to be null terminated when returning them to g_object_get().
This results in g_object_get() walking off into arbitrary memory, and Bad
Things happen shortly afterwards.

Work around this (because it doesn’t look like the Vala bug is going to be
fixed any time soon) by manually null-terminating all const arrays (typically
linkable_properties and writeable_properties lists in backends) and modifying
code which iterates over them to ignore null entries.

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

ofono: Skip whitespace between vCards when splitting them up

This eliminates the following libebook warning which we were triggering.
libebook itself skips the whitespace, so this introduces no functional
changes.

    libebook-contacts-WARNING **: vcard began without a BEGIN:VCARD

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

telepathy: Fix tp-zeitgeist-dummy to match changes in b38f7126

Stop using lowlevel telepathy-glib API

core: Connect to Persona::notify, not all its details, in Individual

In practice, we care about basically every property anyway, and
tests/eds/perf was spending more than 6% of its time connecting
to these signals. Just connect to notify and demultiplex through a
lookup table for about a 5% speed-up.

In principle we could make the lookup table into a
hash table if it proves to be worth it, but life is probably too short.

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

Bug 709674 — Ignore non-phonebook-enabled modems in ofono backend

Ignore ofono modems which don’t support the org.ofono.Phonebook interface,
which is needed for accessing contacts stored on the SIM.

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

Use autotools's parallel test harness

We were already unintentionally using it in some cases (depending upon the
developer's system) without the proper safeguards in place. Without them,
the EDS and Tracker tests will mysteriously result in a ton of segfaulting
background processes because they can't be safely run simultaneously.

The upside of using the parallel harness is that it's still safe to use for
the simpler test suites (eg, key-file, folks, telepathy) and should speed up
their runtimes quite a bit. It also adds cleaner logging facilities and
makes the default output a little cleaner.

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

Use git.mk's definitions to simplify configure.ac

Upgrade git.mk

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

ofono: Work around a Vala bug causing a double-free when adding a modem

Work around Vala bug #709665:
https://bugzilla.gnome.org/show_bug.cgi?id=709665
which allowed an assignment of a string*[] value to a string[] variable.
string*[] values need the container freeing, but not the members of the
array. However, since the variable was a string[], Vala generated C code to
free both the container and the members, resulting in a double-free of each
member.

Telepathy: use can_set_contact_alias() instead of fetching flags

Tests: Do no hardcode individual sha1

With telepathy 1.0 the account prefix is going to change, so all
sha1 will have to be recalculated. It's better to calculate them
so changing one line is enough to port to tp1.

tests: fix trivial whitespace in Makefile

Bug 705403 — Use GLib.BytesIcon instead of Edsf.MemoryIcon

Use GLib.BytesIcon internally instead of Edsf.MemoryIcon, which can
be deleted. This bumps our GLib requirement to 2.37.6, but doesn’t
affect API.

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

Update NEWS

Stop using deprecated EDS API

[l10n]Added Turkish translation

build: Generate backends’ GIR files using valac rather than g-ir-scanner

Vala knows more about the types used in the code, and doesn’t emit any
GIR annotations; so g-ir-scanner is left guessing about (transfer)
annotations (amongst others).

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

Updated Basque language

Finnish translation by Jiri Grönroos

Set the GIR namespace and version for folks-generics

This ends up avoiding a valadoc crasher (bgo#706857) which the maintainers
can't reproduce.  Strictly speaking, we should have it in place anyhow, so
it should be pretty safe.

It also ends up passing these details along to folks-internal.vapi when it's
generated as well.

build: Fix autogen and configure

Include ACLOCAL_FLAGS when setting. ACLOCAL_AMFLAGS.

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

update NEWS

PhoneFieldDetailsTests: make constant static strings unowned

This works around Vala bug GNOME#646970, which is only fixed in a
development version (0.21). Newer Vala versions automatically make
these strings behave as if unowned anyway.

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

Revert "Temporarily disable the Tracker tests, as they're currently broken"

This reverts commit bfa9bad7c100e2f6cdbd8e42367fa515df5470d4.

Revert "Temporarily disable docs for distcheck due to bgo#706857"

This reverts commit 2c13f603fc0b865968cb581a40a0edcb450c9c7b.

Post-release version bump

Release version 0.9.5

Temporarily disable the Tracker tests, as they're currently broken

Temporarily disable docs for distcheck due to bgo#706857

Do not distribute _gen/param-spec-struct.h in tarballs or scan it with g-i

It isn't self-contained, and provokes a (non-fatal) syntax error from
g-ir-scanner.

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

Folks.SmallSet: return a value

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

core: Tidy up documentation in LocationDetails

use IndividualAggregator.dup () in tests and tools

The default constructor has been deprecated.

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

aggregator: add dup() methods returning a singleton

As discussed on bgo#705289 we don't support having different aggregator
instances as they would share the same set of personas. Adding this singleton
makes things easier for client applications.

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

Revert "add unit test for bgo#705289"

It doesn't make sense any more as multiple aggregators are not supported any
more.

This reverts commit 1b7e8d83bb7488aac7bcf2d59df3a7b35f316ae2.

Update Chinese simplified translation