TEMPORARILY build without weather calendars in distcheck.

Will revert this when we see a libgweather-2.90.x release.

Fix E_DATA_SERVER_UI_DEPS.

Was hardcoding gtk+-2.0 instead of using $GTK_PACKAGE.

Show the GTK+ package in configure summary.

Build against gtk+-3.0 when running distcheck.

Require gweather >= 2.90.0 when building against gtk+-3.0.

Note, at this moment libgweather-2.90.0 doesn't exist.  The version
requirement is a guess.  The tarball should appear "any minute now".

Bug #622350 - CamelFolder's "deleted" signal prototype changed

Update libedataserverui versioning for gtk3. 3.0 should build gtk3 and 1.2 with gtk2

Add helper to show the name selector with correct transience, to
fix the umpteen bugs in its use in Evolution.

Update summary uidnext after fetching messages.

Since we don't refetch flags if we've fetched new messages for a previously
empty folder, we weren't updating the stored uidnext -- which meant we would
refetch messages again next time. And if the folder is empty, we'd refetch
messages every time and never update uidnext.

Bug 622828 - Don't recurse to parent class's ->disconnect() method.

That seems to be our caller. Calling it back will just end in tears.

Bug 622822 - Fix failure return from nntp_folder_refresh_info_online()

When invoked with no fmt argument, camel_nntp_command() is only supposed
to select the folder and return zero.

Since commit 08cfec3e we have been interpreting that zero as FALSE, and
this is what causes nntp folders to appear empty as refresh_info appears
to fail.

Fix nntp folder tree building

When it creates a 'virtual' NOSELECT|CHILDREN folder to fill a gap in
the hierarchy, it doesn't correctly insert that into the tree if it is a
top-level folder (i.e. last == NULL).

Once you fix that bug, you end up with two identical copies of the
tree-insertion code. I moved that out into a separate function.

Bug 622719 - Fix lifetime issues with CamelIMAPXServer w.r.t. disconnection

Fix idle thread exit and locking

If we trigger its exit and wait for it while still holding the lock, it may
deadlock on the same lock. Also fix a multiple-unlock bug.

Handle BYE response

Fix idle dwell time

When switching from sleep() to g_usleep() in commit 25b19908, we should
also have converted its argument into µs.

Remove superfluous debug printf

Use LIST-EXTENDED to return subscribed status from LIST

If the server supports LIST-EXTENDED, we don't have to follow the initial
LIST with an LSUB command to see which folders are subscribed -- we can
just ask it to tell us the subscribed status in the LIST responses.

Detect LIST-STATUS and LIST-EXTENDED capabilities

Don't show subscribed but non-existent folders.

The RFCs are clear that it's OK to have non-existent folders in the
subscribed list. Don't treat them as if they were real folders -- just
use their existence in the LSUB response to set the SUBSCRIBED flag if they
were already returned by the LIST response.

Really select INBOX for folder deletion and rename

The folder selection code is a bit screwed -- a string comparison with
ic->select is used to determine which folder a job can run on, but
ic->job->folder is the one that's actually passed to imapx_select().

Fixing that is probably going to involve turning ic->select into a
folder pointer instead of a string -- so these callers might as well
start getting used to the fact that they have to look the INBOX up using
camel_store_get_folder(). I'll come back and actually fix the
imapx_command_start() code later, when there's nothing more important
offending me.

We really do need _not_ to be SELECTed on a renamed or deleted folder, or
entering IDLE will cause some servers (dovecot) to kick us off.

Remove courier-imap rename workaround

This is breaking things on non-buggy servers, because after renaming the
folder in question we also attempt to rename all its subfolders... and
quite reasonably get a failure.

Testing against current versions of Courier (4.4.0) seems to suggest
that it's not actually necessary, either -- perhaps the bug has been
fixed in the 9 years since this hack was first added to the imap back
end in commit bfce883e.

use gtk-3.0 if available

Updated Hebrew translation.

Bug 622392 - Build failure due to missing nspr includes

camel.pc.in already defines @mozilla_nss@ and @mozilla_nspr@ in its
Requires list, but that only works for distros that ship their own nss
and nspr pkg-config files.

Debian, for example, doesn't (but it should).  So @mozilla_nss@ and
@mozilla_nspr@ are never defined.  Instead, the configure script falls
back to the older "manual" settings and defines MANUAL_NSS_CFLAGS and
MANUAL_NSS_LIBS.  So list those as well in camel.pc.in.  They will be
mutually exclusive with the pkg-config variables.

And please, for the love of free software, somebody push pkg-config
files to upstream nss/nspr so we can drop this gunk from configure.
It's 2010 already for cryin' out loud...

Bug 622799 - Fails to link libedataserverui-1.2.la

Bug #622710 - Crash in imapx_idle_supported due to cinfo being NULL

For the sake of portability: sleep -> g_usleep

Bug #620815 - Memory leaks with Evolution

Bug 622008 - Don't trust is->select_folder in imapx_job_scan_changes_done()

We may have already issued a new SELECT command by the time the scan_changes
completion function is called, and is->select_folder may be NULL already.

Disable the special-case for the selected folder in refresh_info

It will make us miss messages on servers like Groupwise which don't give
unsolicited change notices; we really _do_ need to issue STATUS there,
even though it's explicitly forbidden by RFC3501 do so. Or SEARCH UNSEEN.

But the logic is broken anyway -- what is the currently-selected folder NOW
may not be the currently-selected folder by the time our STATUS command hits
the front of the queue anyway, so we do still end up doing STATUS on the
current folder sometimes. Needs more thought.

Revamp imapx_job_refresh_info_start() to make use of modseq and uidnext

Always enable QRESYNC where it's available.

This doesn't have much of an effect until we start using it, except for
enabling the VANISHED response, and causing unsolicited fetches to have UIDs.

Update summary to include uidnext and modseq, and 64-bit uidvalidity

Also set the new fields in imapx_job_scan_changes_done() -- although
the summary's uidvalidity still isn't being set anywhere.

Handle UIDNEXT and MODSEQ/HIGHESTMODSEQ information from server.

UIDNEXT is useful for detecting when new messages have been added to a
folder, and (for servers which support it) MODSEQ is even more useful because
it lets us detect flags changes.

Make uidvalidity field 64-bit

But not in the summary yet; I'll get to that shortly and we aren't even
storing it there anyway right now -- we're always writing zero.

Recalculate unread_on_server count after fetching flags.

Since we're not allowed to issue STATUS for the currently-selected mailbox,
the only way to ask the server for the count of unseen messages is to select
some _other_ mailbox and then issue STATUS. Even re-SELECTing the mailbox
doesn't give it, although a 'SEARCH UNSEEN' would do it, if we count the
responses.

But we _can_ work it out from the flags after we fetch them all.

Decrement ifolder->exists_on_server when handling EXPUNGE/VANISHED

We're supposed to keep track of this for ourselves; the server doesn't send
a new EXISTS.

Permit all characters but dir_sep in imapx folder names.

There's no point in preventing people from creating folders with %#* in the
names; they can always create them from elsewhere. Or, AFAICT, just create
them using evo with a 'permitted' name and then rename them to contain the
'offending' characters.

Simplify relationship between full and path names for imapx

There seems to be no point in the %25-style escapes, which we were applying
in full_to_path() only if dir_sep != '/', and then reversing in path_to_full()
unconditionally. Instead, just swap dir_sep with '/'. Aside from simplifying
the code, this also means that the path displayed to the user will be as close
as possible to the real name -- it means that my "be nasty to imapx" test
folder called fish"%sd is displayed properly, instead of as fish"%25sd.

Queue non-folder specific jobs while waiting for SELECT to complete

Remove this TODO from the list...

Propagate STATUS responses to folders directly from imapx_untagged()

Storing this in the server-global variables like is->exists is broken;
any SELECT command which is sent between the untagged STATUS response
and the tagged completion of the command is going to overwrite the data.
Not to mention the fact that LIST-STATUS and NOTIFY will be totally hosed,
when we get more than one untagged STATUS response between tagged commands.

Update ifolder->exists_on_server from STATUS response

Make camel_imapx_stream_number() return a guint64

Some fields like UIDVALIDITY and MODSEQ may well be larger than can fit in
a 32-bit value. RFC4551 makes this abundantly clear in its examples.

Allow debugging to be enabled with CAMEL_DEBUG environment variable

CAMEL_DEBUG=imapx will enable all debugging, or 'imapx:command',
'imapx:token' etc. will enable certain types.

Not entirely sure if 'debug' and 'extra' are the correct names for the
old 'd()' and 'e()' debug output...

Fix overzealous IDLE handling

We were sometimes entering the IDLE state even during a multi-part message
fetch, between one part and the other. This happened because we call
imapx_command_start_next() which sees an empty queue and triggers IDLE,
before we called the completion handler for the previous command which
puts a new FETCH request into the queue.

Similar behaviour was seen in various other situations, including between
subsequent sync_message() calls from the front end.

Fix this by having a dwell time of 2 seconds between the queue becoming
empty and actually sending the IDLE command. Only if the queue _remains_
empty for 2 seconds do we really enter the IDLE state.

Clean up the IDLE handling to use a state machine instead of a set of
boolean flags, while we're at it.

Improve imapx command queue processing

We were breaking out of the loop which looked for commands to start on
the currently-selected folder, as soon as we reached any command which
was for any other folder.

This meant that we'd only ever run the _first_ command from the queue,
even if it was for another folder and there were other were other commands
with equal priority waiting to run on the _current_ folder. This leads to
a bunch of gratuitous ping-ponging between folders with SELECT, without
actually running any real commands between two consecutive SELECTs.

Silence compiler warning in imapx_job_matches()

It's a false positive, but it's annoying:
camel-imapx-server.c: In function ‘imapx_job_matches’:
camel-imapx-server.c:1004: warning: ‘full_name’ may be used uninitialized in this function

Remove redundant 'unseen' member of CamelIMAPXServer.

This was added in commit ed158d12 ("Fetch new messages first") seemingly by
mistake -- there was already an 'unread' field.

Handle [CLOSED] status (RFC5162).

This indicates precisely when the selected mailbox changes to the new
mailbox during the SELECT command, before its tagged completion.

This allows us to assign unsolicited FETCH responses to the correct
mailbox; with QRESYNC those will arrive before the SELECT completion.

It also theoretically allows us to pipeline SELECT requests, although
there's not a huge amount of point in that and we'll have other issues
with our implementation if we do that.

Cope with unsolicited FETCH having both FLAGS and UID.

This'll happen once we're using CONDSTORE or QRESYNC

Support VANISHED response (RFC5162)

Extract EXPUNGE handling into imapx_expunge_uid_from_summary() function.

We'll want to use the same function for handling VANISHED.

Fix imapx_parse_flags to not just return the first UID repeatedly, export it.

Fix batch fetch to use UID FETCH in imapx_job_fetch_new_messages_start()

Add LITERALPLUS support.

In imapx_command_start(), just keep calling imapx_continuation() to send
the continuations immediately.

For this, we have to pass the CamelIMAPXServer pointer in to
camel_imapx_command_new() so that it can see the server's capabilities.

Fix division-by-zero in imapx message fetch

Use ifolder->exists_on_server, not imap->exists (which could have come
from a STATUS command and be the value for some other folder).

And it's possible to get here when we thought there were no messages, so
add ?:1 to avoid the division by zero in that case too.

Don't bother checking for is->cinfo being NULL. It can never happen now.

Famous last words, I know...

Add IMAPX_INITIALISED state; don't let users run before CAPABILITY/NAMESPACE

Previously, other users would blindly carry on as soon as we reached the
IMAPX_AUTHENTICATED state... and then occasionally fail because ->capa
wasn't set, etc.

Encode folder names as literals when sending commands, where needed

Accept literals as folder names in STATUS

NUL-terminate literal strings when tokenising

Re-fetch capabilities after STARTTLS. There might be new auth methods available

Accept unsolicited CAPABILITY report as part of LOGIN/AUTHENTICATE response

Also fix up freeing of cinfo status response.

Fix handling of unsolicited [CAPABILITY xxx] report in server greeting.

Fix up imapx_untagged() and imap_parse_status() to handle capabilities,
and use imapx_untagged() to read the server greeting.

Unify the greeting handling for the process and socket connections, and
don't re-fetch the capabilities immediately if we have them.

Handle incoming lines of noise before the greeting.

Detect CONDSTORE and QRESYNC capabilities

I have plans for these...

Remove unused variable ‘message’

camel-offline-folder.c: In function ‘offline_downsync_sync’:
camel-offline-folder.c:63: warning: unused variable ‘message’

Use camel_folder_sync_message() to sync messages.

The imapx back end will handle sync_message requests at a much lower
priority than normal get_message requests, as is right and proper.

It would help if the front end would call the correct method, so that
the prioritisation actually works as expected. Otherwise, sync_message
requests end up having higher priority than the requests which are updating
flags and headers for changed folders -- leading to a crappy user experience.

I caused a build break, shame on me.

camel_folder_append_message: 'info' can be NULL.

Bug 316783 - Broken dragging of email addresses

Clarify documentation for e_destination_get_address().

Revert "Bug 619347 - Return formatted address in e_destination_get_address"

This reverts commit db8c986c4f06f0cb57de9bfa6982b4e78f0bb32a.

Despite the misleading documentation, e_destination_get_address() -is-
intended to return an encoded address suitable for use in mail headers.
e_destination_get_textrep() exists for displaying destination addresses
to the user.  The Contact List Editor is apparently calling the wrong
function.

Bug 621851 - CamelFolder description should fall back to full-name

Bug 621961 - Remove GroupWise debugging messages.

Bug 601535 - Categories not translated in menus

s/EVOLUTION_LOCALEDIR/LOCALEDIR/

Remove the generated file camel-imapx-tokenise.h from git.

Do not set exception if its already set

Bug #621877 - Crash in summary_header_to_db at camel-vee-folder.c

Bug 621842 - use 'UID FETCH' instead of 'FETCH' for scan_changes

RFC3501 requires that 'UID FETCH' for non-existent message sets should
return an empty set of data and not complain. Unlike 'FETCH'.

Fix locking issues in cancel_all_jobs()

While debugging bug 621740, the race conditions in here were suspected.
On balance they probably weren't the cause, but fix them anyway for good
measure.

Fix tagprefix for multiple accounts.

It gets very confusing looking at debug logs when all four imapx accounts
are using 'A' as their tagprefix.

Bug 621740 - Fix double-free of is->literal in cancel_all_jobs()

The command which is is->literal is either on the queue or owned somewhere
else; there's no need for it to be freed in cancel_all_jobs() too.

Bug 621740 - Fix double command ->complete() in imapx_command_start_next()

If imapx_command_start() returns FALSE, it'll already have called the
completion function for the command in question. Don't do it again; it's the
second of the _three_ places that command was getting freed.

Bug #617723 - Critical warnings when going offline in contact view

Bug #206061 - Allow normal, non-vFolder, Trash and Junk folder

Bug 610919 - Remove superfluous space in flags in APPEND command

RFC3501 doesn't permit a space between the final flag and the closing
parenthesis, and Exchange will reject the command if it's there.

Set exception correctly while appending message - missed in previous commit

Part of bug 610919, to display the exceptions to the user. We still need to enable displaying exceptions asyncronously

Bug 615239 - Fix GSSAPI (and other SASL) authentication for imapx

Bug 599879 - Fix NULL return without exception on krb5-auth-dialog failure

Bug 585301 - Use ~/.pki/nssdb for NSS SQL database, not Evolution directory.

This is where the user-specific part of the 'shared system database' goes,
and is what Chrome uses. When the system _isn't_ configured to use the
shared system database, this is what we should be using.

Updated Galician translations

Bug 585301 - Use correct NSS system database where appropriate.

It sucks that we have to look in the /etc/pki/nssdb/pkcs11.txt file to work
out whether the system database is being used, but it's the only way to tell.

Bug 621262 - Fix fetch of large mails from broken servers like Exchange

Bug 621237 - Add imap-over-custom-command support for imapx

Check EBook DBus error names without underscores too

As shown in Gnome bug #621232, newer DBus can strip underscores
in the names (see comment #0 there). I guess it does newer DBus.

fixed an error in previous commit

use gdk_drag_context_get_action instead of gdk_drag_context_get_selected_action for gtk version 2.21.1