Start/stop book/cal views in a dedicated thread

Thus the backend, if it does any expensive operation during
the start/stop call, will not block factory's main thread,
thus the factory will not get unresponsive. This could happen
during autocompletion, for example.

Bug #680467 - Crash under camel_folder_search_search()

Updated Indonesian translation

Fix a compiler warning

Updated Portuguese translation

Post-release version bump.

NEWS update for 3.5.90 release.

Coding style and whitespace cleanup.

Bug 682103 - -fPIC build fix

Bug 682102 - Upstream Debian's 20_gettext_intltool.patch

Debian changelog:

    20_gettext_intltool.patch: don't confuse autoreconf by using both
    gettext and intltool.

Taking their word for it...

Bug 681679 - Typo in migration causes signatures to be dropped

Updated Telugu Translations

cache-reaper: Skip directories named "system".

Bug #680744 - IMAP: Does not update folder structure

Bug #678289 - Filters are not applied automatically to local maildir Inbox

Bug #682038 - "Show only this calendar" broken

Updated Serbian translation

ECalBackendWeather: Remove the remove_sync() method.

Cache reaper module takes care of the data and cache cleanup now.
When a data source is removed, the backend for it is simply finalized.

ECalBackendHttp: Remove the remove_sync() method.

Cache reaper module takes care of the data and cache cleanup now.
When a data source is removed, the backend for it is simply finalized.

ECalBackendFile: Remove the remove_sync() method.

Cache reaper module takes care of the data and cache cleanup now.
When a data source is removed, the backend for it is simply finalized.

ECalBackendContacts: Remove the remove_sync() method.

Cache reaper module takes care of the data and cache cleanup now.
When a data source is removed, the backend for it is simply finalized.

ECalBackendCaldav: Remove the remove_sync() method.

Cache reaper module takes care of the data and cache cleanup now.
When a data source is removed, the backend for it is simply finalized.

EBookBackendWebdav: Remove the remove() method.

Cache reaper module takes care of the data and cache cleanup now.
When a data source is removed, the backend for it is simply finalized.

EBookBackendLDAP: Remove the remove() method.

Cache reaper module takes care of the data and cache cleanup now.
When a data source is removed, the backend for it is simply finalized.

EBookBackendGoogle: Remove the remove() method.

Cache reaper module takes care of the data and cache cleanup now.
When a data source is removed, the backend for it is simply finalized.

EBookBackendFile: Remove the remove_sync() method.

Cache reaper module takes care of the data and cache cleanup now.
When a data source is removed, the backend for it is simply finalized.

Updated Galician translations

Deprecate e_client_remove_sync().

e_client_remove_sync() merely calls e_source_remove_sync() now.

Similarly for e_client_remove() / e_client_remove_finish().

cache-reaper: Clean up abandoned data directories.

Cache reaper now cleans up abandoned data directories for local address
books and calendars, but more conservatively than for cache directories.

Whereas a cache directory is marked as trash as soon as its data source
is removed and then allowed to linger for one week, a data directory is
only moved to trash when the registry service starts, and is allowed to
linger for four weeks.  This should give ample time to recover the data
if needed.

Updated Spanish translation

Revert "Export the EDBusAuthenticator interface from an isolated thread."

This reverts commit e13cb4e0ba820694f908fe39255ff8f7a6239038.

Apparently this does not solve the problem it was trying to solve, and
is less stable than what was there before.

Fix typo.

Export the EDBusAuthenticator interface from an isolated thread.

This is similar to the problem I had with GDBusObjectManagerClient.
When a GDBusInterfaceSkeleton is exported, it grabs the thread-default
main context and emits method invocation signals from only that context.

Problem is when e_authenticator_try_password_sync() is called on an
EAuthenticationMediator, something may have pushed a different main
context and so the method invocation signal emissions are inhibited
and we eventually time out thinking the client is unresponsive.

The workaround is to export the GDBusInterfaceSkeleton from an
isolated thread where its signals cannot be inhibited.  The thread
runs its own main loop until the EAuthenticationMediator object is
finalized, at which point the thread terminates and is joined with.

Add e_collection_backend_claim_all_resources().

Claims all previously used sources that have not yet been claimed by
e_collection_backend_new_child() and returns them in a GList.  Note
that previously used sources can only be claimed once, so subsequent
calls to this function for backend will return NULL.

The backend is then expected to compare the returned list with a
current list of resources from a remote server, create new ESource
instances as needed with e_collection_backend_new_child(), discard
unneeded ESource instances with e_source_remove(), and export the
remaining instances with e_source_registry_server_add_source().

Bug 681321 - Support both old and new-buf libxml2 APIs

libxml2 changed the API for xmlOutputBuffer incompatibly.
See https://mail.gnome.org/archives/desktop-devel-list/2012-August/msg00004.html

Bug 681640 - Check for Python interpreter in configure.ac

For /addressbook/libebook/gen-western-table.py.

Assamese translation updated

Bump libcamel soname.

For the previous series of API changes.

Rename camel_service_get_settings().

Applying lessons learned from ESourceRegistry.

Rename camel_service_get_settings() to camel_service_ref_settings()
and have it return a new reference.

When returning a pointer to a reference-counted object in a multi-
threaded environment, always increase the object's reference count
before returning so as to transfer a new reference to the caller.

Otherwise it introduces a potential race where the reference-counted
object may lose its last reference and be freed while the caller is
still using the object.  Even if the caller immediately increments
the object's reference count, it's still a potential race.

Transferring a new reference to the caller means the caller must
unreference the object when finished with it so the object will be
properly freed when it's no longer needed.

Rename camel_session_get_service().

Applying lessons learned from ESourceRegistry.

Rename camel_session_get_service() to camel_session_ref_service()
and have it return a new reference.

When returning a pointer to a reference-counted object in a multi-
threaded environment, always increase the object's reference count
before returning so as to transfer a new reference to the caller.

Otherwise it introduces a potential race where the reference-counted
object may lose its last reference and be freed while the caller is
still using the object.  Even if the caller immediately increments
the object's reference count, it's still a potential race.

Transferring a new reference to the caller means the caller must
unreference the object when finished with it so the object will be
properly freed when it's no longer needed.

Similarly for camel_session_get_service_by_url().

Have camel_session_list_services() return new references.

Applying lessons learned from ESourceRegistry.

When returning a pointer to a reference-counted object in a multi-
threaded environment, always increase the object's reference count
before returning so as to transfer a new reference to the caller.

Otherwise it introduces a potential race where the reference-counted
object may lose its last reference and be freed while the caller is
still using the object.  Even if the caller immediately increments
the object's reference count, it's still a potential race.

Transferring a new reference to the caller means the caller must
unreference the object when finished with it so the object will be
properly freed when it's no longer needed.

Making subtle behavioral changes like this without renaming the API
is usually considered bad, but since Evolution is the only consumer
we can easily keep the side-effects under control.

CamelSession: Have add_service() return a new reference.

Applying lessons learned from ESourceRegistry.

When returning a pointer to a reference-counted object in a multi-
threaded environment, always increase the object's reference count
before returning so as to transfer a new reference to the caller.

Otherwise it introduces a potential race where the reference-counted
object may lose its last reference and be freed while the caller is
still using the object.  Even if the caller immediately increments
the object's reference count, it's still a potential race.

Transferring a new reference to the caller means the caller must
unreference the object when finished with it so the object will be
properly freed when it's no longer needed.

Making subtle behavioral changes like this without renaming the API
is usually considered bad, but since Evolution is the only consumer
we can easily keep the side-effects under control.

CamelSession: Remove camel_session_lock/unlock().

Because exposing mutexes in a public API is horrible.

CamelSession: Avoid camel_session_lock/unlock().

Use an internal mutex to guard the services hash table.

CamelSession: Make forward_to() method asynchronous.

Missed this while converting the rest of Camel to GIO's async pattern,
but came to realize Evolution's forward_to() implementation starts an
asynchronous CamelFolder.append_to() operation and returns TRUE.

So the return value from camel_session_forward_to() does not actually
indicate whether the message was successfully forwarded.  In fact the
caller has no way of knowing!

This calls for an API break.

Split the forward_to() method into synchronous and asynchronous
variations:

  gboolean  (*forward_to_sync)    (CamelSession *session,
                                   CamelFolder *folder,
                                   CamelMimeMessage *message,
                                   const gchar *address,
                                   GCancellable *cancellable,
                                   GError **error);

  void      (*forward_to)         (CamelSession *session,
                                   CamelFolder *folder,
                                   CamelMimeMessage *message,
                                   const gchar *address,
                                   gint io_priority,
                                   GCancellable *cancellable,
                                   GAsyncReadyCallback callback,
                                   gpointer user_data);

  gboolean  (*forward_to_finish)  (CamelSession *session,
                                   GAsyncResult *result,
                                   GError **error);

Update API documentation.

Update git.mk.

Source: https://github.com/behdad/git.mk

e-book-backend-db-cache.h: Cleanups.

Clean up the header file while we're at it.

e-book-backend-db-cache.h: Avoid DB typedef.

Some distros use compiler settings that choke on duplicate typedefs.
Gentoo seems to be one of them.

Replace the DB typedef with a forward declaration of "struct __db"
and fix up all the function arguments that take a DB pointer.

mbox_store_get_full_path(): Improve path building.

Deal with the root path not ending in a directory separator, and use a
GString for heaven sake!

source_registry_server_create_source(): Create parent directories.

Same deal as the previous commit.

server_side_source_write(): Create parent directories.

Had a situation while restoring from a backup tarball where the
"sources" directory was getting deleted and consequently causing
g_file_replace_contents() to fail.  So always make the directory
immediately before writing the key file to disk.

Fix typo in camel_debug() call, should be "pop3", not "pop", for POP3 provider

Do not update unread count in source folder for junk/deleted messages

Bug #681466 - Virtual Trash/Junk not updated on change

Updated Marathi Translations

Bug #679488 - Unable to send mail using SMTP

Updated Lithuanian translation

Bug #681046 - Occasional crash of evolution-source-registry

Updated Spanish translation

vala: Fix srcdir != builddir

Fix regression after move to GSimpleAsyncResult in camel_service_disconnect

IMAP provider called disconnect in its finalize method, it was all fine
until move to GSimpleAsyncResult, which requires "live" GObject, which
is not true when called from finalize, thus everything breaks.
Moving the disconnect call to dispose function fixes this.
This was recognized when creating a new IMAP account and querying
for authentication methods.

Bug #681318 - Filter by Source account doesn't match in local folders

Updated Traditional Chinese translation(Hong Kong and Taiwan)

Updated gujarati file

Post-release version bump.

NEWS update for 3.5.5 release.

Assamese translation updated

Silly typo.

Fix mistake in the previous commit.

Got the signal emissions mixed up.

Bug 679808 - e_source_selector_set_primary_selection() has side-effects

In ESourceSelector parlance, the "primary selection" is the highlighted
tree view item.  A tree view item is "selected" if the check box cell is
ticked.  The two selection states are supposed to be orthogonal.

e_source_selector_set_primary_selection() has apparently for some time
been highlighting the requested item and also activating its check box.
That last part is an unwanted and undocumented side-effect.

This commit removes the side-effect and explicitly states in the API
documentation that the function does NOT alter the item's check box.

Updated Serbian translation

evolution-source-viewer: Add icons for new interfaces.

e_source_registry_commit_source_sync(): Handle collection members.

If a scratch source (with no GDBusObject) references a collection source
as an ancestor call e_source_remote_create_sync() on the collection source
instead of e_source_registry_create_sources_sync().

EServerSideSource: Support creating/deleting remote resources.

Add "remote-creatable" and "remote-deletable" properties which work the
same as the "removable" and "writable" properties in terms of exporting
and unexporting D-Bus interfaces.

Add handlers for Create() and Delete() D-Bus method invocations, which
call e_source_remote_create() and e_source_remote_delete() respectively.

Override the remote_create_sync() and remote_delete_sync() methods to
ferry the request to an associated ECollectionBackend (or set an error
if there is no associated ECollectionBackend).

ESource: Add functions to create/delete remote resources.

ECollectionBackend: Add virual methods to create/delete resources.

Add D-Bus interfaces for remote resource management.

Defines two new D-Bus interfaces for data source object paths:

  org.gnome.evolution.dataserver.Source.RemoteCreatable

    Create() - creates a remote resource

  org.gnome.evolution.dataserver.Source.RemoteDeletable

    Delete() - deletes a remote resource

These methods will be invoked through ESource objects on the client
side.  On the server side, the corresponding EServerSideSource will
receive the method invocation, but forward to an ECollectionBackend
to perform the actual operation.

Add e_source_registry_server_ref_backend().

Returns the ECollectionBackend for a given ESource, if one exists.

This works for any collection member: the "collection" ESource itself
as well as any of its hierarchical descendants.

Remove collection_backend_children_contains().

Decided I don't need this for the moment, so removing it to silence an
"unused-function" compiler warning.

Assamese translation updated

Updated gujarati file

Remove docs/reference/.gitignore from source control.

configure.ac: Remove redundant check for gio-unix-2.0.

libecal: fix memory leak after failed object creation

Apparently e_gdbus_cal_call_create_objects_sync() returns a muids
array even in case of failures. Free it when returning the error.
Found in SyncEvolution unit tests with valgrind.

Conditionally enable bug-buddy/gtk+ also for evolution-source-registry

Debug-print also error message on ERROR authentication

Fix a memory leak in e-cal-system-timezone.c

Updated POTFILES.in

Use e_backend_authenticate_sync() in backends.

ECollectionBackend: Implement authenticate_sync() method.

ECalBackend: Implement authenticate_sync() method.

EBookBackend: Implement authenticate_sync() method.

EBackend: Add authenticate() sync+async methods.

These are convenience functions providing a consistent interface for
backends running in either the registry service itself or a client
process communicating with the registry service over D-Bus.

Dynamically loaded backend classes need not implement these methods.
They will inherit a suitable method implementation from ECalBackend,
EBookBackend or ECollectionBackend.

CamelIMAPXServer: untagged handler descriptor needs untagged response code

* CamelIMAPXUntaggedRespHandlerDesc needs a member
  for the IMAP untagged response code
* this does also simplify much the code which generates
  the initial lookup tables for the untagged response
  handlers

CamelIMAPXServer: fixed API function for running custom commands

* imapx_command_run() deadlocks every now and then
  when running custom CamelIMAPXCommands
* imapx_command_run_sync() does not, so we're using
  this one although it is definitely not the best
  possible solution
* imapx_command_run_sync() expects a CamelIMAPXJob
  to be set on the CamelIMAPXCommand, so we're
  setting a local one if the command does not have
  a job set

Updated Galician translations

Remove flex/bison dependencies.

configure says "You need bison to build evolution-data-server"

No, actually, you don't.  In fact I have no idea why these were ever
needed.  "git blame" shows the requirements were present even in the
initial evolution-data-server commit in 2003, and yet even back then
I can't find evidence that flex or bison were actually used.

Replace e_source_registry_server_queue_auth_session().

I realized ECollectionBackends are going to need a proper asynchronous
authentication function so they know when authentication is achieved.

e_source_registry_server_queue_auth_session() is a "fire-and-forget"
function with no way for the caller to know if and when authentication
is achieved.

The replacement function, e_source_registry_server_authenticate_sync(),
is a variation of e_source_registry_authenticate_sync() for use in the
registry service itself.  It takes an EAuthenticationSession instead of
an ESource.

Since this API has not yet seen a stable release, I'm going to forego
deprecating the old function and just remove it.  This is an API break,
and the libebackend soname has been bumped accordingly.

Bug 678893 - Allow concurrent authentication sessions

I wrote the original queuing algorithm for authentication requests to
serialize requests to ensure password prompts never pile up.  But that
means requests for which a cached password exists may wait longer than
necessary, especially if an authentication session already in progress
is taking a long time to complete.

This was before I started using GcrSystemPrompt, which also serializes
requests to ensure password prompts never pile up.  That frees us up to
process authentication requests for different data sources concurrently.

This commit simplifies the queuing algorithm significantly and also
makes it thread-safe so authentication requests can be submitted from
any thread.  This is going to be important for ECollectionBackends.