Bug #680497 - POP3 re-adds messages to local Inbox

IMAPX: Explicitly pass GCancellable and GError to callbacks.

Further error handling improvements.  This involves some public API
changes, but the impact is limited to evolution-kolab.

 * CamelIMAPXCommandFunc now takes a GCancellable argument in addition
   to its GError argument.

 * The public GCancellable member of CamelIMAPXJob is now private, and
   the public GError member is removed.

 * The start() method of CamelIMAPXJob now takes a GCancellable and
   GError argument.  The GCancellable pass through this method call
   is always the CamelIMAPXJob's own private GCancellable.

 * A new camel_imapx_job_cancel() function which simply cancels the
   CamelIMAPXJob's internal GCancellable.

Additional changes internal to CamelIMAPXServer are:

 * Adapt operations to the new callback function signatures.

 * imapx_command_queue() now takes a GCancellable and GError argument
   and returns a 'success' boolean.  Other core job and command routing
   functions also now have similar function signatures.

 * Add error handling where needed, so that no failable function call
   goes unchecked.

Bug #686727 - POP3 provider deadlocks on finalize

Updated German translation

CamelIMAPXServer: Handle errors in imapx_continuation().

There was no error handling at all on the write-to-stream operations.

CamelIMAPXServer: Return a boolean from imapx_command_start_next().

The function is failable and takes a GError, yet returns void.  This
is how GErrors pile up.  Return a boolean and add the necessary error
checks where the function is called.

Revert "Bug #678400 - ECollectionBackend populates disabled accounts"

This reverts commit 9a7948409852e57721dcb2e9e63ae69d8ac78d40.

Need to find a better solution than corrupting what are supposed to be
separate state flags.

Bug #678400 - ECollectionBackend populates disabled accounts

CamelIMAPXServer: Improve error handling around stopping IDLE.

imapx_stop_idle() returned a boolean but TRUE had an ambiguous
interpretation.  It either meant the DONE command was sent or an I/O
error occurred.  Define a new enum type (CamelIMAPXIdleStopResult) to
resolve the ambiguity.

Also imapx_command_idle_stop() returned a boolean but FALSE had an
ambiguous interpretation.  It either meant an I/O error occurred, or
there was no available CamelIMAPXStream from which to issue the DONE
command.  Resolve the ambiguity by forcing callers to explicitly pass
in a CamelIMAPXStream (and a GCancellable, while we're at it).

CamelIMAPXServer: Declare imapx_uidset_* functions as static.

These functions were never meant to be exported.  The "static" keyword
seems to have been omitted by accident.  Can't find any evidence of them
being called from outside camel-imapx-server.c, so just make them static.

CamelIMAPXServer: Clean up static function declarations.

One argument per line.

CamelIMAPXServer: Make sure all disabled bits have a comment.

Normally I throw away all disabled-with-no-explanation-why code chunks,
but in this case they all seem related, so clarify that in the code.

Updated Hungarian translation

EAuthenticationSession: Close prompt before storing password.

Close our prompt before storing the password in the keyring.
If the keyring is locked, it will need to prompt the user for
a keyring password, but it can't do that if our password prompt
is still open since both prompts are system-modal.  Not sure what
would happen next; probably the store operation would either fail
or deadlock.

Bug #640594 - Save also UID/REV in WebDAV backend (for CardDAV)

Post-release version bump.

NEWS update for 3.7.1 release.

Coding style and whitespace cleanup.

evolution-source-viewer: Select next source after deleting.

Deleting a list of sources is less painful when the next source is
automatically selected.

Further test code cleanups.

Bug 686421 - Restore libebook tests to minimum working condition

Merge cleanups from 'gdbus-codegen' branch.

Merge cosmetic cleanups from my 'gdbus-codegen' branch so I can more
easily review the important changes.

Updated Spanish translation

Updated Slovenian translation

configure.ac: Revert unnecessary soname bumps.

Adding new functions does not break backward-compatibility of the
programming or binary interfaces for these libraries.

libedataserver soname bumps are particularly disruptive to other GNOME
modules, and should be avoided if at all possible.

Bug #268618 - Add: Save replies in the folder of the message being replied to

Bug #310978 - USENET always requires authentication

Updated Slovenian translation

Bug #201807 - Cannot specify sendmail parameters

camel_junk_filter_classify(): Fix return values.

Bug #641825 - Crash in poll_ldap() when finalizing the backend

Updated Greek translation

Updated Czech translation

Change camel_junk_filter_classify() signature.

Add CAMEL_JUNK_STATUS_ERROR to the CamelJunkStatus enum and have
camel_junk_filter_classify() return the status directly, rather than
forcing callers to check both a status value and a 'success' boolean.

Minor API break, but should only affect Evolution.

Updated Galician translations

Updated Slovenian translation

Updated Slovenian translation

Updated Indonesian translation

Updated Indonesian translation

Updated Polish translation

Bug 684655 - Files in data/sources are not getting translated

Bug #681314 - Add address dialog should follow composer's view settings

Fix code examples in ESourceExtension API docs.

The examples did not follow our single-include policy.

Updated POTFILES.in

Rename 'online-accounts' module to 'gnome-online-accounts'.

For better or worse, GNOME Online Accounts isn't the only accounts
service in town anymore.  Use a less generic module name.

ESourceRegistryServer: File monitor should set flags on new sources.

ESourceRegistryServer forgot to set WRITABLE and REMOVABLE flags on
new ESources created when a GFileMonitor detects a new .source file.
Consequently the new ESource could neither be modified nor removed,
which causes problems for unit tests that use predefined key files.

Silence libsecret unstable API warnings.

Deprecate e_source_extension_get_source().

Replace it with a thread-safe version: e_source_extension_ref_source()

Also use GWeakRef within ESourceExtension, which itself is thread-safe.

Remove e_data_cal_view_register_gdbus_object().

Similar to EDataCal, the first thing we do after creating a new
EDataCalView is export its D-Bus interface.  This was a separate and
failable operation.  If the export fails, the EDataCalView is useless.
Perfect use case for GInitable.

Now we pass the GDBusConnection and object path directly to
e_data_cal_view_new(), and if exporting fails the function sets a
GError and returns NULL.

This also introduces a couple accessor functions:

    e_data_cal_view_get_connection()
    e_data_cal_view_get_object_path()

Remove e_data_book_view_register_gdbus_object().

Similar to EDataBook, the first thing we do after creating a new
EDataBookView is export its D-Bus interface.  This was a separate and
failable operation.  If the export fails, the EDataBookView is useless.
Perfect use case for GInitable.

Now we pass the GDBusConnection and object path directly to
e_data_book_view_new(), and if exporting fails the function sets a
GError and returns NULL.

This also introduces a couple accessor functions:

    e_data_book_view_get_connection()
    e_data_book_view_get_object_path()

EDataCalView cleanups.

Some minor API changes with this:

    e_data_cal_view_get_text() is dropped.  Instead use:

        sexp = e_data_cal_view_get_sexp (view);
        query = e_cal_backend_sexp_text (sexp);

EDataBookView cleanups.

Some minor API changes with this:

   e_data_book_view_get_card_sexp() is now e_data_book_view_get_sexp().

   e_data_book_view_get_card_query() is dropped.  Instead use:

       sexp = e_data_book_view_get_sexp (view);
       query = e_book_backend_sexp_text (sexp);

Add e_book_backend_sexp_text().

Similar to e_cal_backend_sexp_text(), returns the original textual
expression used to build the EBookBackendSExp object.

e-cal.c cleanups.

e-book.c cleanups.

EGDataGoaAuthorizer: OAuth 2.0 fixes.

Changes based on Google's OAuth 2.0 Playground.

Authorization type is "OAuth" instead of "Bearer", and no Base64
encoding of the access token is required.

Still no successful test with GOA 3.7, so somewhat flying blind.

online-accounts: Refresh mail account settings on startup.

When matching GOA accounts to existing EDS accounts on startup, refresh
both the collection and mail account settings.  This is primarily to aid
the transition from XOAUTH to XOAUTH2, but can serve as a means of other
types of migration in the future.

EGDataGoaAuthorizer: Prototype OAuth 2.0 support.

Untested, but conforms to the specification as well as I understand it.

vala: Add libedataserver-1.2 to libebook-1.2.deps file.
Fixes part of folks bug https://bugzilla.gnome.org/show_bug.cgi?id=685084
so folks ofono backend will only need --pkg libebook-1.2
not libedataserver-1.2 explicitly.

Drop oauth package requirement for GOA support.

On second thought, Camel's XOAUTH SASL module is just using
g_random_int() to generate the "oauth_nonce" parameter.

Good enough, I suppose.  Do the same for EGDataGoaAuthorizer.

EGDataGoaAuthorizer: Use GHmac to sign the OAuth parameters.

Use GLib's GHmac API in place of oauth_sign_hmac_sha1().

Unfortunately we still have to keep the liboauth requirement for its
oauth_gen_nonce() function.  Considered just copying it out of oauth.c
but it looks rather messy with OpenSSL and NSS dependencies.

configure.ac: Tweak GOA-related error messages.

Drop the check for libgdata >= 0.9.1 for GOA support.

We already require libgdata >= 0.10, so the check is redundant.

Eliminate the assertion in camel_provider_register().

camel_provider_register() should automatically set up the module and
provider hash tables if needed instead of aborting, but the problem
is the GOnce function that sets up these hash tables also calls
camel_provider_register().

Break this cycle by moving the bulk of camel_provider_register() to a
new internal function (provider_register_internal) and calling it from
the GOnce function (provider_setup).

The public camel_provider_register() function is then simply:

    g_once (&setup_once, provider_setup, NULL);

    provider_register_internal (provider);

Bug 685713 - ESourceRegistry's manager thread aborts on error

If the ESourceRegistry's manager thread fails to create a new
GDBusObjectManagerClient during initialization, propagate the GError
back to the caller of e_source_registry_new() instead of aborting the
whole process with a g_error() call.

Bug 685725 - Handle "muser" attributes in %gconf.xml files

The migration of account data from GConf to key files parses raw
%gconf.xml files.  The parsing code neglected to handle "muser"
attributes in <entry> tags, so that when an "muser" attribute was
encountered, g_markup_collect_attributes() would throw an error an
cause the whole GConf migration phase to be aborted.

Don't migrate sources with empty attributes

vala: Don't ignore CalClientSourceType

It's needed by the API after the calendar refactoring.

Signed-off-by: Colin Walters <walters@verbum.org>

Updated Slovak translation

Remove e_data_cal_register_gdbus_object().

The first thing we do after creating a new EDataCal is export its D-Bus
interface.  This was a separate and failable operation.  If the exporting
fails, the EDataCal is useless.  Perfect use case for GInitable.

Now we pass the GDBusConnection and object path directly to
e_data_cal_new(), and if the exporting fails the function sets a GError
and returns NULL.

This also introduces a couple accessor functions:

    e_data_cal_get_connection()
    e_data_cal_get_object_path()

Deprecate e_cal_backend_foreach_view().

Use e_cal_backend_list_views() instead.

ECalBackend: Prefer e_cal_backend_list_views().

Less klunky than e_cal_backend_foreach_view().

e_cal_backend_foreach_view: Use e_cal_backend_list_views().

Holding a locked mutex while invoking callback functions we don't
control raises the potential for deadlock.

Add e_cal_backend_list_views().

Alternative to e_cal_backend_foreach_view().

ECalBackend: Prefer GList over GSList.

EGdbusCal: Remove the remove() method.

Not used anymore.  Remove the ESource instead.

This also takes out the following backend functions:

    e_cal_backend_remove()
    e_cal_backend_sync_remove()
    e_data_cal_respond_remove()

ECal: Reimplement "remove" functions.

Have them just call e_source_remove() or e_source_remove_sync().

libedata-cal: Purge deprecated APIs.

Since we're already bumping libedata-cal's shared object name...

s/E_DATA_CAL_VIEW_TYPE/E_TYPE_DATA_CAL_VIEW/

Bump libedata-cal soname for API changes in the following commits.

Remove e_data_book_register_gdbus_object().

The first thing we do after creating a new EDataBook is export its D-Bus
interface.  This was a separate and failable operation.  If the exporting
fails, the EDataBook is useless.  Perfect use case for GInitable.

Now we pass the GDBusConnection and object path directly to
e_data_book_new(), and if the exporting fails the function sets a GError
and returns NULL.

This also introduces a couple accessor functions:

    e_data_book_get_connection()
    e_data_book_get_object_path()

Deprecate e_book_backend_foreach_view().

Use e_book_backend_list_views() instead.

backends/ldap: Prefer e_book_backend_list_views().

Less klunky than e_book_backend_foreach_view().

backends/ldap: Remove disabled code.

Bits of code disabled with no comment as to why => junk it.

backends/file: Prefer e_book_backend_list_views().

Less klunky than e_book_backend_foreach_view().

EBookBackend: Prefer e_book_backend_list_views().

Less klunky than e_book_backend_foreach_view().

e_book_backend_foreach_view: Use e_book_backend_list_views().

Holding a locked mutex while invoking callback functions we don't
control raises the potential for deadlock.

Add e_book_backend_list_views().

Alternative to e_book_backend_foreach_view().

Rename e_book_backend_remove_book_view().

Rename to e_book_backend_remove_view() [cf. e_cal_backend_remove_view()].

Rename e_book_backend_add_book_view().

Rename to e_book_backend_add_view() [cf. e_cal_backend_add_view()].

Rename e_book_backend_stop_book_view().

Rename to e_book_backend_stop_view() [cf. e_cal_backend_stop_view()].

Also rename the EBookBackend.stop_book_view() method to stop_view()
and adapt the built-in address book backends.

Rename e_book_backend_start_book_view().

Rename to e_book_backend_start_view() [cf. e_cal_backend_start_view()].

Also rename the EBookBackend.start_book_view() method to start_view()
and adapt the built-in address book backends.

EBookBackend: Prefer GList over GSList.

EGdbusBook: Remove the remove() method.

Not used anymore.  Remove the ESource instead.

This also takes out the following backend functions:

    e_book_backend_remove()
    e_book_backend_sync_remove()
    e_data_book_respond_remove()

EBook: Reimplement "remove" functions.

Have them just call e_source_remove() or e_source_remove_sync().

Remove e_data_book_view_ref/unref().

These are just old wrappers for g_object_ref/unref().

Bump libedata-book soname for API changes in the following commits.

Move ECalClientSourceType enum to e-cal-types.h.

Let glib-mkenums generate the GEnumClass for it.

e-cal-client.c cleanups.

Mostly just code rearrangement.

e-book-client.c cleanups.

Mostly just code rearrangement.