Bug #676541 - Unreliable IMAP provider

Bug 677664 - Camel: Remove "SMTP Authentication" status message

Too jargony.  Just leave it on "Sending message (xx%)".

Omit warning flags for generated GDBus code.

Silence warnings when building with GOA < 3.5.

Bug #677610 - Avoid modification of CPPFLAGS in Makefile.am

Bug #677183 - maildir account doesn't open

Remove unused code

configure.ac: Remove test for "jw".

I don't know what "jw" is (or was), but HAVE_JW is not used anywhere.
It's a useless definition, as far as I can tell.  Remove the test.

Fix source_registry_prune_nodes() algorithm.

Exclude any disabled ESource and its descendants even if some of its
descendants are enabled.  We were also leaking ESource references in
this scenario; use e_source_registry_free_display_tree() to free the
whole subtree rooted at the disabled ESource.

ESourceRefresh: Handle "notify" signals from an idle callback.

ESourceRefresh runs e_source_refresh_force_timeout() in response to its
ESource becoming enabled.  The problem is "GObject::notify" signals can
be emitted from any thread.

Schedule an idle callback on the ESource's GMainContext to call
e_source_refresh_force_timeout(), rather than calling it directly from
the "GObject::notify" signal handler.

EAuthenticationMediator: Resolve some thread-safety issues.

The D-Bus method handlers in EAuthenticationMediator run in separate
threads because we set possibly the longest enum value ever:

G_DBUS_INTERFACE_SKELETON_FLAGS_HANDLE_METHOD_INVOCATIONS_IN_THREAD

Connect to the signals with g_signal_connect_object() so the handlers
will not be dispatched after the mediator is finalized (hopefully...
still not sure if it's 100% thread-safe).

Avoid possible GError memory leak

Avoid loading Camel modules during introspection.

libedataserver: Add libcamel to introspection flags.

Updated Galician translations

Fix some distcheck errors.

Some, not all.  E-D-S tests are still borked.

Add --with-private-docs to DISTCHECK_CONFIGURE_FLAGS.

Remove --without-weather from DISTCHECK_CONFIGURE_FLAGS.

Bug #673817 - Crash in e_book_backend_file_dispose()

Updated Spanish translation

Updated Russian translation

Updated Spanish translation

Fix build (typo in e-source-weather.c)

Camel: Avoid C++ keywords in header files.

Bump Camel's soname for the function removals.

Remove camel_settings_save_to_url().

Camel settings are no longer stored as a URL string in Evolution.
This function was always meant to be temporary.  Its time has come.

Remove camel_settings_load_from_url().

Camel settings are no longer stored as a URL string in Evolution.
This function was always meant to be temporary.  Its time has come.

Fix a build break when introspecting libecal.

Implement single-file includes for all E-D-S libraries.

The top-level headers are named after the library:

    #include <libebackend/libebackend.h>
    #include <libebook/libebook.h>
    #include <libecal/libecal.h>
    #include <libedata-book/libedata-book.h>
    #include <libedata-cal/libedata-cal.h>
    #include <libedataserver/libedataserver.h>
    #include <libedataserverui/libedataserverui.h>

Including individual headers will cause a compilation error.

Update po/POTFILES.in.

Bump all shared library and D-Bus API versions.

Everything but Camel broke API.

Add 'cache-reaper' module.

Add 'yahoo-backend' module.

Add 'google-backend' module.

Add 'online-accounts' module.

This replaces the 'online-accounts' module in Evolution.

Adapt libedataserverui tests to the new ESource API.

Adapt e-client-utils.c to the new ESource API.

Add unit tests for ESource.

Adapt ESourceSelectorDialog to the new ESource API.

Adapt ESourceSelector to the new ESource API.

Adapt ESourceComboBox to the new ESource API.

Adapt ENameSelectorList to the new ESource API.

Adapt ENameSelectorEntry to the new ESource API.

Adapt ENameSelector to the new ESource API.

Adapt e-book-auth-util.c to the new ESource API.

Deprecate e-passwords.[ch].

Add an ESource extension for the weather backend.

Add an ESource extension for the contacts backend.

Add an ESource extension for the file (cal) backend.

Add an ESource extension for the ldap backend.

Add an ESource extension for the vcf backend.

Adapt calendar backends to the new ESource API.

Adapt libedata-cal to the new ESource API.

Adapt libecal to the new ESource API.

Adapt calendar/libegdbus to the new ESource API.

Adapt address book backends to the new ESource API.

Adapt libedata-book to the new ESource API.

Adapt libebook to the new ESource API.

Adapt addressbook/libegdbus to the new ESource API.

Adapt libebackend to the new ESource API.

Adapt EClient to the new ESource API.

Add built-in ESource key files.

Add GSettings schemas for default sources.

Schema: org.gnome.Evolution.DefaultSources

Add new ESource classes.

ESource
ESourceExtension
ESourceRegistry

Add a new "evolution-source-registry" D-Bus service.

This new service manages data source key files and serves them to
clients, through the ESource and ESourceRegistry client-side APIs.

Generate GDBus classes for a new D-Bus service.

Require gcr-base-3 for secure password exchange over D-Bus.

Deprecate EList, EIterator, EListIterator.

Remove EAccount and EAccountList.  Breaks build.

Remove ESourceGroup.  Breaks build.

Remove ESourceList.  Breaks build.

Updated French translation

Post-release version bump.

NEWS update for 3.5.2 release.

Updated Galician translations

Update API documentation.

Whitespace cleanups.

Bug #666327 - IMAP deadlock when moving message and checking new mail

with service lock removal the fix for bug #666327
went out (commit 74fcab535c0f50a27742c05e94036b8370ea9173).
Was a good thing as this version is less a hack.
To summarize:
imap folder deadlock:
1. do_copy :
.imap command on source lock the imap store
2. camel_imap_folder_fetch_data on destination:
. lock the destination folder
3. do_copy :
. call the hande user tag : wait for lock on the destination
folder cache .
4. camel_imap_folder_fetch_data on destination:
. wait for lock on the imap store to be freed.

Fix: get the lock on the destination folder cache before locking the
imap store (ie before calling the imap command).

Updated Spanish translation

Updated Spanish translation

Bug #673946 - Cannot delete search folders

Simplify vFolder's code and other bunch of related time optimizations

Bump goa_minimum_version to 3.2.

Just to make it a stable release number.

EFileCache cleanups.

Updated Slovenian translation

libebackend documentation touch ups.

Camel documentation touch ups.

Add e_util_strdup_strip().

Duplicates an input string and strips off any leading or trailing
whitespace.  The resulting string is returned unless it is empty or
NULL, in which case the function returns NULL.

Useful in "set" functions for string properties.
Helps avoid expressions like (str != NULL && *str != '\0').

camel_service_disconnect(): Missing break in switch statement.

Bug 667725 - imapx_untagged: code should not be reached

This code is evil.

When we scan a folder for new messages, we issue a 'FETCH 1:* (UID FLAGS)'
or similar command.

When we receive an untagged FETCH from the server telling us flags for a
message, we make a decision about whether that information was solicited
by such a command, or whether it was unsolicited.

If it was unsolicited, we process it normally as an asynchronous flags
update and all is well.

If it was solicited, we add the UID to a list. When the FETCH (UID FLAGS)
command *completes*, we'll sort that list and then fetch the full headers
for each message.

However, we weren't very good at telling when an update was solicited.
Assuming that only solicited messages will have a UID is bogus.

This was failing if an unsolicited update came in when the (UID FLAGS)
fetch had completed, and we were already fetching the message headers.
The "new" UID would be added to the end of the list, even if we were
already fetching that message or if we already had it in cache. We'd
issue a FETCH command for it, and the barf when the server complied,
because when the UID list wasn't sorted we wouldn't find the offending
uid when we looked for it.

The simple "fix" for this is to keep a boolean flag 'scan_changes' which
is TRUE only when that FETCH (UID FLAGS) command is running. If a flags
change comes in at any other time, it is definitely unsolicited and
should *not* be added to the uidset. This at least protects us from
having UIDs added after we've sorted the list and started to do other
things with it, which was causing the crash.

In fact, this whole 'solicited' vs. 'unsolicited' thing is a design
mistake. In imapx_untagged() we should never care about what we asked
for and what we didn't. That's why the responses are *untagged*. The
server tells us things about the state of the mailboxes, and we should
process that information into our own local cache — it shouldn't
*matter* what we asked for. But that's a more intrusive fix for another day.

In addition, we were reliably *triggering* this behaviour in some cases
because we had to issue a SELECT for the folder in question before
issuing the FETCH (UID FLAGS) command. And on completion of the SELECT,
if UIDNEXT had increased, we were automatically issuing a *new* FETCH
(UID FLAGS) command starting from the last-known-uid in our cache. This
was entirely gratiutous, so use the same scan_changes boolean flag to
avoid it in that situation.

Fix off-by-one in uidset handling for fetches

When we get to the end of the for() loop processing the uidset, the index
variable is one *more* than the last item. That's how for() loops in C
work.

But when we break out of the middle because we are limiting the batch
size, we were forgetting to do the same and we were storing the index
of the last item that we'd already fetched.

This was causing us to re-fetch the last message of one batch, as the
first message of the next batch. Mostly harmless, but spotted and fixed
as part of bug 667725.

Bug 676506 - imapx: After QRESYNC leaves us out of sync, a "full resync" doesn't work either.

Always fetch from the beginning of the folder.

Bug 676403 - imapx: Syncing folder after server-side deletion of many messages is *very* slow

Fix this by introducing a new camel_folder_change_info_remove_uids()
function, and using it for the 'vanished' code. This reduces the time taken
from about 40ms per message, to about 50µs.

Bug 676108 - OpenBSD needs _XOPEN_SOURCE_EXTENDED for strptime(3)

strptime(3) was introduced in XPG4. On OpenBSD, just defining
_XOPEN_SOURCE gets you the older XPG3 which doesn't include strptime(3),
so we need to define _XOPEN_SOURCE_EXTENDED to get XPG4v2.

Updated Galician translations

google: Fix a gcc warning in the Google backend

Signed/Unsigned comparison.

google: Fix some gcc warnings in the GOA authoriser

Variable shadowing, missing default cases and signed/unsigned comparisons.

Bump Camel's shared object name.

For CamelService connect/disconnect API changes.

Remove camel_service_lock/unlock().

This has been the cause of many deadlocks, and is no longer needed with
CamelService's new asynchronous connect/disconnect API.

CamelService: Rewrite connect/disconnect API.

CamelService now has proper cancellable asynchronous functions for
connect and disconnect operations.  The way this works is as follows:

The first service connect request runs asynchronously internally, even
if camel_service_connect_sync() is called.  Subsequent connect requests
are queued until the first connect request finishes, then all requests
finish simultaneously with the same result.  There are no retries; if
the first request fails, they all fail with the same error.

If a service disconnect is requested with one or more connect requests
outstanding, the connect requests are all cancelled and the disconnect
request runs asynchronously and plays by the same queueing rules.

CamelService's "connection-status" property will immediately reflect any
connect or disconnect requests in progress.  Change notification signals
for this property, however, are emitted from CamelSession's main context.