Re: segfaults with cyrus-imapd 3.0.9 on latest arch linux

2019-03-20 Thread Patrick Goetz
that manpages are actually useful to have around). These are only needed at buildtime and need not be present on the actual server system. On 3/19/19 5:37 PM, Andreas Piesk wrote: Am 19.03.19 um 22:00 schrieb Patrick Goetz: Have you tried the 3.0.9 AUR package?    https://aur.archlinux.org

Re: segfaults with cyrus-imapd 3.0.9 on latest arch linux

2019-03-20 Thread Patrick Goetz
ration option you can't actually use. On 3/19/19 5:37 PM, Andreas Piesk wrote: Am 19.03.19 um 22:00 schrieb Patrick Goetz: Have you tried the 3.0.9 AUR package?    https://aur.archlinux.org/packages/cyrus-imapd Once you get the dependencies down, this one compiles and runs. I n

cyrus-imapd build dependencies

2019-03-18 Thread Patrick Goetz
This page on compiling cyrus-imapd: https://www.cyrusimap.org/imap/developer/compiling.html shows a number of build dependencies; however I was just able to compile cyrus-imapd without these installed: gperf libbsd Are these actually necessary? Later in the page, under "Alternate

Re: Unifying the Cyrus World

2015-12-27 Thread Patrick Goetz via Cyrus-devel
Was this issue ever resolved? I'm still finding the documentation split between cyrus.foundation and cyrusimap.org, and it's incomplete in both places. On 09/16/2015 08:50 PM, Nicola Nye wrote: Dear Cyrus folks, After nominally completing the migration of the docs from the wiki at

virtdomains

2015-09-25 Thread Patrick Goetz
It seems that everyone is using virdomains now, and I need to start doing this soon myself. 2 questions: 1. Can I use virtdomain accounts alongside the existing non-domain-specific accounts; or more precisely 2. Can I just add the virtdomain functionality to my existing install without

Re: 2.5.3: Inconsistency in subscription list separator?

2015-09-25 Thread Patrick Goetz
ids writing those), but may contain other confidential user data - for example the content of emails - so check them over before sharing publicly! Cheers, Bron. On Fri, Sep 25, 2015, at 11:21, Patrick Goetz wrote: Hi - I see how to enable telemetry, but I'm clueless as to how to collect the du

2.5.3: Inconsistency in subscription list separator?

2015-09-24 Thread Patrick Goetz
I posted a question about this to the info list, but think it might be more of a devel issue. I recently upgraded from 2.4.17 to 2.5.3. This went more of less smoothly; however, one user appears to have lost access to some of his mail sub-folders. They're grayed out in Thunderbird and don't

Re: 2.5.3: Inconsistency in subscription list separator?

2015-09-24 Thread Patrick Goetz
olders and messages are definitely all there. On 09/24/2015 05:11 AM, Bron Gondwana wrote: On Thu, Sep 24, 2015, at 19:54, Patrick Goetz wrote: I posted a question about this to the info list, but think it might be more of a devel issue. Yeah, I saw it there and almost replied, but I got busy wi

Re: 2.5.3: Inconsistency in subscription list separator?

2015-09-24 Thread Patrick Goetz
hunderbird connecting and getting the folder listing and trying to access On Thu, Sep 24, 2015, at 23:08, Patrick Goetz wrote: Hi Bron - Just a quick clarification: - The folder exists - The subscription exists - The ACLS on the folder correspond to full permissions for the user:

2.4.17 --> 2.5.3

2015-09-19 Thread Patrick Goetz
I recall Bron telling us that the upgrade from 2.4.x to 2.5.x would be completely painless. That was mostly, but not completely true. A bunch of variable names in /etc/cyrus/imapd.conf changed (OK, that was easy to fix), and the upgrade did mostly work out of the box. There was one issue,

Possible TLS dir option name discrepancy?

2015-01-12 Thread Patrick Goetz
The 2.5 documentation here (http://www.cyrusimap.org/~vanmeeuwen/imap/release-notes/2.5.0.html) states that some of the TLS options will change in 2.5, namely tls_client_ca_dir (was: tls_ca_dir) However, there is no tls_ca_dir option given here

usenet

2014-12-05 Thread Patrick Goetz
excerpted from my /etc/cyrus/cyrus.conf file: # these are only necessary if receiving/exporting usenet via NNTP # nntp cmd=nntpd listen=nntp prefork=0 # nntpscmd=nntpd -s listen=nntps prefork=0 I wonder if it's finally time to trim all the usenet/news cruft out of

backing up a cyrus mail store?

2014-01-30 Thread Patrick Goetz
Hi - I'm sure I've asked about this before, but is there an approved/canonical way to backup a cyrus mailstore? It's not clear from the documentation what cyrdump does (if anything) and short of stopping the server in order to backup the mailboxes and /var/lib/cyrus with guaranteed

Re: Coding standards / spaces vs tabs

2010-10-21 Thread Patrick Goetz
expandtab -- Patrick Goetz

Re: Coding standards / spaces vs tabs

2010-10-21 Thread Patrick Goetz
and needs to be rewritten from scratch using modern programming paradigms. I know talking about indentation is like starting a vi vs. emacs -- which is better? discussion, but you asked. -- Patrick Goetz

Re: Reconstruct behaviour on IO error

2010-10-04 Thread Patrick Goetz
On 10/04/2010 02:07 PM, Bron Gondwana wrote: Keep going and remove entires for messages with IO errors reading their file, or bail and make the administrator fix the underlying permissions first? I can see arguments for both - though remember that once you've removed the index record you can

Re: Wiki, information on database back ends.

2010-09-14 Thread Patrick Goetz
On 09/14/2010 04:06 PM, Jeroen van Meeuwen (Kolab Systems) wrote: This file is also written out by the same script. The location of the script (the mentioning of 'rpm/' in the path) is completely arbitrary (for all the script cares the file lives in /boot/). Small correction: created while

Re: master process handling patch

2010-07-22 Thread Patrick Goetz
On 07/21/2010 05:12 PM, Bron Gondwana wrote: We decided to pull map_stupidshared. Are you on the cyrus-devel mailing list? Yeah, but I've only been on it for a couple of months -- maybe this was discussed previously. Trying to evaluate C code that consists of lots of little functions

Re: master process handling patch

2010-07-21 Thread Patrick Goetz
On 07/15/2010 05:23 PM, Henrique de Moraes Holschuh wrote: Heh, you can already use whatever we have in the experimental branch, it is good enough for limited use :) This isn't entirely true. This patch, which I mentioned previously, looks like it could lead to header corruption in the

Re: master process handling patch

2010-07-21 Thread Patrick Goetz
On 07/21/2010 02:35 PM, Wesley Craig wrote: On 21 Jul 2010, at 15:27, Patrick Goetz wrote: Meanwhile, I'm going over the patches the redhat people added to cyrus-imapd-2.3.16-5.src (actually, first comparing the differences between this newer version and cyrus-imapd-2.3.16-3.fc13.src) to see

Re: master process handling patch

2010-07-09 Thread Patrick Goetz
On 07/02/2010 10:11 PM, Henrique de Moraes Holschuh wrote: +int read_msg(int fd, struct notify_message *msg) +{ +ssize_t r; +size_t off = 0; +size_t s = sizeof(struct notify_message); No. As I said in the debian cyrus devel list, either ssize_t s =, or leave it as

master process handling patch

2010-07-02 Thread Patrick Goetz
This is another debian patch with lots of changes (see attached). I'm running out of time for tracing the validity of these and am hoping that someone else will take a look. This seems to apply to a rather important part of the whole system, so is probably worth a look. (Yes, I can submit this

Fwd: RE: {Spam?} master process handling patch (verification)

2010-07-02 Thread Patrick Goetz
Hopefully it's clear to (almost) everyone that this is the wrong way to solve the problem, inconveniencing legitimate users and creating a lot of backscatter otherwise. Original Message Subject:RE: {Spam?} master process handling patch (verification) Date: Fri, 2

Re: ~/lib/map_stupidshared.c

2010-06-30 Thread Patrick Goetz
On 06/30/2010 11:10 AM, Wesley Craig wrote: I don't think the people reading the developer list can necessarily answer that question. The existence of a patch (depending on what the patch is about) implies that someone cared enough about it to patch it. Asking on the user list might turn up an

Re: Putting cyrus-future code in CVS. Also, code style

2010-06-29 Thread Patrick Goetz
Bron Gondwana wrote: How do you feel about: if (a) function_one(a); else function_two(); This is hard to read. My rule is one liner only if it's really a one liner. Otherwise: if (a) { function_one(a); } else {function_two();} is a reasonable compromise between brevity and

minor bug in ~/tools/masssievec

2010-06-18 Thread Patrick Goetz
Unused variable in this script (can be dangerous in perl) -- see attached. #! /bin/sh /usr/share/dpatch/dpatch-run ## 35_masssievec_remove_unused_variable.dpatch by Sven Mueller deb...@incase.de ## ## All lines beginning with `## DP:' are a description of the patch. ## DP: Remove the

imapparse.c / getnum

2010-06-17 Thread Patrick Goetz
On 06/16/2010 10:09 PM, Bron Gondwana wrote: I'm also replacing everywhere that we advance a character pointer through a string and do the same algorithm with parsenum in lib/util.c. It will have the same logic. No more MAXLITERAL. The getnum code looks basically the same as the bit that

debian patches to Shell.pm and sieveshell.pl

2010-06-16 Thread Patrick Goetz
This time I'm just attaching a patch file (unified diff format) for reference -- hopefully it's relatively clear how to read it. I've never used Shell.pm, but I do know perl. With the exception of the first patch (Shell.pm, line 127), which I don't understand, these all seem fairly

Re: debian patches to Shell.pm and sieveshell.pl

2010-06-16 Thread Patrick Goetz
Michael Loftis wrote: Just documents the ability to set an (arbitrary) annotation with a specific value on a mailbox. That totally lost me -- any documentation available to look at? Or where in the code is this functionality realized?