[Bug 437924] New: Update cyrus to 2.3.13
https://bugzilla.novell.com/show_bug.cgi?id=437924 Summary: Update cyrus to 2.3.13 Product: openSUSE 11.0 Version: Final Platform: Other OS/Version: Linux Status: NEW Severity: Normal Priority: P5 - None Component: Other AssignedTo: bnc-team-screening@forge.provo.novell.com ReportedBy: richard.bos@xs4all.nl QAContact: qa@suse.de Found By: Community User Cyrus-2.3.13 has been released. Please update cyrus to provide the improvement mentioned below (from the file doc/changes.html): Changes to the Cyrus IMAP Server since 2.3.12 • Added an experimental "sql" backend for cyrusdb. Currently MySQL, PostgreSQL, and SQLite are supported. See the new sql_* options for details. • Added support for IMAP [CAPABILITY] response code to client-side of Murder proxies. • Added support for ManageSieve auto-capability response after STARTTLS and after AUTH with a SASL security layer. • Made MAXWORD and MAXQUOTED sizes configurable via imapd.conf -- courtesy of Fastmail.fm. • Rewrote cyrusdb_quotalegacy.c to use readir() rather than glob.c. This avoids a potential crash due to conflicts between glibc and Heimdal implementations of glob(). • Added support for fulldirhash to ctl_mboxlist -v. • More skiplist transaction fixes -- courtesy of Fastmail.fm. • cyr_expire no longer has a default of 0 (zero) for -X and -D. These options must be used explicitly in order to have the desired effect. • Added sieve_utf8fileinto option. • Added sieve_sasl_send_unsolicited_capability and sieve_sasl_expect_unsolicited_capability options. • Several 32/64-bit compatibility fixes. • Fixed miscellaneous bugs and build issues. Changes to the Cyrus IMAP Server since 2.3.11 • Added statuscache.db to cache IMAP STATUS data which significantly reduces the amount of I/O necessary when neither the mailbox nor \Seen state has changed -- courtesy of Fastmail.fm. • Added option to unexpunge to restore messages by time interval -- courtesy of David Carter. • Implemented undocumented IMAP SCAN extension, which allows Pine/Alpine to do cross-mailbox searches -- based on work of David Carter. • Implemented incremental squat updates (see squatter.8) -- courtesy of David Carter. • Fixed major bugs in reconstruct -k implementation -- courtesy of David Carter. • Added code to promote transaction safety and to track the lock status of skiplist databases -- courtesy of Fastmail.fm. • Added option to mbexamine to compare quota usage in cyrus.index to the actual message file sizes. • Allow long lines in imapd.conf to be split across multiple lines by using backslash as a continuation character. • The Sieve compiler now translates UTF-8 mailbox names used in fileinto to modified UTF-7 so that messages will be successfully inserted into the mailbox. • Use a more appropriate argument with LHLO command. • Added serverinfo option to control the information displayed in banner greetings and capability responses. • Fixed miscellaneous bugs and build issues. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=437924 Cristian Rodríguez <crrodriguez@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |crrodriguez@novell.com AssignedTo|bnc-team-screening@forge.provo.novell.com |rhafer@novell.com -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=437924 User rhafer@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=437924#c1 Ralf Haferkamp <rhafer@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Severity|Normal |Enhancement Status|NEW |ASSIGNED Priority|P5 - None |P4 - Low --- Comment #1 from Ralf Haferkamp <rhafer@novell.com> 2008-10-23 01:49:34 MDT --- Unfortunately the 2.3.13 release (IIRC it was released on Monday) came to late for 11.1 as we are already in Versionfreeze. I plan to update to 2.3.13 with the beginnig of the 11.2 cycle. If I find some time I'll make the new version available in the buildservice (server:mail) before that. (Hint: Help with that is of course always welcome ;) ) -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=437924 User richard.bos@xs4all.nl added comment https://bugzilla.novell.com/show_bug.cgi?id=437924#c2 --- Comment #2 from Richard Bos <richard.bos@xs4all.nl> 2008-11-15 15:14:28 MST ---
(Hint: Help with that is of course always welcome ;)
Please, find the new version at: http://download.opensuse.org/repositories/server:/Kolab:/UNSTABLE/openSUSE_1... The build service url is: https://build.opensuse.org/package/show?package=cyrus-imapd&project=server%3AKolab%3AUNSTABLE The BS log shows some rpmlint warnings, I'll list them below. I compared the warnings with openSUSE:Factory. The latter shows less warnings, although the specfile is almost the same... Is this patch cyrus-imapd-2.3.1x-autocreate-0.10-0.diff still needed? If the patch is not accepted upstream, it makes opensuse's cyrus version, almost a fork of cyrus. Is this really wanted, is it maintainable? It makes it so much harder to support! The rpmlint warnings I got: /var/tmp/rpm-tmp.4796: line 2: /usr/sbin/usermod: No such file or directory cyrus-imapd-2.3.13-60.1 warning: user cyrus does not exist - using root warning: user cyrus does not exist - using root RB: I think that those are due to BS, perhaps some base package is installed by default??? E: cyrus-imapd no-return-in-nonvoid-function auth_krb.c:388 RB: add NULL to the function I guess. Is it possible to those fixed upstream? (if not, why not?) cyrus-imapd.i586: W: description-shorter-than-summary cyrus-imapd.src: W: description-shorter-than-summary The package description is shorter than the summary. be a bit more verbose, please. RB: perhaps the description should be changed to: Cyrus is a highly scalable enterprise mail system designed for use in enterprise environments of various sizes using standards based technologies. Cyrus technologies scale from independent use in email departments to a system centrally managed in a large enterprise. cyrus-imapd.i586: W: files-duplicate /etc/pam.d/sieve /etc/pam.d/pop:/etc/pam.d/imap RB: I guess the %fdupes macro should be used cyrus-imapd.i586: W: incoherent-init-script-name cyrus The init script name should be the same as the package name in lower case, or one with 'd' appended if it invokes a process by that name. RB: change the package name to cyrus? cyrus-imapd.i586: W: non-remote_fs-dependency /etc/init.d/cyrus Your package contains a /etc/init.d script that does not specify $remote_fs as a start dependency, but the package also contains files packaged in /usr. Make sure that your start script does not call any of them, or add the missing $remote_fs dependency. Updated the init.d script? cyrus-imapd-devel.i586: W: static-library-without-debuginfo /usr/lib/libcyrus.a cyrus-imapd-devel.i586: W: static-library-without-debuginfo /usr/lib/libcyrus_min.a The static library doesn't contain any debuginfo. Binaries linking against this static library can't be properly debugged. cyrus-imapd.i586: W: unstripped-binary-or-object /usr/bin/synctest cyrus-imapd.i586: W: unstripped-binary-or-object /usr/bin/smtptest cyrus-imapd.i586: W: unstripped-binary-or-object /usr/bin/sivtest ..... No idea what to do about these warnings... Hopefully you can use the updated package, and hopefully the rpmlint warnings can be solved. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=437924 User rhafer@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=437924#c3 --- Comment #3 from Ralf Haferkamp <rhafer@novell.com> 2008-11-26 02:52:38 MST --- (In reply to comment #2 from Richard Bos)
(Hint: Help with that is of course always welcome ;)
Please, find the new version at: http://download.opensuse.org/repositories/server:/Kolab:/UNSTABLE/openSUSE_1... The build service url is: https://build.opensuse.org/package/show?package=cyrus-imapd&project=server%3AKolab%3AUNSTABLE
Cool, thanks.
The BS log shows some rpmlint warnings, I'll list them below. I compared the warnings with openSUSE:Factory. The latter shows less warnings, although the specfile is almost the same...
Is this patch cyrus-imapd-2.3.1x-autocreate-0.10-0.diff still needed? If the patch is not accepted upstream, it makes opensuse's cyrus version, almost a fork of cyrus. Is this really wanted, is it maintainable? It makes it so much harder to support! Yes, currently it is needed. AFAIK the configuration created by the mail-server YaST module makes use of it. The patch is BTW maintained here: http://email.uoa.gr/projects/cyrus/autocreate/ Uptodate Version of the patch for recent cyrus-imapd release can be found here: http://email.uoa.gr/download/cyrus/cyrus-imapd-2.3.13/
The rpmlint warnings I got: /var/tmp/rpm-tmp.4796: line 2: /usr/sbin/usermod: No such file or directory cyrus-imapd-2.3.13-60.1 warning: user cyrus does not exist - using root warning: user cyrus does not exist - using root
RB: I think that those are due to BS, perhaps some base package is installed by default??? I guess this is because, usermod (from pwdutils) ist not installed while build
That patch is BTW included into the cyrus-imapd package by other distributions as well. (e.g. Fedora/RedHat). I agree though that it make maintaining the cyrus-imapd package somewhat harder. the package, havn't looked into this yet, Probalby just a missing BuildRequires.
E: cyrus-imapd no-return-in-nonvoid-function auth_krb.c:388
RB: add NULL to the function I guess. Is it possible to those fixed upstream? (if not, why not?) IIRC, I submitted a patch upstream sometime ago. It should be in their bugzilla. I'll check again.
cyrus-imapd.i586: W: description-shorter-than-summary cyrus-imapd.src: W: description-shorter-than-summary The package description is shorter than the summary. be a bit more verbose, please.
RB: perhaps the description should be changed to: Cyrus is a highly scalable enterprise mail system designed for use in enterprise environments of various sizes using standards based technologies. Cyrus technologies scale from independent use in email departments to a system centrally managed in a large enterprise. Thanks for the text. I'll add it to our package.
cyrus-imapd.i586: W: files-duplicate /etc/pam.d/sieve /etc/pam.d/pop:/etc/pam.d/imap
RB: I guess the %fdupes macro should be used
cyrus-imapd.i586: W: incoherent-init-script-name cyrus The init script name should be the same as the package name in lower case, or one with 'd' appended if it invokes a process by that name.
RB: change the package name to cyrus? I'd rather leave it the way it is. Changeing the package name or the man of the init script if alway a hassle. If not for the packager, then for the users. After all this is just a warning.
cyrus-imapd.i586: W: non-remote_fs-dependency /etc/init.d/cyrus Your package contains a /etc/init.d script that does not specify $remote_fs as a start dependency, but the package also contains files packaged in /usr. Make sure that your start script does not call any of them, or add the missing $remote_fs dependency.
Updated the init.d script? Yes, I'll look into this.
Hopefully you can use the updated package, and hopefully the rpmlint warnings can be solved. I'll definitely have a look at it. I'll see what I can do about the warnings
[..] that are issues during build. Though I won't promise to fix them all ;) -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=437924 User richard.bos@xs4all.nl added comment https://bugzilla.novell.com/show_bug.cgi?id=437924#c4 --- Comment #4 from Richard Bos <richard.bos@xs4all.nl> 2008-11-26 03:22:12 MST --- Op woensdag 26 november 2008 10:52:39 schreef u:
The BS log shows some rpmlint warnings, I'll list them below. I compared the warnings with openSUSE:Factory. The latter shows less warnings, although the specfile is almost the same...
Is this patch cyrus-imapd-2.3.1x-autocreate-0.10-0.diff still needed? If the patch is not accepted upstream, it makes opensuse's cyrus version, almost a fork of cyrus. Is this really wanted, is it maintainable? It makes it so much harder to support!
Yes, currently it is needed. AFAIK the configuration created by the mail-server YaST module makes use of it. The patch is BTW maintained here: http://email.uoa.gr/projects/cyrus/autocreate/ Uptodate Version of the patch for recent cyrus-imapd release can be found here: http://email.uoa.gr/download/cyrus/cyrus-imapd-2.3.13/
Perhaps this reference can be included in the specfile. Like a comment just above the PatchXX line, like: # From: http://email.uoa.gr/projects/cyrus/autocreate/ # http://email.uoa.gr/download/cyrus/cyrus-imapd-2.3.13/ PatchXX: autocreate.patch
That patch is BTW included into the cyrus-imapd package by other distributions as well. (e.g. Fedora/RedHat). I agree though that it make maintaining the cyrus-imapd package somewhat harder.
Is there a reason, that the patch is not included by the cyrus project, besides being hesitant to accept patches? If 2 big linux distro's include the patch, than why not cyrus itself :(
The rpmlint warnings I got: /var/tmp/rpm-tmp.4796: line 2: /usr/sbin/usermod: No such file or directory cyrus-imapd-2.3.13-60.1 warning: user cyrus does not exist - using root warning: user cyrus does not exist - using root
RB: I think that those are due to BS, perhaps some base package is installed by default???
I guess this is because, usermod (from pwdutils) ist not installed while build the package, havn't looked into this yet, Probalby just a missing BuildRequires.
Correct, that was indeed solved that way. In my copy with version r6: cyrus-imapd> osc diff -r5 cyrus-imapd.spec | head Index: cyrus-imapd.spec =================================================================== --- cyrus-imapd.spec (revision 5) +++ cyrus-imapd.spec (working copy) @@ -20,6 +20,7 @@ Name: cyrus-imapd BuildRequires: cyrus-sasl-devel db-devel drac ed krb5-devel openslp-devel openssl-devel opie tcpd-devel +BuildRequires: pwdutils
Hopefully you can use the updated package, and hopefully the rpmlint warnings can be solved.
I'll definitely have a look at it. I'll see what I can do about the warnings that are issues during build. Though I won't promise to fix them all ;)
:) Can you let me know, when you have a new(er) version. I would like to sync my version with yours (or perhaps get rid of my version). -- Richard -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=437924 User rhafer@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=437924#c5 Ralf Haferkamp <rhafer@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |RESOLVED Resolution| |FIXED Target Milestone|--- |Future/Later --- Comment #5 from Ralf Haferkamp <rhafer@novell.com> 2009-01-15 07:46:34 MST --- New package submitted home:rhafer:Factory. Submit request for Factory is pending. The package should show up there pretty soon. I was able to remove 3 of our custom patches and fix some of the rpmline warnings. The "unstripped-binary-or-object" warnings however are still present, but as far as I can see they only show up when building locally not when the package is built inside the buildservice. Additionally they won't show up when building with "--debuginfo". -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
participants (1)
-
bugzilla_noreply＠novell.com