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.