1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
|
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 3.2//EN">
<HTML>
<HEAD>
<TITLE> [Mageia-dev] cyrus-imapd
</TITLE>
<LINK REL="Index" HREF="index.html" >
<LINK REL="made" HREF="mailto:mageia-dev%40mageia.org?Subject=Re%3A%20%5BMageia-dev%5D%20cyrus-imapd&In-Reply-To=%3C201302270759.03037.thomas%40btspuhler.com%3E">
<META NAME="robots" CONTENT="index,nofollow">
<META http-equiv="Content-Type" content="text/html; charset=us-ascii">
<LINK REL="Previous" HREF="023147.html">
<LINK REL="Next" HREF="023157.html">
</HEAD>
<BODY BGCOLOR="#ffffff">
<H1>[Mageia-dev] cyrus-imapd</H1>
<B>Thomas Spuhler</B>
<A HREF="mailto:mageia-dev%40mageia.org?Subject=Re%3A%20%5BMageia-dev%5D%20cyrus-imapd&In-Reply-To=%3C201302270759.03037.thomas%40btspuhler.com%3E"
TITLE="[Mageia-dev] cyrus-imapd">thomas at btspuhler.com
</A><BR>
<I>Wed Feb 27 15:58:56 CET 2013</I>
<P><UL>
<LI>Previous message: <A HREF="023147.html">[Mageia-dev] cyrus-imapd
</A></li>
<LI>Next message: <A HREF="023157.html">[Mageia-dev] cyrus-imapd
</A></li>
<LI> <B>Messages sorted by:</B>
<a href="date.html#23156">[ date ]</a>
<a href="thread.html#23156">[ thread ]</a>
<a href="subject.html#23156">[ subject ]</a>
<a href="author.html#23156">[ author ]</a>
</LI>
</UL>
<HR>
<!--beginarticle-->
<PRE>
On Tuesday, February 26, 2013 11:38:57 AM Colin Guthrie wrote:
><i> Hiya Thomas,
</I>><i>
</I>><i> Firstly, thanks for taking time to update cyrus-imapd a while back. It's
</I>><i> a horrible package with lots of "gotchas".
</I>><i>
</I>><i> Sadly, I've just recently been bitten by fallout due to some of the many
</I>><i> dropped patches when you updated the package to the 2.4 series in this
</I>><i> commit:
</I>><i> <A HREF="http://svnweb.mageia.org/packages/cauldron/cyrus-imapd/current/SPECS/cyrus-">http://svnweb.mageia.org/packages/cauldron/cyrus-imapd/current/SPECS/cyrus-</A>
</I>><i> imapd.spec?r1=130115&r2=196262
</I>
><i>
</I>><i>
</I>><i> What the commit message said was:
</I>><i> "upgrade to 2.4.13"
</I>
Colin thanks for the fixes. I assumed most of the problems were solved with the update.
I commented them out first ( so they could easily be re-added) and would have expected some bug
reports if they would cause a problem. Since none were reported, I removed them later.
><i> Unfortunately, this is really no where near enough information for the,
</I>><i> very brutal changes that were introduced in that commit.
</I>><i>
</I>><i> 14 patches were disabled! That's a lot to go unmentioned!
</I>><i>
</I>><i>
</I>><i> In my case some of the patches were very important - namely the
</I>><i> autocreate and autosieve patches which are an important part of my
</I>><i> setup. I don't add users very often but today I spent a long time trying
</I>><i> to debug why my new user's mailbox didn't "just work" as it always has
</I>><i> in the past.
</I>><i>
</I>><i>
</I>><i> As I investigated the issue I saw this "cleanup" patch:
</I>><i> <A HREF="http://svnweb.mageia.org/packages/cauldron/cyrus-imapd/current/SPECS/cyrus-">http://svnweb.mageia.org/packages/cauldron/cyrus-imapd/current/SPECS/cyrus-</A>
</I>><i> imapd.spec?r1=259043&r2=259168
</I>><i>
</I>><i>
</I>><i> Sadly this just compounded the above problem, and all the patch files
</I>><i> themselves still remained in the SOURCES folder.
</I>><i>
</I>><i> As this is needed functionality for my setup (and likely other users
</I>><i> too), I've taken the time today to investigate all the dropped patches
</I>><i> and find replacements/updates where appropriate and drop those ones
</I>><i> committed (or which have alternative fixes) upstream.
</I>><i>
</I>><i> Please can you take a look over the changes:
</I>><i> <A HREF="http://svnweb.mageia.org/packages?view=revision&revision=400402">http://svnweb.mageia.org/packages?view=revision&revision=400402</A>
</I>><i> <A HREF="http://svnweb.mageia.org/packages?view=revision&revision=400404">http://svnweb.mageia.org/packages?view=revision&revision=400404</A>
</I>><i>
</I>><i>
</I>><i> As you'll see it took a little time and I documented all the changes as
</I>><i> clearly as I could (I would have done it in individual commits but that
</I>><i> isn't really easy with subversion - no ability to stage changes and I
</I>><i> was too lazy to use git-svn here).
</I>><i>
</I>><i> The rmquota patch is still disabled, but only two others are slightly
</I>><i> concerning now. The rest either still applied or had appropriate
</I>><i> upstream fixes.
</I>><i>
</I>><i>
</I>><i>
</I>><i> Finally, I notice you made this change:
</I>><i> <A HREF="http://svnweb.mageia.org/packages/cauldron/cyrus-imapd/current/SPECS/cyrus-">http://svnweb.mageia.org/packages/cauldron/cyrus-imapd/current/SPECS/cyrus-</A>
</I>><i> imapd.spec?r1=389214&r2=394129
</I>
This was recommended <A HREF="http://www.mail-archive.com/info-cyrus@lists.andrew.cmu.edu/msg38696.html">http://www.mail-archive.com/info-cyrus@lists.andrew.cmu.edu/msg38696.html</A>
><i>
</I>><i> In theory, this is what one of the patches you disabled should deal
</I>><i> with. It silenced the user_deny.db verbosity. Sadly when rediffing the
</I>><i> patch it seems the latest version has refactored that code. I cannot
</I>><i> find anywhere where this message would be printed now (I see the error
</I>><i> on mga2 tho').
</I>><i>
</I>><i> Can you confirm if this change is really needed or if it was added after
</I>><i> looking at the mga2 package rather than the current code?
</I>><i>
</I>><i> Cheers
</I>><i>
</I>><i> Col
</I>
--
Best regards
Thomas Spuhler
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part.
URL: </pipermail/mageia-dev/attachments/20130227/7c02848f/attachment.asc>
</PRE>
<!--endarticle-->
<HR>
<P><UL>
<!--threads-->
<LI>Previous message: <A HREF="023147.html">[Mageia-dev] cyrus-imapd
</A></li>
<LI>Next message: <A HREF="023157.html">[Mageia-dev] cyrus-imapd
</A></li>
<LI> <B>Messages sorted by:</B>
<a href="date.html#23156">[ date ]</a>
<a href="thread.html#23156">[ thread ]</a>
<a href="subject.html#23156">[ subject ]</a>
<a href="author.html#23156">[ author ]</a>
</LI>
</UL>
<hr>
<a href="https://www.mageia.org/mailman/listinfo/mageia-dev">More information about the Mageia-dev
mailing list</a><br>
</body></html>
|
