Introduce e_gdbus_templates_init_main_thread()

This is required to properly distinguish between EGDbusTemplates'
sync calls from the main thread and from a dedicated thread, because
the check with g_main_context_is_owner (g_main_context_default ())
doesn't work when there is no main loop running. The call to
e_gdbus_templates_init_main_thread() is good to do, but it is not
mandatory.

Add camel_operation_cancel_all().

Replaces camel_operation_cancel(NULL).

For non-NULL cases, use g_cancellable_cancel() instead.

Remove camel_operation_uncancel().

Use g_cancellable_reset() instead.

Remove camel_operation_cancel_check().

Use g_cancellable_is_cancelled() instead.

Remove camel_operation_cancel_fd().

Use g_cancellable_get_fd() instead.

Remove camel_operation_cancel_prfd().

No longer needed by anything.  Also allowed me to remove all the
CamelMsgPort muck in CamelOperation, which simplifies things nicely.

Use the same technique in imapx_parser_thread().

Call PR_Interrupt() from a GCancellable::cancelled signal handler to
cancel the blocking PR_Poll() call in imapx_parser_thread().

CamelTcpStreamRaw: Use PR_Interrupt() to cancel blocked I/O.

Call PR_Interrupt() from a GCancellable::cancelled signal handler to
cancel blocking PR_Connect(), PR_Read() and PR_Write() calls.

Way easier and more responsive than using camel_operation_cancel_prfd().

camel_settings_load_from_url(): Forgot to handle "auth-mechanism".

camel_settings_save_to_url(): Forgot to handle "auth-mechanism".

CamelNetworkSettings: host and user should never be NULL.

Otherwise it creates nuisance runtime warnings when binding the
properties to widgets like GtkEntry.

Updated Galician translations

Updated Spanish translation

Bug #655167 - Messages moved from IMAP Inbox reappear

Added test case showing photo data stored as uris.

The test asserts:
  o binary inlined photos added to the EBook come out
    as uris in the EBookView signals
  o that it is still possible to use an external uri that the
    addressbook does not recognize at which point the addressbook
    is simply expected to store the provided URI string without
    any extra management (the test does this, however it only
    asserts that a uri is indeed returned).
  o When sharing an addressbook owned uri fetched from one contact
    and assigning it to the next contact, the second contact's uri
    is still accessible on disk after deleting the first contact.

Make local addressbook backend store image data as URIs.

Whenever the local address book receives a uri as a binary
blob it proceeds to transform it to a uri accessbile on
the system somewhere under $XDG_DATA_HOME. The local backend
cleans up old photo uris when contacts are removed or modified
to use new photos.

Additionally, whenever it is detected that the user is
cross-referencing a uri which belongs to another contact,
a hard-link will be created to the addressbook owned file
on disk and a new uri will be assigned to any additional
contacts which try to share a uri owned by the addressbook.

Bug #663062 - Missing attachment icon on a message with it

Avoid possible crash when url->path is NULL

Add a comment explaining CamelStore's mode field.

I was about to get rid of this field thinking it was unused, but as
usual it's a GroupWise-specific hack for non-writable proxy accounts.
Add a comment to the header file for future reference.

Remove g_thread_init() calls.

g_type_init() already calls it, and has for several GLib releases.

IMAP: Split off a new mutex for commands and responses.

Define a new public GStaticRecMutex named "command_and_response_lock"
on CamelImapStore for IMAP command and response operations to use.

Fixes some deadlocks I was encountering due to the overreliance on
CAMEL_SERVICE_REC_CONNECT_LOCK, particularly during authentication.

camel_imap_store_connected() cleanup.

If camel_service_connect_sync() set a GError, don't overwrite it with a
generic "you must be working online" message.  The connect_sync() error
message has useful information to help identify the real problem.

Updated Slovenian translation

Bug 662524 - POP3: Use a safer default value for "delete-after-days"

Default to not deleting mail kept on the POP3 server at all rather than
automatically deleting after X days.  Helps avoid accidental data loss.

Updated Spanish translation

Bug #660182 - Error dialog on LDAP query with multiple referrals

Bug 662643 - camel_url_decode_path() crashes on empty string

Move remaining CamelURL elements to CamelSettings.

This is a continuation of the CamelSettings work I started in 3.1.

CamelService no longer stores a CamelURL.  The URL components are being
moved to CamelSettings.

For network services, the host, port, user and authentication mechanism
URL components now live in CamelNetworkSettings.

For local services, the path URL component (aka, the file path to the
root of the local mail store) now lives in CamelLocalSettings.

For now these new settings are configured the same as other settings:
through camel_settings_load_from_url().  But again, that function will
be dropped as soon as we move to key files for account data storage.

Also:

- Remove camel_service_get_url(), which is no longer used.

- camel_session_add_service() now takes a protocol name instead of a
  URL string.  The protocol name must match a registered CamelProvider.

- camel_service_get_camel_url() is renamed camel_service_new_camel_url().
  It now returns a new CamelURL which must be freed with camel_url_free().
  The returned CamelURL is built from user, host, port and path settings,
  and does not have any parameters.  You MUST access settings through
  CamelSettings now.

- CamelServices need to be prepared for CamelSettings values changing or
  even the entire CamelSettings object being replaced at any time.  Most
  services were already prepared, but I caught a few services caching
  settings during initialization.  Always consult CamelSettings when a
  setting value is needed so the service is acting on a fresh value.

- Camel's SONAME is hereby bumped (yet again) for 3.3.2.

Call g_thread_init() conditionally

Post-release version bump.

NEWS update for 3.3.1 release.

Avoid a crash in camel_session_get_service_by_url().

Check that we found a CamelProvider before dereferencing it.

Add camel_folder_refresh_info()/_finish().

I forget to write wrapper functions for CamelFolderClass.refresh_info()
and CamelFolderClass.refresh_info_finish() even though the class methods
have been there for awhile.

Thanks to Srini for pointing this out.

Updated Slovenian translation

Added test to show that fields-of-interest is working in libecal

Test is based on test-client-get-view.c, this version of the test
sets fields-of-interest to UID, RECURRENCE-ID and LAST-MODIFIED
fields, tests that all those fields are returned and asserts that
the event summary (which is indeed set) is always NULL.

Updated Polish translation

Updated Polish translation

Updated Polish translation

Bug #660870 - Empty Contacts list when pressing "To:" in composer

Revert "Remove of -Werror-deprecated-function-declaration", it doesn't work

Bug #660656 - Fix e_iterator_get annotation

Updated Belarusian translation.

Remove -Werror-deprecated-function-declaration to make it build for the 3.4 moduleset

Bug #527614 - Valgrind shows a nice amount of lost memory

Updated Spanish translation

Updated Spanish translation

Fix compiler warnings from GOA code in book factory

Really implement e_cal_client_view_set_fields_of_interest().

As discussed on bug https://bugzilla.gnome.org/show_bug.cgi?id=652180,
this patch adds many '_component' variants to functions that are named
with '_object', all the '_component' variants take an 'icalcomponent *'
instead of an ical string.

The magic filtering happens in e_data_cal_view_get_component_string()
which creates an ical string representation while omitting properties
that are not mentioned in the fields of interest.

[l10n] Updated German translation

[l10n] Updated German translation

Avoid possible deadlock with CamelStoreSummary locks

Recently added store_summary_sync_folder_summary_count_cb() could
cause deadlock when locking SUMMARY_LOCK first, and then calling
functions which lock REF_LOCK before SUMMARY_LOCK. Locking REF_LOCK
first avoids this.

Bug #661549 - Avoid fetching from backend when calculating summary

Bug #656603 - Add support for generating vCard 2.1 in libebook

Updated Norwegian bokmål translation

Updated Galician translations

Updated Japanese translation

l10n: updated Italian translation

Update libedata-book API docs.

Update libebook API docs.

Update libedata-cal API docs.

Update libedataserver API docs.

Update Camel API docs.

Updated Korean translation

Updated British English translation

smtp: Adapt to Camel's new authentication API.

pop3: Adapt to Camel's new authentication API.

nntp: Adapt to Camel's new authentication API.

imapx: Adapt to Camel's new authentication API.

imap: Adapt to Camel's new authentication API.

Camel: New authentication API.

Currently each provider has to implement its own authentication loop.

The idea for these changes is add a CamelSession method that executes
a common authentication loop for all providers:

    CamelSessionClass:

    gboolean  (*authenticate_sync)   (CamelSession *session,
                                      CamelService *service,
                                      const gchar *mechanism,
                                      GCancellable *cancellable,
                                      GError **error);

    void      (*authenticate)        (CamelSession *session,
                                      CamelService *service,
                                      const gchar *mechanism,
                                      gint io_priority,
                                      GCancellable *cancellable,
                                      GAsyncReadyCallback callback,
                                      gpointer user_data);

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

Each CamelService is then responsible for implementing a new
authenticate method that makes only ONE authentication attempt and
returns an appropriate status code:

    CamelServiceClass:

    CamelAuthenticationResult
              (*authenticate_sync)   (CamelService *service,
                                      const gchar *mechanism,
                                      GCancellable *cancellable,
                                      GError **error);

    void      (*authenticate)        (CamelService *service,
                                      const gchar *mechanism,
                                      gint io_priority,
                                      GCancellable *cancellable,
                                      GAsyncReadyCallback callback,
                                      gpointer user_data);

    CamelAuthenticationResult
              (*authenticate_finish) (CamelService *service,
                                      GAsyncResult *result,
                                      GError **error);

The status codes are defined by the CamelAuthenticationResult enum:

    CAMEL_AUTHENTICATION_SUCCESS    : auth credentials were accepted
    CAMEL_AUTHENTICATION_FAILURE    : auth credentials were rejected
    CAMEL_AUTHENTICATION_ERROR      : something went horribly wrong!

CAMEL_AUTHENTICATION_SUCCESS and CAMEL_AUTHENTICATION_ERROR both
terminate the authentication loop, but CAMEL_AUTHENTICATION_FAILURE
cycles the loop and asks the CamelService to retry authentication,
usually after prompting the user to enter a different password.

Updaged Galician translations

Require libgdata >= 0.10.

Drop all the backward-compatibility cruft.

Updated Spanish translation

Bug #661448 - e_util_copy_*_slist(): Avoid using append function

The current implementation calls g_slist_append() for each element
of the list to copy. However, g_slist_append() is expensive
because it iterate over the whole destination list.

This patch alters the behavior to first make a deep copy of the
list and then concatenate the destination list and the list copy.
The concatenation only iterates of the destination list once.

Updated Hungarian translation

EBookClient: Avoid useless copying of valid UTF-8 strings

Make use of e_util_ensure_gdbus_string() instead of
e_util_utf8_make_valid() whenever possible to avoid useless
copying/freeing of valid UTF-8 strings (likely case).

e-book-backend-file: Don't share DB_ENV between address books from different sources

A DB_ENV is associated to the folder when the database is stored, in
order to store transaction logs. As a consequence, a DB_ENV should
never be shared between address books from different sources, since
they are kept in separate folders.

ENameSelectorEntry::user_delete_text: Delete whole text when user deleted it

In cases like setting an empty string to a GtkEntry the event
passes -1 as end_pos, where ENameSelectorEntry didn't behave properly.

Bug #661432 - 'Any field contains' search doesn't work in contact view

Bug #660829 - Allow 'Mark as read' in right click menu on spam folder

Revert "gdbus_proxy_call_sync: stop using g_usleep()"

This reverts commit cf78ee827d3c5157e44a1e4c32d395942a816461.

EBookClientView: Use e_util_free_object_slist() utility function

e-book-client: Refactor bulk methods code

Add a contact_slist_to_utf8_vcard_array() utility function to
avoid code duplication in EBookClient bulk methods.

gdbus_proxy_call_sync: stop using g_usleep()

Instead, tell g_main_context_iteration() that it may block if no
event source is ready.

Don't free vcard memory in e_data_book_view_notify_update*_vcard()

Instead, the vcard memory is freed by the caller which feels more
consistent with the rest of the code and less error-prone.

e_book_backend_sqlitedb_new: Fix database connection reuse

Remove EBookBackendSqliteDB objects from the db_connections
hash table in their dispose function. It used to be done in
their finalize function but this would cause
g_hash_table_lookup() to sometime return EBookBackendSqliteDB
GObjects whose ref_count is 0, under heavy load.

This patch also makes sure the database mutex is properly unlocked
in case of error. It also properly reports errors when
book_backend_sqlitedb_load() fails.

Updated Telugu Translations

libebook: Add unit test for bulk modification function

e-book-client: Add bulk contact modification methods.

This reduces the number of DBus round-trips and enables
optimizations in the backend.
This patch also makes use of Berkeley DB transactions in the file
backend.

Workaround possible crash in imap_body_decode()

Updated Vietnamese translation

vi.po: import from Damned Lies

Updated Norwegian bokmål translation

Updated Swedish translation

Updated Spanish translation

configure.ac: Bump ADDRESS_BOOK_DBUS_SERVICE_NAME to 2

The DBus API has changed with the addition of bulk contact
addition methods.

addressbook: Improve bulk contact removal code

The patch makes use of Berkeley DB transations in the file backend
and roll back the transaction in case of error to make the
behavior consistent with bulk addition.

This patch also makes the webdav backend behave as expected since
it does not support bulk removal and the implementation was broken.

libebook: Add test for bulk methods

e-book-client: add e_book_client_add_contacts*() methods

Enables optimizations for batch additions of contacts such as
less DBus round trips and use of database transactions in file
backend.

build_sqlitedb(): remove useless free() for vcard_dbt.data