Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 27645 - cyrus-imapd 2.1.14 and cyrus-imap-admin 2.1.14 fail to apply patch before building.
Summary: cyrus-imapd 2.1.14 and cyrus-imap-admin 2.1.14 fail to apply patch before bui...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High critical (vote)
Assignee: Net-Mail Packages
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-08-31 05:50 UTC by James
Modified: 2004-01-26 09:39 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
Output from console showing bug. (cyrus-imap-admin-fail,1.61 KB, text/plain)
2003-08-31 05:51 UTC, James
Details
Output wanting to be included with cyrus-imap-admin-fail (cyrus-imapd-2.1.12-includepath.patch-27598.out,3.57 KB, text/plain)
2003-08-31 05:53 UTC, James
Details
Console output showing failure (cyrus-imapd-fail,1.59 KB, text/plain)
2003-08-31 05:53 UTC, James
Details
Output wanting to be included with cyrus-imapd-fail (cyrus-imapd-2.1.12-includepath.patch-27217.out,3.55 KB, text/plain)
2003-08-31 05:54 UTC, James
Details

Note You need to log in before you can comment on or make changes to this bug.
Description James 2003-08-31 05:50:31 UTC
I attach file showing the output on the console demonstrating the problem.
Summary: -
 * Applying cyrus-imapd-2.1.12-includepath.patch...
 * Failed Patch: cyrus-imapd-2.1.12-includepath.patch!


Reproducible: Always
Steps to Reproduce:
1. emerge -u cyrus-imapd
2. fails with attached output.

Actual Results:  
Fails to complete patch.

Expected Results:  
Should build without any problems.
Comment 1 James 2003-08-31 05:51:25 UTC
Created attachment 16852 [details]
Output from console showing bug.
Comment 2 James 2003-08-31 05:53:28 UTC
Created attachment 16853 [details]
Output wanting to be included with cyrus-imap-admin-fail
Comment 3 James 2003-08-31 05:53:51 UTC
Created attachment 16854 [details]
Console output showing failure
Comment 4 James 2003-08-31 05:54:41 UTC
Created attachment 16855 [details]
Output wanting to be included with cyrus-imapd-fail
Comment 5 Robert Moss (RETIRED) gentoo-dev 2003-09-04 16:10:42 UTC
Hmmm, looks like these patches could be being applied in the wrong order... just out of curiosity, what version of sys-devel/patch are you using? Is it pre-2.5.9?
Comment 6 James 2003-09-04 20:07:56 UTC
I have patch version 2.5.9
I don't think that ./configure should ever be patched in gentoo, because ./configure is a generated file. One should patch the file that is used to generate it. E.g. configure.in, or configure.ac
Another point, I managed to install this package ok, if I removed db version 4 and installed db 3.2 instead, and also edited the ebuild to not do any patches.
I have talked to the cyrus people, and they confirm that the ./configure script cannot detect the db4 used in gentoo, and also, even if the ./configure script is modified to detect it, cyrus makes calls to db4 that do not exist, and only existed for db3.2, so cyrus needs some reworking to talk db4.
Maybe you could take the latest cyrus version, and work with the cyrus developers to get it to compile on gentoo with db4 installed.
Comment 7 Max Kalika (RETIRED) gentoo-dev 2003-09-18 19:25:42 UTC
Please try new 2.1.14 or 2.1.15 (cyrus-imapd only for now until I get some confirmations) version that is in portage now.
Comment 8 James 2003-09-19 06:16:48 UTC
I am still downloading the .tar.gz file, but initally I found a problem.
Summary: -
SRC_URI is set wrong.

--- cyrus-imapd-2.1.15.ebuild.org       2003-09-19 14:07:32.887241176 +0100
+++ cyrus-imapd-2.1.15.ebuild   2003-09-19 14:07:53.211151472 +0100
@@ -9,7 +9,7 @@

 DESCRIPTION="The Cyrus IMAP Server"
 HOMEPAGE="http://asg.web.cmu.edu/cyrus/imapd/"
-SRC_URI="ftp://ftp.andrew.cmu.edu/pub/cyrus-mail/OLD-VERSIONS/imap/${P}.tar.gz
+SRC_URI="ftp://ftp.andrew.cmu.edu/pub/cyrus-mail/${P}.tar.gz
        ipv6? ( http://www.imasy.or.jp/~ume/ipv6/${IPV6_P}.diff.gz )"

 LICENSE="as-is"
Comment 9 James 2003-09-19 06:43:37 UTC
This does not fix anything.
The patches included in the ebuild still do not patch in, because it is still trying to patch an emerge generated "configure" file, instead of the original "configure.in"
Also, the db4 patch does not help, because cyrus ./configure still does not detect db4, and thus compiling fails, because cyrus is trying to use the db3.2 api, and not the db4 api.

Comment 10 Max Kalika (RETIRED) gentoo-dev 2003-09-19 08:51:51 UTC
The SRC_URI is fixed.  My bad on the copy/paste. 
 
James: The includepath patch gets applied after configure is recreated.  I'm 
not sure what is breaking for you.  I tried it here on several boxes and 
everything seems fine. 
 
ike cyrus-imapd # emerge =cyrus-imapd-2.1.15 
Calculating dependencies ...done! 
>>> emerge (1 of 1) net-mail/cyrus-imapd-2.1.15 to / 
>>> md5 src_uri ;-) cyrus-imapd-2.1.15.tar.gz 
>>> Unpacking source... 
>>> Unpacking cyrus-imapd-2.1.15.tar.gz to 
/var/tmp/portage/cyrus-imapd-2.1.15/work 
 * Applying 2.1.15-db4.patch...                             [ ok ] 
 * Applying cyrus-imapd-2.1.12-libwrap.patch...             [ ok ] 
Remember to add `AC_PROG_LIBTOOL' to `configure.in'. 
Using `AC_PROG_RANLIB' is rendered obsolete by `AC_PROG_LIBTOOL' 
You should update your `aclocal.m4' by running aclocal. 
 * Applying cyrus-imapd-2.1.12-includepath.patch...         [ ok ] 
>>> Source unpacked. 
 
Comment 11 James 2003-09-19 10:28:13 UTC
My point is that different versions of aclocal/automake etc. will create different looking ./configure scripts.
So, the patch to the ./configure script will only work on systems that have exactly the same version of automake etc. as you have.
But if yon do the patch on the ./configure.in script, it will work for ALL automake versions.
Comment 12 Max Kalika (RETIRED) gentoo-dev 2003-09-22 13:51:01 UTC
Side issue: Is this patch absolutely necessary? Can people try to remove that 
line from the ebuild and try to compile?  Things seem to work just fine here 
without it. 
Comment 13 Max Kalika (RETIRED) gentoo-dev 2004-01-26 09:39:54 UTC
Fixed in 2.2.3