camel_flag_list_copy: Avoid false 'changed' notifications

The function, as written, returned that the destination list
changed even in cases where both flag lists were empty. This
made for example IMAPx report that each message in a folder
hanged, even there did not change any.

imapx_write_flags: Fix a typo which flag to skip

The code was surely trying to avoid writing CAMEL_IMAPX_MESSAGE_RECENT
flag back to the server, but the actual behaviour was that each flag
had been ignored, if the CAMEL_IMAPX_MESSAGE_RECENT was set.
This could have an effect of copied read message being marked as unread
in the destination IMAPx folder.

[Esource] Do not set properties on load which did not change

When calling g_object_set(), g_object_set_property() and similar
set functions, the GObject itself notifies about the property change,
even if the value didn't change at all. This was exhibited by false
notifications when (re)loading ESource on a change on all its saved
properties, which, in particular, was causing bug #722399.

Update Chinese simplified translation

Bug #720943 - vFolder does not refresh automatically

Assamese translation updated

Bug 722275 - IMAPX: Parser trips on mailbox containing '+'

Mailbox names containing a '+' character were tripping up the parser due
to it being treated as one of a special set of characters to distinguish
identifiers from IMAP syntax.  But I don't think '+' is necessary in the
set of characters.  For continuations, at least, it should always follow
a newline character, with is also a special character.

IMAPX: Remove some unused character macros.

Bug #702709 - Do network background operations only if the CamelSession is online

This may prevent any network background operations, when associated
CamelSession is offline, like when the machine gets suspended or is
just resumed.

[CamelGPGContext] Prefer gpg2 over gpg

If there is a newer version installed, then use it, rather than
the old.

Server-side memory leaks in calendar factory

This change addresses:
- typo for GDestroyNotify (was being used ref, instead of unref)
- leaking data_cal object when opening a calendar
- DataCalView dispose didn't remove view from the backend

These led to memory leaks on both view and backend objects.

[CalDAV] Force password ask on soup authenticate failure

Some servers may allow OPTIONS request on authenticated account with
a wrong password, but fail on REPORT and other queries, which makes
the calendar open routine not recognize the wrong password case.

Bug #722415 - Memo does not support Start Date property

Add camel_imapx_server_shutdown().

Lets CamelIMAPXStore explicitly tell its server to shut down command
processing.  Better than doing this implicitly from dispose(), since
both the idle and parser threads now hold a reference to the server,
and we no longer join with the idle thread.

CamelStore: Remove noop_sync() method.

No longer used.

Removed functions:

  camel_store_noop_sync()
  camel_store_noop()
  camel_store_noop_finish()

Remove imapx_store_noop_sync().

No longer needed; see the previous commit.

CamelIMAPXServer: Ping the server after a period of inactivity.

If an IDLE command is in progress, stop and restart the command.
Otherwise issue a NOOP command.  This just lets the IMAP server
know we're still here to (hopefully) avoid being logged off.

This obviates the need for CamelStore's noop() method, which can
now be removed since it was only needed for IMAP.

CamelIMAPXServer: A little code reorganization.

CamelIMAPXServer: Work around refcounting issue in IDLE thread.

Use the ol' weak-reference-as-callback-closure trick to avoid IDLE
command callbacks holding the last CamelIMAPXServer reference.

Add missing include to git.mk in examples/Makefile.am

Bug #720676 - [IMAPX] Do more searching server-side

The SEARCH command doesn't support all the things evolution can
search for, but it can cover the most common cases, which speeds
searching significantly (especially when the folder content is not
required locally).

CamelIMAPXServer: Use a main loop in the parser thread.

The "while" loop previously used in the parser thread just blocked the
thread until incoming data became available.  Using a GMainLoop allows
us to add other event sources, like periodic timeouts to help keep the
connection alive without relying on Evolution.

I'm also hoping this will help eliminate the separate thread for IDLE,
since it now uses a GMainLoop as well.  But I don't quite have my head
around all the implications yet.

CamelIMAPXServer: Remove unused casting macro.

Updated Traditional Chinese translation(Hong Kong and Taiwan)

Bug #687640 - Crash in caldav's initialize_backend()

Two threads could call initialize_backend() simultaneously,
overwriting data one to the other, which caused the crash.

CamelIMAPXServer: Give IDLE state enum a proper type definition.

CamelIMAPXServer: Use a main loop in the IDLE thread.

Instead of mucking with infinite "while" loops and condition variables,
run a GMainLoop in the IDLE thread and attach a GSource from the parser
thread when an IDLE command should be issued.

CamelIMAPXServer: Remove IDLE_LOCK/UNLOCK() macros.

Just call g_mutex_lock/unlock() directly.

CamelIMAPXServer: Move IDLE structs to private section.

Bug #721581 - Unable to write to ldap addressbook

Post-release version bump.

NEWS update for 3.11.4 release.

Bug 720657 - IMAPX: No new messages are received

Regression caused by removal of "mobile mode".  I misinterpreted some
complex expression embedded in a loop condition, which is why I don't
like complex expressions in loop conditions - difficult to for others
grok, which means high potential for mistakes.

Updated dutch translation Master (3.12)

IMAPX: Handle BODY[HEADER] in FETCH response.

A broken IMAP server (whatever they use at Masaryk University) was
seen having this conversation:

  C: A001 UID FETCH 1:* (RFC822.SIZE RFC822.HEADER FLAGS)
  S: * 1 FETCH (RFC822.SIZE 4220 BODY[HEADER] {523} ...)
  ...

This confuses Camel into thinking it was a request for a full message
and it in turn fails to update the folder summary and issues a warning
about not being able to locate an appropriate FETCH job.

This works around the issue by treating BODY[HEADER] as equivalent to
RFC822.HEADER in untagged FETCH responses.

CamelIMAPXNamespace: Allow NIL separator character.

RFC 2342 allows it:

  Namespace_Response = "*" SP "NAMESPACE" SP Namespace SP Namespace
                       SP Namespace

  Namespace = nil / "(" 1*( "(" string SP  (<"> QUOTED_CHAR <"> /
              nil) *(Namespace_Response_Extension) ")" ) ")"
              ^^^

configure.ac: trivial typo fix in last commit

-ldb vs. -ldl

tests: #include <sys/wait.h> for WIFEXITED

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

Remove #include <malloc.h> from a file

This ancient header has long been deprecated in favour of <stdlib.h> but
this file uses GLib memory functions anyway, so neither is required.

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

Bug 721733 - Tweak Berkeley DB detection logic

Make two changes to the logic that we use to find the Berkeley DB:

 - in the default case where --with-libdb is not given, do not attempt
   to add the default ${prefix}/include and ${prefix}/lib to the CFLAGS
   and LIBS.  Only do this if --with-libdb is explicitly given.

 - in the case where --with-libdb is not given, and DB_CFLAGS or DB_LIBS
   is already set, use those values instead of our own guesses

This allows the build to proceed on FreeBSD, where the headers are in an
unusual location and the library has an unusual name.

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

Bug 721527 - License text contains obsolete FSF postal address

Remove camel/README.COPYRIGHT.

The instructions are out-of-date, and advise contacting Novell Inc.

Bump SOURCES_DBUS_SERVICE_NAME version.

To cover both the new "ServerError" D-Bus signal and the new built-in
"system-proxy" data source.

Updated Indonesian translation

Updated Czech translation

Updated Brazilian Portuguese translation

Updated Galician translations

Updated Hungarian translation

Updated Greek translation

Update project homepage URL

Post-release version bump.

NEWS update for 3.11.3 release.

CamelIMAPXServer: Fix handling of LSUB responses.

If we receive an LSUB response on a mailbox name but no corresponding
LIST response, ignore it.  IMAP allows subscriptions to mailbox names
regardless of whether the mailbox actually exists.  Not useful to us.

CamelStore: Fix typo preventing valid folder renames.

IMAPX: Remove unused member from GetMessageData struct.

Coding style and whitespace cleanups.

Update API documentation.

Bug 715035 - cache-reaper: Demote g_message() -> g_debug()

No need to tell the system administrator about these.

CamelMimeFilter: Function signature cleanup.

CamelGpgContext: Fix broken debugging statements.

Camel likes to use this debugging idiom:

  #define d(x)

and then wraps debugging statements like so:

  d (printf (...));

One is supposed to enable debugging by changing the macro to:

  #define d(x) x

This idiom makes debugging statements hard to grep for, but moreover
they don't get compiled regularly and so they tend to bit rot.

As was the case in CamelGpgContext, where the debugging statements
didn't build because they were still written for Camel's old home-
grown object system.

CamelStreamFilter: Fix broken debugging statements.

Camel likes to use this debugging idiom:

  #define d(x)

and then wraps debugging statements like so:

  d (printf (...));

One is supposed to enable debugging by changing the macro to:

  #define d(x) x

This idiom makes debugging statements hard to grep for, but moreover
they don't get compiled regularly and so they tend to bit rot.

As was the case in CamelStreamFilter, where the debugging statements
didn't build because they were calling CamelStream functions as they
existed YEARS ago.

Updated POTFILES.skip

Migrated tests/cursor-example to examples/cursor

Examples can be added to the examples subdirectory which does
not belong in 'tests'. Note that the new documentation contains
an examples section, the file eds-cursor-example.sgml is manually
written and includes source code inline, along with some commentary
on the source code example.

Restructuring documentation build.

This commit unifies the various EDS related reference manuals into
a single 'eds' reference manual. This comprises libedataserver, libebackend,
libebook, libebook-contacts, libedata-book, libecal, libedata-cal into
a single reference manual. The camel and private reference manuals are
left alone.

ETestServerFixture: Fixing gtk-doc comments.

Libedataserver: Fixing gtk-doc comments and adding gtk-doc section headers

Libebackend: Fixing gtk-doc comments and adding gtk-doc section headers

Calendar: Fixing gtk-doc comments and adding gtk-doc section headers

Addressbook: Fixing gtk-doc comments and adding gtk-doc section headers

CamelFolderSummary: Add a FIXME comment.

Resolving corner cases so I can get rid of camel_folder_lock/unlock().

CamelTransport: Stop using the internal 'send_lock'.

Use camel_service_queue_task() instead of locking an internal mutex.
This ensures methods are run in the order they were called, and also
makes cancellation of a waiting method immediate instead of blocking
until it acquires the mutex it's competing for.

This requires flip-flopping the synchronous and asynchronous functions,
so that the synchronous function calls its asynchronous counterpart with
help from CamelAsyncClosure.

CamelStore: Stop using the internal 'folder_lock'.

Use camel_service_queue_task() instead of locking an internal mutex.
This ensures methods are run in the order they were called, and also
makes cancellation of a waiting method immediate instead of blocking
until it acquires the mutex it's competing for.

This requires flip-flopping some synchronous and asynchronous functions,
so that the synchronous function calls its asynchronous counterpart with
help from CamelAsyncClosure.

Removed functions:

  camel_store_lock()
  camel_store_unlock()

Add CAMEL_CHECK_LOCAL_GERROR.

Same as CAMEL_CHECK_GERROR, but for direct GError pointers.

Example:

    success = class->foo (object, some_data, &local_error);
    CAMEL_CHECK_LOCAL_GERROR (object, foo, success, local_error);
    return success;

Add camel_service_queue_task().

Adds a GTask to a queue of waiting tasks with the same source object.
Queued tasks execute one at a time from a separate thread in the order
they were added.

This is primarily intended for use by CamelStore, CamelTransport and
CamelFolder to achieve ordered invocation of synchronous class methods.

This will finally fix Camel's long-standing (mis)use of mutexes as a
dispatch mechanism, which doesn't preserve order and isn't cancellable.

Bug 710361 - GError corruption in imapx_command_fetch_message_done()

camel_imapx_job_take_error() was being called twice on the same GError.
The 2nd call freed the GError set in the 1st call, which corrupted the
memory.  Put a safeguard in camel_imapx_job_take_error() to catch this
in the future.

Bug 719720 - Refcount error in imapx_command_select_done()

We were finalizing the CamelIMAPXCommand too early, and then trying
to use it further down in the logic.  This of course led to crashes.

Failed command structures were also leaking.  Fixed that too.

Remove --enable-strict configure option.

GLib/GTK+ uses different deprecation macros nowadays.

The MIN_REQUIRED macros do a better job of catching deprecated symbol
usage up to the minimum build requirement, and the MAX_ALLOWED macros
catch symbol usage that violates our minimum build requirement.

These newer deprecation macros are always enabled, so we don't need a
special configure flag for them.

Deprecate ECancellable[Rec]Mutex.

Bug 719774 - Missing linker flag in evolution-addressbook-factory

Bug 719307 - trust-prompt: Squash a compiler warning

gcc tells us this could be uninitialized, which will only happen if we
fail to base64 decode.

Moved all libecal/client tests to have a namespace

Avoid naming conflicts when installing tests into ${pkglibexecdir}/installed-tests

Moved all libebook/client tests to have a namespace

Avoid naming conflicts when installing tests into ${pkglibexecdir}/installed-tests

tests: Fix potential g_object_unref(NULL) calls in libebook tests

Found by scan-build.

https://bugzilla.gnome.org/719533

libedata-cal: Fix a potential g_strstrip(NULL) call

If the calendar contained any attendees _not_ prefixed with ‘mailto:’,
the text variable would be NULL, and would cause an assertion failure in
g_strstrip().

Found by scan-build.

https://bugzilla.gnome.org/719533

libecal: Refactor system timezone key file parsing code

Previously, the code could call g_strstrip(NULL) if a line started with
a quotation mark but didn’t end with one, e.g.:
    MyKey="some-unbalanced-value

The code now ignores unbalanced quotation marks, returning the odd one
in the value rather than crashing (or stripping it).

Found by scan-build.

https://bugzilla.gnome.org/719533

EBookBackendCache: Remove an unnecessary NULL check.

A newly-created EContact is never NULL, so the check is redundant.
It was confusing the static analyser.

https://bugzilla.gnome.org/719533

CamelPOP3Folder: Fix a potential g_object_unref(NULL) call

If all the branch conditions in this function evaluate to FALSE, the
stream is never set, and hence g_object_unref(NULL) is called on exiting
the function.

Found by scan-build.

https://bugzilla.gnome.org/719533

CamelTransport: Port to GTask.

CamelSubscribable: Port to GTask.

CamelStore: Port to GTask.

CamelSession: Port to GTask.

CamelService: Port to GTask.

CamelSasl: Port to GTask.

CamelOfflineFolder: Port to GTask.

CamelNetworkService: Port to GTask.

CamelMimePart: Port to GTask.

CamelFolder: Port to GTask.

CamelDataWrapper: Port to GTask.

CamelCipherContext: Port to GTask.

CamelSession: Remove all asynchronous class methods.

For consistency with CamelStore, CamelFolder, etc.