Silence generally-useless Tp test message.

Don't add a duplicate candidate individual if one matches by user and IID

If an individual matches a persona being aggregated both by them both being
the user, and by IID match, don't list the individual as a candidate
individual twice (which would cause their personas to appear twice in the
final individual).

Added valgrind support in make check.

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

Add policy for keeping the build healthy.

Finish catching BackendStore up to our coding style.

Fix dependencies for internal g-i files.

As it was, these libraries wouldn't be re-created after a maintainer-clean (or
on the very first build).

Also print a stack trace upon a failed test assertion.

Don't double-list files in MAINTAINERCLEANFILES.

Distribute sources for generated files.

This un-breaks distcheck by ensuring the generated files don't stick around in
_build after the final distclean.

This fulfills the requirements laid out in the Automake manual page:
http://www.gnu.org/software/hello/manual/automake/distcleancheck.html

Define the Telepathy BACKEND_NAME directly.

Properly distributing the backend.mk file proved more difficult than it was
worth (it was breaking 'make check' during distcheck), so we've reverted to
direct definition.

Force Vala for non-release builds.

This should make buildbot builds a little more consistent.

Add Vala requirement checks for newer tests.

Bug 639113 — Add static aggregation tests

Add some tests which cover several of the ways which personas can be
statically aggregated at present. These tests are “static” (as opposed to
“dynamic”) because they don't change the inputs to the aggregator after
it's been prepared. Consequently, they don't cover much of the interesting
and bug-prone behaviour of the aggregator. Still, they're a start.

Closes: bgo#639113

Don't announce removal of Individuals twice.

Before this patch, we would announce an Individual's removal, "dispose" of the
individual, then announce their removal again (at which point they would have
zero Personas, which can make it harder for clients to do anything useful with
the notification anyhow).

Helps: bgo#639113

Allow multiple test accounts to be connected simultaneously

This doesn't affect any existing tests, but will be useful for testing that
aggregation across different persona stores works correctly.

Override the test log handler for Telepathy tests.

This avoids aborting and printing unnecessary stack traces for the
well-known Telepathy-logger warning message.

Fixes bgo#639195 - Print stack traces for failed tests to improve
remote debugging

Factor Tp logger warning out of Tp tests.

Helps bgo#639195 - Print stack traces for failed tests to improve
remote debugging

When a test would print a message >= warning, print a stack trace.

This is largely to benefit remote debugging of build bots but also makes
debugging test errors easier.

Helps bgo#639195 - Print stack traces for failed tests to improve
remote debugging

Cut unnecessary log handler for key-file test.

Helps bgo#639195 - Print stack traces for failed tests to improve
remote debugging

Cut unnecessary Tp log handler for the general test.

Helps bgo#639195 - Print stack traces for failed tests to improve
remote debugging

Note the fix of bgo#638609 in the NEWS

Register core debugging as global G_LOG_DOMAIN to reduce magic strings.

Fixes bgo#638609 - libfolks hard-codes backend names for debugging

Pin backends' names to global BACKEND_NAME to reduce magic strings.

Helps bgo#638609 - libfolks hard-codes backend names for debugging

Make the main debug domain 'folks' to match the backends.

Helps bgo#638609 - libfolks hard-codes backend names for debugging

Make the backends' log domains match their type_id.

Helps bgo#638609 - libfolks hard-codes backend names for debugging

Push flag setting into the Debug constructor.

Helps bgo#638609 - libfolks hard-codes backend names for debugging

Add new API for setting debugging levels.

Helps bgo#638609 - libfolks hard-codes backend names for debugging

Make Folks.Debug a class so we can store state.

Helps bgo#638609 - libfolks hard-codes backend names for debugging

Don't use a magic string in the Tp backend.

Add a comment for bgo#638609 - backend debug hard-coding

Bug 638395 — folks-inspect prevents default Vala-free builds

Add notices and errors to the configure script to notify the user of why
the inspect tool is not being built. Closes: bgo#638395

Add a missing backslash to the configure.ac file

Don't break the build if building folks-inspect without Vala

Fix some variable names in the inspector signal parsing code

See: https://bugzilla.gnome.org/show_bug.cgi?id=629075#c8

Bug 629075 — Add folks command line application

Add an interactive command line inspection utility for libfolks.
Closes: bgo#629075.

Add NEWS entry for bgo#629083 fix

Add NEWS entry for bgo#638311 fix

Refer to non-local variables and methods with full name.

The benefit here is to make it easier to tell when variables are local. Methods
are referred to similarly for consistency.

Fixes bgo#629083

Cut unnecessary casting.

Excessive casting can hide real compiler warnings/errors and is best avoided.

Helps bgo#629083

Maximize use of 'unowned' keyword.

This is used when (and only when) variables:
1. would avoid a non-trivial copy (eg, a string or array but not an int or
object ref-count increase)
AND
2. will only be assigned unowned values (excessive use of 'unowned' here will be
caught by the compiler)

The net benefit is avoiding unecessary memory allocation/freeing.

Helps bgo#629083

Maximize use of the 'var' keyword.

This is used whenever a variable:
1. is declared and initialized in the same line
AND
2. would not avoid a copy by using the 'unowned'. In the future, we may be able
to declare variables 'unowned var', in which case this point won't matter. See
bgo#638199.

The net benefit is less noise on variable declaration lines.

Helps bgo#629083

Ensure private and internal methods begin with a _

Helps bgo#629083

Ensure private and internal data members begin with a _

Helps bgo#629083

Write down development policies.

Fixes bgo #638311.

Bug 637240 — libfolks-telepathy.so exports private symbols

Limit the symbols exported by the libfolks-telepathy.so library to those
starting with “tpf_”. Based on a patch by
Emilio Pozuelo Monfort <pochu27@gmail.com>. Closes: bgo#637240

Post-release version bump

Release version 0.3.3

Add missing files to POTFILES.in and POTFILES.skip

Remove outdated translation files from EXTRA_DIST

They're no longer copied into the build tree since we no longer call
autopoint. See: bgo#635649

Ensure each Persona is added to the relinked_personas list at most once

In some cases when relinking the user's Individual when taking Empathy
offline, several of the user's Personas can get into the relinked_personas
list multiple times. This causes an infinite loop of linking and relinking
those Personas. Closes: bgo#637136

Bug 636714 — Assertion failure on invalid IM address

Handle invalid IM addresses in relationships.ini more gracefully by throwing
an exception from IMable.normalise_im_address() if it detects an invalid
address, rather than aborting due to an assertion failure. Closes: bgo#636714

Don't trust personas from IRC

Rename the Favourite interface to Favouritable

Closes: bgo#627397

Rename the Presence interface to HasPresence

Helps: bgo#627397

Rename the Avatar interface to HasAvatar

Helps: bgo#627397

Add recent bug fixes to NEWS file

Add documented libraries to LDFLAGS in the documentation Makefile.am

gtkdoc-scangobj expects to have the libraries it's linking its scanner program
to present in LDFLAGS. gtk-doc.make normally does this automatically for
normal gtk-doc-enabled projects, but we need to do it manually.

Add missing documentation for method parameters

Fix documentation of enum members

The documentation comments for enum members have to be proper valadoc comments
(i.e. starting with /** and ending with */ on separate lines).

Fix string length calculation in the BackendStore for Vala 0.12

Remove use of deprecated GLib API in Vala 0.12

Remove use of Unicode characters in documentation comments

The normal valadoc doclet is fine with them, but the gtkdoc doclet doesn't
like them. See: bgo#637072

Generate gtk-doc for libfolks and libfolks-telepathy.

Don't delete in-tree m4 macros when doing maintainer-clean

Remove libtool m4 macros from the git tree

They're copied into the build tree from the system libtool installation by
libtoolize, and don't belong in git.

Bug 636251 — Fails to add contact

Vala compiles this line of code wrong, and tries to double check
that there is a GValue boxed inside of the GValue.

Closes: bgo#636251

Bug 635823 — Link telepathy test program to GIO

Closes: bgo#635823

Bug 635703 – Leak in connection_get_requestable_channel_classes_cb

Tidy up memory handling in the function so that it doesn't leak in odd
conditions. Closes: bgo#635703

Fix return types and documentation for a tp-lowlevel.c method

folks_tp_lowlevel_connection_get_requestable_channel_classes_finish() had
incorrect return values in assertion failure conditions, and the
documentation said to use the wrong function to free its return value.

Bug 635649 — autopoint fails to run

Since the changes due to bgo#635134, we should no longer call autopoint in
autogen.sh. Closes: bgo#635649

Don't use IT_PROG_INTLTOOL and AM_GNU_GETTEXT

They conflict. See bgo#635134.

Fix leak in tp-lowlevel.c

Closes: bgo#635178

Release 0.3.2

Relocate the g-i scanner args in configure for consistency.

Fix syntax errors for the release version of configure.

Ensure vapi files have an appropriate age.

Vapigen doesn't update the age of vapi files if they don't change. This means
that make will always generate them during distcheck, implicitly requiring
vapigen.

Helps bgo#633718.

Cast Individuals to Presence as necessary for the tests.

Fix notification order in backends

Remove the exception from Tpf.Persona's constructor

It was not possible for it to ever be thrown (since TpContact.get_identifier()
is guaranteed to always return a valid ID, and having *_new() methods throw
exceptions is never a good idea. (GInitable should be used instead.)

This breaks both the Vala and C APIs.

Helps: bgo#629078

Hide setter for Persona.linkable_properties

Remove casting convenience functions in Individual

They're not needed for Vala, and weren't particularly well thought
out (or complete) in C.

Hide setter for Backend.persona_stores

Hide setter for Backend.name

Hide setters for PersonaStore.type_id, .display_name and .id

Ensure we use string's byte length in both Vala 0.10 and 0.12

string.length gives the byte length in Vala 0.12 and string.size() is
deprecated, however string.length gives the character length in Vala 0.10,
so we need to conditionally change the code for Vala <= 0.10.
Re-closes: bgo#628930

Consolidate persona link removal code

Remove links to personas before they're relinked in the aggregator

If one persona is removed from a non-singleton individual (e.g. because they
went offline), the other personas in the individual will be relinked into new
individuals (singleton or not). We previously didn't ensure that the entries
in the link map corresponding to these personas were removed before the
personas were relinked which, under certain circumstances (which I haven't
fully explored), could cause the personas to be relinked to the individual
which had just been removed. This would cause the individual's persona list
to grow exponentially, and generally Bad Things to happen.

I could reproduce this situation by logging into two accounts, both of which
had the other added to their roster. The resulting four personas corresponding
to the two accounts (2 user personas and 2 normal personas) would be linked
to form one user individual. Going offline and then online again would trigger
the bug.

Closes: bgo#633080

Combine personas_changed() emissions for Individuals replaced due to relinking

This re-fixes bgo#632564 so that the optimisation of grouping emissions of
the personas_changed() signal is preserved. Closes: bgo#632564

Improve debug output of the aggregator

Add an indication as to whether personas are the user.

Note the recent build clean-ups in the NEWS.

Un-break distclean for the general test library

Un-break distclean for the Telepathy test library

Un-break distclean for the tests

Un-break distclean for the Telepathy backend

Distribute the key-file test library VAPI

Add the Telepathy test library g-i and Vala files to DISTCLEANFILES

Add missing general test VAPI distribution.

Simplify the tp contactlist test lib build

Simplify the Telepathy backend Makefile.

Remove deprecated use of string.size()