Update API documentation.

Miscellaneous cleanups.

camel_store_get_folder_info_sync() cleanups.

Bug #704740 - WebDAV - Do not store ETag into REV attribute

Updated German translation

Bug #704618 - Missing detached instances in get_object() response

Bug #386113 - show custom alarm message in pop-up alerts

Enable this feature only for backends we are completely sure it works.
Only enabled for local calendars (file backend) for now.

Fix typo in e_cal_backend_file_get_backend_property()

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

Do not use generic error codes in camel_getaddrinfo()/camel_getnameinfo()

It's too hard to distinguish from other provider errors when using
general error codes here, thus rather use CAMEL_SERVICE_ERROR_URL_INVALID.

Updated Czech translation

Fix memory leak in camel_imapx_command_add_qresync_parameter().

Bug 704003 - IMAPX: Untagged BYE message while connecting is lost

Do not disconnect the service in imapx_untagged_bye() if we're still
connecting.  camel_service_disconnect_sync() will cancel the connect
operation and the server message will get replaced with a generic
"Operation was cancelled" message.

Also, check for and handle errors when calling imapx_untagged() from
imapx_connect_to_server().

CamelGpgContext: Enclose email addresses in brackets.

The recipient list for encrypting can be specified by either key ID or
email address.  Enclose email addresses in brackets to ensure an exact
match, as per the gpg man page:

HOW TO SPECIFY A USER ID

       ...

       By exact match on an email address.
              This is indicated by enclosing the email address in the
              usual way with left and right angles.

         <heinrichh@uni-duesseldorf.de>

Without the brackets gpg uses a substring match, which risks selecting
the wrong recipient.

Bug #692904 - Do not call camel_service_disconnect during connection phase

Bug #704181 - Froze while filtering messages

Bug #691526 - Memory corruption under e_book_backend_google_start_book_view()

Updated Norwegian bokmål translation

CamelGpgContext: Be aware of gpg2

There can be installed only gpg2, without old gpg executable, but
CamelGpgContext was not aware of that and simply failed to do any
gpg operation due to missing gpg executable.

This was reported as: https://bugzilla.redhat.com/show_bug.cgi?id=985599

Bump libecal / libedata-cal soname for API breaks.

Insufficient return values from e_cal_backend_get_object/_list()

These also returned ECalComponent, but it cannot hold a vCalendar,
which is returned when the component has detached instances.

Bug #704220 - Incorrect runtime check in e_data_cal_respond_send_objects()

Fix imapx_parse_flags() regression.

CamelIMAPXServer: Untagged handler cleanups.

CamelIMAPXStream: Return TRUE/FALSE instead of 0/-1.

Affected functions:

  camel_imapx_stream_atom()
  camel_imapx_stream_astring()
  camel_imapx_stream_nstring()
  camel_imapx_stream_nstring_stream()
  camel_imapx_stream_text()
  camel_imapx_stream_number()
  camel_imapx_stream_skip()

Simplify camel_imapx_stream_number().

Simplify camel_imapx_stream_nstring_stream().

Simplify camel_imapx_stream_nstring().

Simplify camel_imapx_stream_astring().

Simplify camel_imapx_stream_atom().

CamelIMAPXStore cleanups.

Remove IMAPX_TOK_PROTOCOL.

This just makes it awkward to check for errors.

An error is an error, the GError domain should indicate the type.

Remove unused CAMEL_STORE_INFO_FOLDER_CHECK_FOR_NEW flag.

Checked for but never set.

Also remove unused CAMEL_FOLDER_CHECK_FOR_NEW flag.

Remove unused camel_imapx_namespace_list_copy().

Remove unused camel_imapx_store_summary_set_namespaces().

Remove unused camel_imapx_store_summary_namespace_set().

imapx_reconnect() cleanups.

camel_folder_transfer_messages_to_sync: Connect the CamelServices first.

camel_folder_expunge_sync: Connect the CamelService first.

camel_folder_append_message_sync: Connect the CamelService first.

CamelTcpStreamRaw: Don't timeout on reads.

This is needed for IMAP IDLE to work, which sits in a read call for
long periods of time.  If this times out then we disconnect from the
server and miss out on mailbox updates.

The timeout was orignally added to work around suspend/resume issues,
but that should be handled by way of GCancellables, not by prematurely
terminating a read call.  Prior to reading we connect to GCancellable
and call PR_Interrupt() on cancellation to terminate the read call.
Need to make sure the GCancellable is actually firing when it should,
and that we're not racing it.

Leaving the timeout on write calls in place for now, since that seems
more reasonable.

camel_folder_get_message_sync: Recover from dropped connection.

If the requested message is not cached, and the service is online but
disconnected, reconnect before requesting the message.  This is meant
to help recover from dropped server connections.

CamelIMAPXServer: Remove fixed "use_idle" flag.

Always obtain a fresh value from CamelIMAPXSettings.

Rename imapx_idle_supported() to imapx_use_idle() for better readability
(even if IDLE is supported, the user may prefer not to *use* it).

Further, only call imapx_use_idle() when deciding whether to issue an
IDLE command.  Don't need to check imapx_use_idle() when handling state
changes -- if we're already *in* IDLE then we're obviously *using* IDLE.

imapx_connect_sync: Fix runtime warning.

Follows from the previous commit.

imapx_disconnect_sync: Fix runtime warning.

Bad assumption here.  The server could hang up while the
CamelService is still in a CAMEL_SERVICE_CONNECTING state.

CamelIMAPXStore: Further cleanups.

Updated Chinese simplified translation

Updated Chinese simplified translation

camel_folder_synchronize_sync: Connect the CamelService first.

camel_folder_refresh_info_sync: Connect the CamelService first.

CamelIMAPXSettings: Remove "concurrent-connections" property.

No longer used.

Remove CamelIMAPXConnManager.

It turns out CamelIMAPXConnManager's server selection logic has been
broken for some time and we've been operating on a single connection
regardless of the "concurrent-connections" setting.

And you know what?  A single connection works fine.  The only thing
CamelIMAPXConnManager actually does nowadays is occasionally deadlock
because it tries to connect to a server while holding a locked mutex,
which is a major no-no.

CamelIMAPXConnManager is not documented, it's not only NOT doing its
job but is actively causing problems, and I can't seem to wrap my head
around the implications of multiple connections in a CamelService.  So
I decided to just junk it.

New functions:

  camel_imapx_store_ref_server()

Removed functions:

  camel_imapx_store_get_server()
  camel_imapx_store_op_done()

CamelIMAPXServer: Stop joining with the parser thread.

Don't ever block in a dispose() method.  Pass the parser thread its
own CamelIMAPXServer reference and just let it terminate on its own.

CamelIMAPXStore cleanups.

CamelIMAPXFolder cleanups.

Updated Czech translation

google: Use CalDAV v2 if OAuth 2.0 support is available.

google: Minor module cleanups.

CalDAV: Add support for OAuth 2.0 authentication.

Add ESoupAuthBearer.

SoupAuth subclass for use with OAuth 2.0 HTTP authentication.

See http://tools.ietf.org/html/rfc6750

EBackends should use e_source_get_oauth2_access_token() to obtain
the access token and token expiry for an ESource, then pass them to
e_soup_auth_bearer_set_access_token().

source_webdav_user_to_method: Don't stomp on custom method names.

This binding might have just been a bad idea on my part, but this
should salvage it without breaking API behavior.

Updated Indonesian translation

Post-release version bump.

NEWS update for 3.9.4 release.

Add new symbols to API documentation.

e_book_backend_sqlitedb_has_contact() used nonexistent column

The function used partial_content column of a respective folder table,
but this is not available anymore, and references to this column are
also no-op these days, thus I made this use the 'uid' column instead.
The failure broke contact removal in evolution-ews.

[IMAPX] Server connection is not mandatory when searching folder content

In case the store is kept online, but the server cannot be reached,
like when it's hidden behind the VPN, the IMAPX folder should show
offline content, rather than nothing at all. After all, live server
connection is needed for body-contains only, and only that to not
download all messages in the folder.

Avoid a crash in cal_backend_add_cached_timezone().

Apparently icaltimezone_get_tzid() can sometimes return NULL.  Check for
NULL before we use it in a hash table lookup, which crashes on NULL keys.

Add a similar check to ECalClient for consistency.

Bug #703194 - Custom alarm message is REMINDER

Bug #700894 - LDAP 'Copy for offline' blocks online searches

GError instance use-after-free in collection_backend_load_resources()

The function uses a shared GError instance in a cycle, and on error
freed the instance with g_error_free() which left the pointer set,
only populated with freed memory, thus the next round, when it eventually
loaded correct source, the function thought it ended with an error
and a GSlice allocator aborted on the g_error_free() call. Resetting
the error variable to NULL after free (by using g_clear_error()) fixes
the crash.

Bug #703053 - Error on setting reminder 'after start of appointment'

As EWS doesn't support setting 'after start' reminders, let's create
a new capability to help us when we should avoid to show the option
to the user

Allow automatic NTLM authentication unconditionally

As part of bug 703181 we retested imapx+NTLM authentication, and it seems
that the NTLM SASL mechanism wasn't allowing the ntlm_auth helper to be used
unless its ->try_empty_password_sync() method was called.

That made sense when this support was first added, since the logic in the
back end would go something like:

  if (try_empty_password())
    { /* Try it and return OK if it worked */ }
  demand_password_from_user()
  ...

However, the setup of the authentication is now different, and the imapx
back end (for example) doesn't even *care* about the password. Adding an
artificial call to camel_sasl_try_empty_password_sync() into the imapx code
seemed like the wrong thing to do; it should be enabled *automatically*.

So do that: try to set up the ntlm_auth helper in sasl_ntlm_challenge_sync()
if we are invoked with no password.

Bug #697598 - [imapx] Freeze due to calling thread join from main thread

Bug #682322 - SIGSEGV in pop3_folder_refresh_info_sync

Updated Spanish translation

CamelFilterDriver: Remove ":filter" debug messages.

There's not enough debugging output here to be useful, the debug key
syntax is weird, and I'd like rewrite filter logging to be activated
by CAMEL_DEBUG=filter and give each output line a prefix that's easy
to isolate with grep.

EBookBackendLDAP cleanups.

More consistent error messages for missing opt dependencies.

UOA: Add service-specific descriptions for E-D-S.

This causes each service for a provider to be listed separately in the
Control Center UOA panel, with an independent switch for each service.

Add a desktop file for Ubuntu Online Accounts.

This is required for E-D-S to show up in the Ubuntu Online Accounts
configuration UI.  It's only installed when built with UOA support.

CamelIMAPXStore: Try harder to track settings.

Connect a "notify" signal handler to a newly-installed CamelSettings
instance, and emit a "folder-info-stale" signal for certain settings
changes that would ordinarily require an application restart.

Specifically, listen for changes to:

    "use-real-junk-path"
    "use-real-trash-path"
    "use-subscriptions"

CamelStore: Add a "folder-info-stale" signal.

This signal indicates significant changes have occurred to the folder
hierarchy of the CamelStore, and that previously fetched CamelFolderInfo
data should be considered stale.

Applications should handle this signal by replacing cached
CamelFolderInfo data for the CamelStore with fresh data by way of
camel_store_get_folder_info().

More often than not this signal will be emitted as a result of user
preference changes rather than actual server-side changes.  For example,
a user may change a preference that reveals a set of folders previously
hidden from view, or that alters whether to augment the CamelStore with
virtual Junk and Trash folders.

This increases sizeof(CamelStoreClass) and its descendants, therefore
the libcamel shared object name has been bumped.

Fix e_book_backend_summary_is_summary_query()

It didn't check on correct summary fields being used, but on a result
of the expression, which is not correct when there comes a 'not' in it.
This broke "Unmatched" filter in evolution's Contacts view, which is
testing nonexistence of a CATEGORIES attribute in a vCard.

Updated Norwegian bokmål translation

data_book_error_from_gdata_error: Make sure we actually set an error.

Had an instance where the error code didn't match one of the predefined
cases.  The GError never got set, and we crashed trying to deference it.

Updated Lithuanian translation

Post-release version bump.

NEWS update for 3.9.3 release.

Make the statements explicit when using ternary if

Based on:
https://git.gnome.org/browse/evolution-data-server/commit/?id=53a161d1eed173a

Bug 701797 - UOA: Remove ResponseType and AuthPath from service files

Remove ResponseType and AuthPath from the Google Contacts / GMail
service files, as it should be handled by the provider and it breaks
OAuth2 authentication with the latest changes to QUrl.

Bug 701838 - Work around a semantic API break in GObject

Recent versions of GObject fire critical messages if properties are
installed outside of class initializers (causing all of EDS tests to
fail).  This patch adds some class data and initializes the class in
the dynamic subclass initializer.

Note that the class data which should normally be freed in the class
finalizer is not freed, since it is a static type the finalizer will
never run (and GLib also takes the liberty of firing critical messages
for specifying a class finalizer).

Updated slovak translation

Bug #699448 - Crash under e_book_backend_ldap_stop_view()

Updated Slovenian translation

Updated Czech translation

Be able to debug SMTP provider separately

One can use CAMEL_DEBUG=smtp to debug only SMTP camel provider.

Updated Galician translations

Bug #672837 - Crash in camel_pop3_engine_command_free

Add camel_imapx_conn_manager_ref_store().

Replaces camel_imapx_conn_manager_get_store(), which is not thread-safe.

Also, track the CamelStore internally with a GWeakRef instead of a weak
pointer, which is also not thread-safe.

Add camel_imapx_command_add_qresync_parameter().

This splits a big wad of extra logic in imapx_select() out into a nice
and tidy utility function.

One important change, however, is the message sequence match data stops
at the lesser message count reported by the IMAP server vs. our summary.

This is to work around a frequent Zimbra IMAP failure where listing a
message sequence number in a QRESYNC parameter which exceeds the true
message count results in a so-called "parse error" from Zimbra:

   A00010 SELECT INBOX (QRESYNC (... (1,5,10 ...)))
   * 9 EXISTS
   ...
   A00010 BAD parse error: invalid message sequence number: 1,5,10

This scenario can happen if messages have been expunged from the
folder, but our summary has not yet been updated to reflect that.

Not sure if this is a Zimbra bug.  I certainly would not call it a
"parse error", but RFC 5162 is not clear on this detail.