Skip to content
This repository has been archived by the owner on Jun 10, 2024. It is now read-only.

Include the necessary ifdefs for DAV functionality #2

Open
wants to merge 988 commits into
base: master
Choose a base branch
from
Open

Include the necessary ifdefs for DAV functionality #2

wants to merge 988 commits into from

Conversation

kanarip
Copy link
Contributor

@kanarip kanarip commented Mar 3, 2014

This patch includes or excludes the necessary *DAV bits in Cyrus IMAP when compiled --with or --without httpd/dav

Greg Banks and others added 30 commits February 5, 2014 16:25
it works just fine in the old message API but we may as
well test for it.
"" as a seqset really doesn't make sense anywhere, and it broke
QRESYNC's VANISHED (EARLIER) response.
Re-build the structure in the same shape that it went in during
message_write_section()
…children"

This reverts commit f23947db227e6215a52697ad00a485a9aea83d76.

Obviously still buggy despite passing tests, joy - will try again later
This reverts commit 6cf39343f3c3d08384c62ebf532ef994b7f14252.

Just shut up the noise for now - but let's not forget it's broken!
Not only does it handle RFC822 inside an RFC822, but it's
actually reflecting the same codepath that wrote the section
in the first place.
This should not be necessary in some future release of xapian,
so we can remove it again then
brong and others added 28 commits February 5, 2014 16:26
It's not necessary, since nothing in cleanup uses it, and it's a
lock inversion.  This is one of a bunch of options here (including
keeping it locked throughout the cleanup) - but it's the least
invasive, and it's perfectly correct.
brong pushed a commit that referenced this pull request Mar 15, 2016
OpenIO containers autocreation and accounts
brong pushed a commit that referenced this pull request Jul 25, 2016
[Docs] Source repository has been moved out of Phabricator and onto GitHub
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants