Updated Galician translations

Post-release version bump.

NEWS update for 2.31.5 release.

Bug #618938 - Crash when reading calendar acknowledgement message

Bug #623936 - 75 second delay when creating new DAV entries in Zimbra

Fix another g_propagate_error() runtime warning.

imapx: Ensure imapx_stream_fill() sets the error on zero-sized read.

We treat an empty read as an error, since we should never be here unless
we're either in the middle of reading a response, or poll() says the fd
is readable. Before commit 46938c348 (the conversion to GError), we'd set
a generic 'IO Error' exception in camel_imapx_stream_token() when
imapx_stream_fill() returned -1 to indicate an error. But now we don't
do that any more, so imapx_stream_fill() needs to reliably set the GError.

Otherwise we end up in an endless loop in imapx_step().

Fix runtime warning in camel_filter_driver_flush().

Only propagate the error if there's an error to propagate.

Improve error handling in CamelStreamBuffer::read().

If we managed to successfully read some data before camel_stream_read()
failed, discard the GError and return the number of bytes successfully
read.  If camel_stream_read() failed immediately, propagate the GError
and return -1.

imapx: camel_imapx_stream_astring() returns 0 on success.

Update API documentation.

Coding style and whitespace cleanup.

[i18n] Updated German translation

Updated Norwegian bokmål translation

Coding style and whitespace cleanup.

imapx: Avoid running FETCH_NEW_MESSAGES and REFRESH_INFO jobs simultaneously

There are various places where we interpret FETCH results and use
imapx_match_active_job to find the current job, which will behave badly
if there are two jobs which could potentially be responsible for the FETCH.

In particular, this was causing a problem when we triggered a fetch of new
messages from select_done(), and that command was submitted at the same time
as a refresh_info command to fetch all flags. The server (Dovecot) was
returning all the untagged FETCH results before either completion line,
and all the flags were getting "assigned" to the fetch_new_messages job,
causing a bunch of 'g_array_append_vals: assertion `array' failed' warnings,
and all messages to disappear because the refresh_info job didn't see them.

Updated Norwegian bokmål translation

imapx: Fix handling of return from camel_imapx_stream_text()

It's another of those functions that return 0 for success.

imapx: More improvements to server unseen count tracking

We're using the unseen count as part of our check for when things have
changed on the server (or when QRESYNC has gone wrong), and we need to
rescan all flags. But the server doesn't *tell* us about changes to the
unseen count, like it does for the total count of messages. So we have to
be more careful about tracking it -- especially since the server doesn't
even tell us in SELECT, and we can't issue STATUS to find it out when the
folder is selected.

One place that it was getting out of sync was when the server notified us
about new messages in SELECT, which weren't there when we last issued STATUS.
For messages with a UID equal to or higher than what we thought was the
folder's UIDNEXT, we also have to update the unseen count.

So far, the _only_ time we've hit the QRESYNC sanity check which triggers
a full rescan has been due to this kind of cosmetic unseen count discrepancy.
It's half tempting to disable that check in the QRESYNC sanity check, but
actually it's useful to get it right for the legacy servers too, because it
might cause us to miss updates if we get out of sync.

nntp: Fix two more 'returns FALSE on error' vs. 'returns 0 on success' errors

We really need to start being consistent about this.

Pass the GError pointer through to camel_folder_summary_save_to_db()
from nntp_folder_sync() too... although it seems to be NULL a lot of the
time anyway, but that's the fault of our caller. No need to compound it.

Fix some linker issues.

Bug 624018 - Fix error handling in groupwise_get_folder()

Updated Spanish translation

Bug #623988 - build failure in git/master.

Add the marshal list and also a typo in weather.

Update user documentation about E_CAL/BOOK_DISABLE_DEPRECATED

Bug #623204 - Be able to report detailed errors from backends

[i18n] Updated German translation

Bug 623919 - Error while opening any newly received message

Use correct format specifier for 64bit unsigned int

Do not use uninitialized variable

Use the new base directory functions.

Put an end to hard-coding "$HOME/.evolution".

Updated Spanish translation

Add functions to get Evo/EDS base directories.

Use these functions instead of hard-coding "$HOME/.evolution".  This
sets the stage for complying with the XDG Base Directory Specification.
http://standards.freedesktop.org/basedir-spec/basedir-spec-latest.html

e_get_user_cache_dir()   returns (for now)  "$HOME/.evolution/cache"
e_get_user_config_dir()  returns (for now)  "$HOME/.evolution"
e_get_user_data_dir()    returns (for now)  "$HOME/.evolution"

Bug #623795 - URI at the end of text isn't clickable

Bug #621609 - Can pass unknown UID to e_source_combo_box_set_active_uid

Migrate from CamelException to GError.

Use text/vcard content type on WebDAV PUT request

See bug #604650 for more details.

Updated Galician translations

Add a function to get the job queue info

Add signal handlers when folder selection changes and when connection shuts down

Bug 623431 - Use gweather-3.0 when building with gtk3

Bug 623430 - libedataserverui.pc.in hardcodes 1.2 API version

Update unseen count as new messages come in

When we get notified of new messages in the idle thread, or when they arrive
during our re-fetch of all flags (and we suddenly find we've got flags for a
message that didn't previously exist), ensure that we update the unseen count
according to the flags of the new messages.

This _doesn't_ handle the case where we called fetch_new_messages() directly
from refresh_info_start() because we _knew_ there were new messages, but in
fact there are _more_ new messages than we expected. We may have to switch to
a model where we say 'update flags for any UID greater than $X'.

Make NSS database initialisation more robust, handle errors better

Since commit 9116943e we only attempt to open the new SQL database, but
some users are reporting issues. Be more careful about reporting errors
when that happens, and also fall back to the old DBM database if the SQL
database fails to initialise.

Keep better track of server-side UNREAD count.

There are various circumstances in which the UNREAD count will change --
new unseen messages arriving, flags changes from the client, unseen
messages being expunged, etc. Keep track of those to improve the
accuracy of the unread count, to avoid the QRESYNC sanity check
triggering so often on minor issues that weren't the bug I was looking
for. For this, we have to keep track of whether a FETCH or VANISHED response
was solicited or unsolicited in certain cases.

Don't refetch message headers we already have.

We were issuing a headers FETCH starting from the last known UID. But we
already _had_ the headers for the last known UID. Start from $UID+1 instead.

Add sanity check after QRESYNC

If the total number of messages, unseen count and modseq don't match
what we think they should be, then fall back to refetching all flags as
we would have done without QRESYNC.

Updated Spanish translation

Fix memory leak in imapx_continuation() with SASL auth

==4945== 1 bytes in 1 blocks are definitely lost in loss record 48 of 23,077
==4945==    at 0x4A0515D: malloc (vg_replace_malloc.c:195)
==4945==    by 0x319A044AE2: g_malloc (gmem.c:132)
==4945==    by 0x319A05C3BD: g_strdup (gstrfuncs.c:102)
==4945==    by 0x6478E63: camel_sasl_challenge_base64 (camel-sasl.c:326)
==4945==    by 0x12C54E72: imapx_continuation (camel-imapx-server.c:1698)
==4945==    by 0x12C552CC: imapx_step (camel-imapx-server.c:1862)

Fix memory leak in summary_header_to_db()

==4945== 9 bytes in 1 blocks are definitely lost in loss record 1,482 of 23,077
==4945==    at 0x4A0515D: malloc (vg_replace_malloc.c:195)
==4945==    by 0x319A044AE2: g_malloc (gmem.c:132)
==4945==    by 0x319A05C3BD: g_strdup (gstrfuncs.c:102)
==4945==    by 0x6466990: summary_header_to_db (camel-folder-summary.c:296)
==4945==    by 0x12C48A2B: summary_header_to_db (camel-imapx-summary.c:289)
==4945==    by 0x6463B69: camel_folder_summary_save_to_db (camel-folder-summary.c:2316)
==4945==    by 0x12C507B2: imapx_command_sync_changes_done (camel-imapx-server.c:4227)

Store Junk and NotJunk flags in IMAP server

Strictly speaking we shouldn't be doing this if the server won't understand
them and doesn't have \* in PERMANENTFLAGS -- but there are already bugs with
that anyway, when we try to set receipt-handled. I'll get to that shortly...

Update API documentation.

Calculate can_qresync after fetching STATUS

Fix starting UID in QRESYNC SELECT.

The indices in the folder summary count from zero, not one.

Add proper QRESYNC support on SELECT; use it for refresh_info

We aren't yet correctly updating the folder's modseq every time, so we're
often asking for more than we need. That's better than asking for _less_ than
we need though -- will go over that part very carefully in separate commit(s).

We still aren't handling uidvalidity either. RSN...

Propagate SELECT error properly to dependent commands

Add support for * VANISHED (EARLIER) [uidset...]

(Actually just throw away anything in the parentheses).

We'll need this for QRESYNC SELECT

Work around sudden disappearance of GdkRegion in GTK+ 2.90.5.

API was deprecated and removed in less than two hours!  Sheesh!

post release version bump

News update for 2.31.4 release

Set server_flags after APPENDUID; don't send them to the server again.

This is just silly....

Starting command (active=1, literal) A00452 APPEND outgoing (\SEEN) {551+}
Starting command (active=1,) A00454 SELECT outgoing
Starting command (active=1,) A00453 UID STORE 14443 +FLAGS.SILENT (\SEEN)

Fix APPENDUID handling of uidvalidity.

We shouldn't be comparing with is->uidvalidity; that's just short-term
storage for use _during_ a SELECT. Compare with ifolder->uidvalidity instead.

Oh, and let's try to make sure that's actually set somewhere...

Select folder based on CamelFolder *, not its name

As threatened in commit 6900bf35807c9be0cb2498a63cef4684ff9144d6, sanify the
way that commands indicate which folder (if any) should be selected when they
run.

Until now, they've set a folder name in ic->select, which was passed as an
argument to camel_imapx_command_new(). That's been used for comparisons to
decide whether the command can run... but when it actually comes to select
a folder specifically for the benefit of the command, it wasn't used! Instead,
the folder in ic->job->folder was used instead.

Clean this up so that ic->select is now a CamelFolder * instead of a string,
and actually use it consistently for both the comparisons and the eventual
SELECT.

Lower gtk+-3.0 requirement to 2.90.4.

Since 2.90.5 isn't release yet and the release team would probably
like to assemble a 2.31.4 moduleset.

Make libedataserverui API docs build with gtk+-3.0.

Make libedataserverui test programs build with gtk+-3.0.

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.