Bug #687865 - Shows old reminders for GOA accounts

Revert "EBookBackendGoogle: Make room for other authorizers."

This reverts commit a5dae1a780f898ab453f574440ca02e108b4ebef.

Actually no, don't want to do that.  Misread the code.

EBookBackendGoogle: Make room for other authorizers.

Update German translation

EDataBookFactory: Remove the GOA-specific bits.

Defer to the gnome-online-accounts module.  See the previous commit.

gnome-online-accounts: Add an EDataFactory extension.

This extension takes over for the GOA bits in EDataBookFactory.

Install this extension to both the addressbook and calendar backend
directories, even though only EBookBackendGoogle uses it at present.

EDataFactory: Add a "backend-created" signal.

Allows EDataFactory extensions to catch newly-created backends and
perform additional initialization on it.

The use case I have in mind is to move all the GOA initialization in
EDataBookFactory to the gnome-online-accounts module as an extension.

Note this uses one of the reserved pointer slots in EDataFactoryClass
and therefore does not break ABI.

Remove unused OAUTH_CFLAGS / OAUTH_LIBS.

Missed these when I removed the liboauth dependency.

Coding style and whitespace cleanup.

Updated POTFILES.in

EBookBackendSqliteDB cleanups.

Improve error handling and make commits/rollbacks less awkward.

Use g_type_ensure() where applicable.

CamelIMAPXServer: Work around crash in imapx_exit_idle().

g_cond_clear() doesn't like being given an uninitialized GCond.

The "start_watch_cond" and "start_watch_exit" structures are being
cleared unconditionally, so we should initialize them unconditionally.

Updated Slovenian translation

Restore e_flag_timed_wait().

Just deprecate the function, don't delete it.  GTimeVal isn't going
away any time soon.  It's not worth a libedataserver soname bump.

Disabled debug print in EBookBackendFile

This was somehow missed in my last commit.

Local file backend modified to use sqlite db exclusively

Patch includes a migration routine to migrate data from a previously
installed BDB.

EBookBackendSqliteDB: Added e_book_backend_sqlitedb_get/set_revision()

In preperation for the migration, there needs to be a way to persist the
overall addressbook revision. This patch also increments the internal
SQLite DB format version and adds the new revision column to for main
folders table of an addressbook (and increments the stored DB version)
if the DB version is out of date.

EBookBackendSqliteDB: Fixed sexp interpretation for 'id' field.

In the sqlite backend, the 'id' is stored as the 'uid' column.

Revert "calendar: Drop old ECalSourceTypeEnum from introspected API"

This reverts commit 3183733023f022a89980cbf3f7292bde0bbd9954.

We no longer introspect deprecated API.  Remove the cruft.

Disable runtime check in source_registry_initable_init().

Rationale explained in the source code.

Bug 687694 - imapx_untagged_vanished() is inefficient

For uid->len = 318320, timing the loop yields:

g_list_append(): 60+ seconds
g_list_prepend(): 0.3 seconds

Bug #687642 - Crash under caldav_synch_slave_loop()

Bug #669082 - Workaround crash in imapx_command_copy_messages_step_done()

Google: allow writing X-GOOGLE-SYSTEM-GROUP-IDS

We expose GData system groups as an extended attribute with that
name, but if we found it in the EContact we would turn it into
an extended property. Instead, marshal the system group IDs into
Evolution categories, and then those into GData groups or system
groups.

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

Bug #687776 - E-D-S crash in local address book

Introduce --enable-strict configure option

This is currently used to add extra AM_CPPFLAGS to disable
deprecated symbols from GLib.

Updated Norwegian bokmål translation

Use soup_message_headers_get_list() as a replacement of soup_message_headers_get()

CalDAV backend didn't receive proper value for access right, which
caused reject of the login to the server.

Replace deprecated soup_message_headers_get()

Replace deprecated GLib symbols (as of GLib 2.34.x)

[l10n] Fixed German translation, bug #687562

EBookBackendSqliteDB: Add argument guards.

Updated Russian translation

vala: Remove unused metadata.

Exclude deprecated API from introspection.

Deprecated APIs were deprecated for a reason.  The last thing we
should be doing is enabling their continued use in other languages.

Henceforth, any symbols tagged with EDS_DISABLED_DEPRECATED will be
automatically excluded from introspection and language bindings.

Introspection cleanups.

Remove E_BOOK_DISABLE_DEPRECATED and E_CAL_DISABLE_DEPRECATED.

We already use EDS_DISABLE_DEPRECATED, these other definitions only
confuse matters.

Suppress g-ir-scanner warnings.

They're far too noisy, and make it difficult to spot new C compiler
warnings, which are the only ones I care about.  We can enable them
again when we're serious about cleaning them up.  In the meantime,
they're just getting in the way.

calendar: Drop old ECalSourceTypeEnum from introspected API

It clashes when generating the vala bindings, and we're just keeping
it around for C API consumers.

e-cal-view.c: Remove unnecessary #includes.

e-cal.c: Remove unnecessary #includes.

e-book-view.c: Remove unnecessary #includes.

e-book.c: Remove unnecessary #includes.

Update libedataserverui API documentation.

Fix invalid Gtk-Doc syntax in comments.

Not certain this is everything, just what I could find.

Reimplement ECal using ECalClient.

This is primarily so I don't have to duplicate work while overhauling
the D-Bus API, but it's good to do anyway; it simplifies the deprecated
logic and helps ensure the two APIs behave consistently.

Reimplement ECalView using ECalClientView.

This is primarily so I don't have to duplicate work while overhauling
the D-Bus API, but it's good to do anyway; it simplifies the deprecated
logic and helps ensure the two APIs behave consistently.

Reimplement EBook using EBookClient.

This is primarily so I don't have to duplicate work while overhauling
the D-Bus API, but it's good to do anyway: it simplifies the deprecated
logic and helps ensure the two APIs behave consistently.

Reimplement EBookView using EBookClientView.

This is primarily so I don't have to duplicate work while overhauling
the D-Bus API, but it's good to do anyway: it simplifies the deprecated
logic and helps ensure the two APIs behave consistently.

e-cal-view.h cleanups.

e-book-view.h cleanups.

e_async_closure_callback: Do not test 'object' parameter validity

It is unnecessary to test validity of the 'object' parameter,
especially when this callback does nothing with it. The reason
is that the secret_password_store() calls the callback with NULL
'object', which leads to a deadlock on the source registry side.

Bug #685090 - GOA EWS module doesn't save all required values

Collection's child source Enabled should match collections Enabled

Revert "Revert "Bug #678400 - ECollectionBackend populates disabled accounts""

This reverts commit 1c4fb39c131a03428035a2035fca8ea462fc7714,
because the initial revert was a misunderstanding of the change.

Add AllowAuthPromptAll() method to SourceManager interface.

Equivalent to calling AllowAuthPrompt() on each exported data source,
but cuts way down on the D-Bus traffic.  Since the registry service is
commonly activated during a desktop session startup, this can actually
clog up the session bus and slow down login.

This leaves the source-specific AllowAuthPrompt() method unused, but
there are still use cases for it which we haven't implemented yet, so
leave it be.

Remove version number from ${privincludedir}.

Last man standing, so may as well.  Version not needed there either.

Remove erroneous version number from ${privlibdir}.

This slipped through because we don't actually reference ${privlibdir}
in our backend/module/provider directory definitions.  Fix that too.

Also, some pkg-config files needed to be tweaked as a result of this.

Remove version number from ${privdatadir}.

In talking to some people at Red Hat about how to deploy pre-configured
Evolution accounts, one thing I suggested is to provide a package which
installs .source files to one of the system-wide locations:

    /usr/share/evolution-data-server-3.8/ro-sources
    /usr/share/evolution-data-server-3.8/rw-sources

But that's not going to work in general with the version number in the
path.  Upgrading E-D-S to, say, 3.10 would invalidate such a package.

Remove the version number from our ${privdatadir}.  It's not needed.

Bug #681962 - Identify server certificates by hostname and fingerprint

Bug #685034 - [imapx] Read status of email is not persisted correctly

Bug #687100 - Crash under cal_backend_contacts_remove_book_record()

Bug #686924 - Doesn't parse Gerrit encoded From address correctly

imapx_command_start: Improve handling of write-to-stream error.

For some reason we were calling camel_stream_write_string() without
passing a GError, and on failure setting a very generic error message
and losing potentially useful information about the actual error.

Use code coverage macros from gnome-common.

The GNOME_CODE_COVERAGE macro in gnome-common automates the process
nicely.  Just run "make check-code-coverage" from the top-level source
directory (or from any of the "tests" subdirectories) and open the nice
coverage analysis report:

    evolution-data-server-x.y.z-coverage/index.html

Bug 685986 - ESourceRegistry: Wait for signals after creating sources

e_source_registry_create_sources_sync() now waits for "object-added"
signals from its GDBusObjectManagerClient before returning.  This allows
the newly-created ESources to be obtained immediately after the function
returns.  Prior to this there was a small delay before the newly-created
ESources became available through the ESourceRegistry API.

There is a caveat to this.  If any of the scratch ESources reference a
parent UID which is not presently exported by the registry service, the
registry service will refuse to export the newly-created ESource until
its parent becomes available.

In this scenario, to avoid waiting forever for an "object-added" signal
that isn't coming, we place a limit on the wait time (two seconds).  If
this time limit expires, we return from the function regardless.

Add a test case for bug 685986.

Managed to reproduce the bug using the test case.

Prototype a D-Bus unit test harness.

Add functions for programmatically overriding XDG base directories
and launching a private D-Bus session bus using GTestDBus.

This requires Evolution-Data-Server to be installed, since the test
harness references D-Bus service files by the installation prefix.

Bump minimum GLib requirement to 2.34.

Need to utilize the GTestDBus API, which is new in 2.34.

Updated Indonesian translation

Bug 686719 - Missing linker flags in Camel tests

Updated Slovenian translation

Prefer e_source_registry_check_enabled().

Prefer e_source_registry_check_enabled() over e_source_get_enabled()
for most instances where we want to exclude disabled data sources.

Add e_source_registry_check_enabled().

Checks the enabled state of a given #ESource and of its ancestors to
determine if the #ESource should be displayed or acted on.

This is somewhat similar to widget sensitivity in GTK+.
cf. gtk_widget_get_sensitive() versus gtk_widget_is_sensitive()

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.