EGoaClient nitpicks.

Bug 698726 - Handle goa-daemon crashes/restarts gracefully

EGoaClient is an improved GoaClient that handles daemon crashes and
restarts gracefully so E-D-S accounts are not destroyed unnecessarily.

Updated Czech translation

Tamil Translation Updated

Updated Greek translation

Improve error handling in imapx_server_get_message().

If a GET_MESSAGE job for the same UID is already in progress, wait for
the job to finish, but don't assume it was successful.  Try retrieving
the message from cache, but if that fails then we have to re-fetch the
message from the IMAP server.

ECalClient test: test-client-get-revision not broken anymore

Not sure how this is passing again, but it is.

Do not consider PGP and S/MIME subparts as attachments

test-client-custom-summary.c: Optimize test.

Reuse the same two addressbooks for every test, so that only the
first 2 tests populate the addressbooks with contacts and the remaining
tests only create a client and execute a query.

Also add all test contacts with a single batch command and dont fetch
the test contacts, also removed the #if 0'd case which is the only
case which required the local contact copies.

test-server-utils: Allow reuse of existing addressbook.

If committing the new scratch source ends with an error that
the addressbook/calendar already exists, then just use the existing
source instead of bailing out.

This is needed for tests which reuse the addressbook from a previous test.

Conflicts:

tests/test-server-utils/e-test-server-utils.c

Updated Spanish translation

[l10n] Updated Catalan (Valencian) translation

[l10n] Update Catalan translation

ECalClientView: Remove unnecessary fetching of ECalClient.

Left over from the D-Bus error stripping work.

EBookClientView: Remove unnecessary fetching of EBookClient.

Left over from the D-Bus error stripping work.

Bug #698261 - Skip invalid flags in imapx_parse_flags()

Previous patch had incorrectly implemented skip of the broken flag,
which fixes this change.

Bug #698261 - Skip invalid flags in imapx_parse_flags()

CamelStoreSettings::filter-inbox: Change default to TRUE

LC_ADDRESS is a GNU extension. Treat it as such.

Unconditional use of the GNU extension LC_ADDRESS causes compiler
errors on non-GNU systems.

In test cases, use LC_MESSAGES if LC_ADDRESS is unavailable.
Also fail gracefully with a log message if setlocale() fails.

ESource: Move "remember-password" to Authentication extension.

It's a more suitable place since the option is only relevant to
EAuthenticationSession.

Bug #691939 - Password prompt has "save password" always checked

Remove backward-compatibility cruft for goa-1.0 < 3.8.

In particular, GOA's Google provider uses OAuth 2.0 now, so we can
drop the OAuth 1.0a support in EGDataGoaAuthorizer and with it the
last lingering goa-1.0 dependency outside of the GOA module.

Bump minimum goa-1.0 requirement to 3.8.

Still an optional dependency.  This is to purge backward-compat cruft.

Switch NULL to empty string in e_util_utf8_{data_}make_valid()

It's used on various places which expect a valid string, but which
doesn't check for NULL values of the string, which can cause a crash,
like xtian found with e_data_cal_respond_get_object(), which can have
passed in a NULL object. This change is also to avoid NULL-checking
on too many places.

EBookBackendSqliteDB: Avoid summary introspection on initial creation.

The summary introspection does not introspect whether regular prefix
indexes are created, which is acceptable since it does not effect queries.

However since we introspect before creating the indexes, we lose the
configured index information which is needed to initially create those
indexes.

This patch simply checks if the table exists when initially creating
the summary and avoids the introspection if the table is being created
for the first time.

Automatically strip D-Bus info from error messages.

I mistakenly assumed code generated by the gdbus-codegen tool would
automatically strip off the D-Bus error name from error messages, but
that apparently is not possible according to my discussion with David
Zeuthen in [1], since it would break g_dbus_error_is_remote_error().

Distinguishing between local and remote errors strikes me as a rarely
needed corner case, and is certainly not worth the extra step imposed
on applications when dealing with something as fundamental as GError.

This commit prevents the D-Bus error name from leaking through in any
Evolution-Data-Server client-facing library functions.

Also rename any stack-allocated GError pointers to 'local_error' to
help distinguish them from GError "out" parameters which are always
named 'error'.  This is just to improve variable name consistency.

[1] https://bugzilla.gnome.org/697819

Bug 695578 - Trailing space in translatable string

Bug 697705 - e_cal_client_remove_objects_sync() crashes on empty UID

Test suite cleanups.

build: Fix srcdir != builddir from git

We can't do (cd $(srcdir); ...) and inside reference $(top_srcdir)
because that variable uses a *relative* path.  Thus we copy the
approach from gnome-shell of explicitly using addprefix to append the
source directory.

Bug #696757 - Do not use dynamic keys in g_object_set_data()

Bug #681837 - Make enum type registration thread safe

Bug 697734 - Memory leak in local calendar backend

We were leaking an icalproperty while seeding the "revision" D-Bus
property during instance initialization, because the backend does
not yet have an icalcomponent to hold the icalproperty.

This commit avoids the leak during instance initialization, but also
makes sure the "revision" property gets updated once the backend does
have an icalcomponent.

ECalBackendFile: Indentation cleanups.

Tabs, not spaces.

Updated Hungarian translation

CamelIMAPXServer: Rewrite duplicate message fetching.

The g_cond_wait() call in imapx_server_get_message() seems to be the
source of many deadlocks.  Possibly because some of the error handling
branches in that function fail to broadcast the GCond being waited on,
and cancellation is a very common error condition, especially when the
user is rapidly browsing through messages.

The code is attempting to wait for the completion of a GET_MESSAGE job
for the same UID which is already queued.  It's doing this by defining
its own GMutex and GCond, but it turns out we already have a function
for this: camel_imapx_job_wait().

Let's try using camel_imapx_job_wait() there.  It can't do worse than
the current situation, and we can kill off the specialized GMutex and
GCond variables which were unfortunately made public.

Bug #271262 - Allow Send/Receive of local stores in offline

EBookBackendWebdav: Defer to default get_contact_list_uids_sync().

EBookBackendGoogle: Defer to default get_contact_list_uids_sync().

EBookBackend: Make get_contact_list_uids_sync() optional.

This method is now optional.  By default, get_contact_list_uids_sync()
simply calls get_contact_list_sync() and extracts UID strings from the
matched EContacts.  Subclasses may override this if they can implement
it more efficiently.

EBookBackend: Minor code rearrangement.

EBookBackendGoogle: Implement the new backend API.

At the same time I converted back to using synchronous GData functions,
which makes the logic flow much easier to follow (in my opinion).

I'm not sure why the asynchronous functions were used in the first place
when each backend method is invoked from a worker thread where it's safe
to block.  Maybe the threading model was different at the time, I really
don't know.  The commit history does not reveal the rationale.

EBookBackendGoogle cleanups.

Updated Danish translation

EBookBackendWebdav: Remove unused macros.

EBookBackendWebdav: Implement the new backend API.

EBookBackendWebdav cleanups.

Updated POTFILES.in

sqlite addressbook: fix memory corruption in get_revision

Instead of setting a pointer, the pointer pointing to the pointer was
overwritten. As a result, e_book_backend_file_load_revision() always saw a
NULL pointer when opening the address book (regardless whether in
evolution-addressbook-factory or a DRA client) and used
e_book_backend_file_bump_revision() to write a new revision.

It is not entirely clear whether overwriting the wrong pointer (which was on
the stack and thus should not have been that important) or these concurrent
writes were to blame, but testing shows that this fix solves a mysterious
sqlite query failure ("corrupt database") in the DRA client when running
SyncEvolution's testpim.py TestContacts.testFilterStartupRefineSmart.

Fixes BGO #697173.

Remove EBookBackendSync.

Useless subclass, added no value other than a slightly less awkward API.

EBookBackendFile: Implement the new backend API.

EBookBackendFile cleanups.

EBookBackend: Add the new backend API alongside the old one.

The new and old EBookBackend class methods will co-exist temporarily,
so I can start converting the simpler EBookBackendSync-based backends
and make final adjustments to the new API.

Remove e_data_cal_is_opened().

With the new public API in place, ECalBackend can own its "opened"
state again.

Remove e_data_book_is_opened().

With the new public API in place, EBookBackend can own its "opened"
state again.

EBookBackend: Fix a GError typo.

Remove e_cal_backend_sync_set_lock().

ECalBackendSync purports to offer serialized method dispatching but
implements it completely wrong.  Backends which rely on this should
instead set use_serial_dispatch_queue in their class_init() method.

ECalBackendClass: Add a 'use_serial_dispatch_queue' option.

ECalBackend subclasses can set this to TRUE to use a serial dispatch
queue, instead of a concurrent dispatch queue.  A serial dispatch queue
executes one method at a time in the order in which methods were called.
This is generally slower than a concurrent dispatch queue, but can help
ECalBackend subclasses avoid thread-safety issues.

EBookBackendClass: Add a 'use_serial_dispatch_queue' option.

EBookBackend subclasses can set this to TRUE to use a serial dispatch
queue, instead of a concurrent dispatch queue.  A serial dispatch queue
executes one method at a time in the order in which methods were called.
This is generally slower than a concurrent dispatch queue, but can help
EBookBackend subclasses avoid thread-safety issues.

ECalClient: Use G_DBUS_MESSAGE_FLAGS_NO_REPLY_EXPECTED for close().

Invoke the close() method without a callback function so the server
knows not to reply.  ECalClient tests that use GTestDBus generally
close their GDBusConnection immediately after disposing ECalClient,
and because we call close() asynchronously the server sometimes has
no chance to reply before the GDBusConnection closes, resulting in
a spurious console warning during the test.

EBookClient: Use G_DBUS_MESSAGE_FLAGS_NO_REPLY_EXPECTED for close().

Invoke the close() method without a callback function so the server
knows not to reply.  EBookClient tests that use GTestDBus generally
close their GDBusConnection immediately after disposing EBookClient,
and because we call close() asynchronously the server sometimes has
no chance to reply before the GDBusConnection closes, resulting in
a spurious console warning during the test.

Updated Basque language

EBookBackendFile: Prefer EClientError domain.

The E_DATA_BOOK_ERROR domain will die when I'm finished with the backend
API rewrite, but I'm not quite ready to go all out with it yet.  This is
just to make the direct access mode tests happy again.

Remove e_return_data_cal_error_val_if_fail() macro.

No longer used, and backends had been mainly misusing it anyway.

Remove e_return_data_cal_error_if_fail() macro.

No longer used, and backends had been mainly misusing it anyway.

Remove e_return_data_book_error_val_if_fail() macro.

No longer used, and backends had been mainly misusing it anyway.

Remove e_return_data_book_error_if_fail() macro.

No longer used, and backends had been mainly misusing it anyway.

ECalBackendSync: Prefer EClientError domain.

Also, don't return GErrors for argument checks.

EBookBackendSync: Prefer EClientError domain.

Also, don't return GErrors for argument checks.

ECalBackend: Prefer EClientError domain.

EBookBackend: Prefer EClientError domain.

EDataCal: Move data_cal_convert_to_client_error() calls.

Move these calls out of the "respond" functions so we don't lose them as
we remove the "respond" functions.  Instead, place the calls immediately
before g_dbus_method-invocation_take_error().

EDataBook: Move data_book_convert_to_client_error() calls.

Move these calls out of the "respond" functions so we don't lose them as
we remove the "respond" functions.  Instead, place the calls immediately
before g_dbus_method_invocation_take_error().

ECalBackend: Convert get_backend_property() method.

This method does not block, so simplify its signature and ditch the
asynchronous wrappers.  Also, ECalBackendSync does not need its own
get_backend_property_sync() method.

Removed functions:

    e_cal_backend_get_backend_property_sync()
    e_cal_backend_get_backend_property_finish()
    e_cal_backend_sync_get_backend_property()
    e_data_cal_respond_get_backend_property()

EBookBackend: Convert get_backend_property() method.

This method does not block, so simplify its signature and ditch the
asynchronous wrappers.  Also, EBookBackendSync does not need its own
get_backend_property_sync() method.

Removed functions:

    e_book_backend_get_backend_property_sync()
    e_book_backend_get_backend_property_finish()
    e_book_backend_sync_get_backend_property()
    e_data_book_respond_get_backend_property()

ECalBackend: Make the "cache-dir" property thread-safe.

New function:

    e_cal_backend_dup_cache_dir()

EBookBackend: Make the "cache-dir" property thread-safe.

New function:

    e_book_backend_dup_cache_dir()

Remove e_cal_backend_set_is_removed().

Not used by any current backend implementations, and I can't even make
sense of the documentation for it.

Remove e_cal_backend_is_removed().

Not used by any current backend implementations, and I can't even make
sense of the documentation for it.

Remove e_book_backend_set_is_removed().

Not used by any current backend implementations, and I can't even make
sense of the documentation for it.

Remove e_book_backend_is_removed().

Not used by any current backend implementations, and I can't even make
sense of the documentation for it.

ECalBackend: Remove set_backend_property() method.

No longer used.

EBookBackend: Remove set_backend_property() method.

No longer used.

ECalBackend: Remove CLIENT_BACKEND_PROPERTY_CACHE_DIR definition.

Backends should use the ECalBackend:cache-dir property directly.

ECalBackend: Remove CLIENT_BACKEND_PROPERTY_READONLY definition.

Backends should use the ECalBackend:writable property directly.

ECalBackend: Remove CLIENT_BACKEND_PROPERTY_ONLINE definition.

Backends should use the EBackend:online property directly.

EBookBackend: Remove CLIENT_BACKEND_PROPERTY_CACHE_DIR definition.

Backends should use the EBookBackend:cache-dir property directly.

EBookBackend: Remove CLIENT_BACKEND_PROPERTY_READONLY definition.

Backends should use the EBookBackend:writable property directly.

EBookBackend: Remove CLIENT_BACKEND_PROPERTY_ONLINE definition.

Backends should use the EBackend:online property directly.

e_data_cal_view_new: Add documentation.

e_data_book_view_new: Update documentation.

[l10n] Updated German translation

Fix implicit declaration of WIFEXITED.

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

Updated Hungarian translation

Simplify direct access to address books.

EBookBackend's new public API allows Direct Access Mode to bypass
EDataBook and talk to the EBookBackend directly, which is appropriate
since EDataBook is just glue code for remote D-Bus method invocations.

Modernize ECalBackend's public API.

This leaves ECalBackend and EDataCal engaging in a somewhat ridiculous
kind of ping-pong game for the moment, but the desired public API (in as
much as a backend has a public API) is starting to take shape.

With the scaffolding now in place, we can begin converting backends to
use modern GIO conventions one method at a time, and eventually remove
all the "respond" functions in EDataCal.

Modernize EBookBackend's public API.

This leaves EBookBackend and EDataBook engaging in a somewhat ridiculous
kind of ping-pong game for the moment, but the desired public API (in as
much as a backend has a public API) is starting to take shape.

With the scaffolding now in place, we can begin converting backends to
use modern GIO conventions one method at a time, and eventually remove
all the "respond" functions in EDataBook.

Updated Polish translation

Assamese translation updated

Updated Czech translation