Bug 118449 – cyrus-imapd-2.2.12-r3 fails to build wit GCC 4.0.2-r3 (patch included)

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description steveb 2006-01-09 12:22:50 UTC]

cyrus-imapd-2.2.12-r3 fails to build with GCC-4.0.2-r3. After searching the net for a solution, I found a gcc4 patch from Ondrej Sury:
--- cyrus-imapd-2.2.12/tools/config2header.orig 2006-01-09 20:54:46.000000000 +0100
+++ cyrus-imapd-2.2.12/tools/config2header      2006-01-09 20:56:56.000000000 +0100
@@ -214,7 +214,6 @@
 print HFILE <<EOF
   IMAPOPT_LAST
 };
-extern struct imapopt_s imapopts[];

 enum enum_value {
   IMAP_ENUM_ZERO = 0,
@@ -260,6 +259,7 @@

 print HFILE <<EOF
 };
+extern struct imapopt_s imapopts[];
 #endif /* INCLUDED_IMAPOPTIONS_H */
 EOF
     ;


This patch allowed me to emerge/compile cyrus-imapd-2.2.12-r3 with GCC 4.0.2. Can we include this patch into portage?

cheers

Steve

[Comment 1 Mark Loeser (RETIRED) 2006-01-09 12:45:57 UTC]

Looks fine.

[Comment 2 Tuan Van (RETIRED) 2006-01-09 13:14:22 UTC]

I believe that cyrus-imapd-2.3 series can build with gcc-4 without a patch. Unless you are planning to stable gcc-4 soon, I'd like not to patch the 2.2 series. 

P.S. 2.3.1 is a testing release so I'd like to run on my test setup for a while before commit it to the tree.

[Comment 3 steveb 2006-01-09 14:51:58 UTC]

(In reply to comment #2)
> I believe that cyrus-imapd-2.3 series can build with gcc-4 without a patch.
> Unless you are planning to stable gcc-4 soon, I'd like not to patch the 2.2
> series. 
> 
> P.S. 2.3.1 is a testing release so I'd like to run on my test setup for a while
> before commit it to the tree.
> 

I don't know about when gcc-4 will get into stable, but I think that it will soon be moved to ~arch and since the current 2.2 release of Cyrus-IMAPD is not able to be compiled with gcc4, I think it would not be such a bad idea to add the patch for the 2.2 series of Cyrus-IMAPD.

[Comment 4 Tuan Van (RETIRED) 2006-01-11 21:15:16 UTC]

(In reply to comment #3)
> 
> I don't know about when gcc-4 will get into stable, but I think that it will
> soon be moved to ~arch and since the current 2.2 release of Cyrus-IMAPD is not
> able to be compiled with gcc4, I think it would not be such a bad idea to add
> the patch for the 2.2 series of Cyrus-IMAPD.
> 

cyrus-imapd-2.3.1 is in the tree. Please try that one too if you can. Note it is package.masked.

[Comment 5 Mark Loeser (RETIRED) 2006-01-27 21:31:47 UTC]

2.3 looks good for gcc-4.  When is it coming out of p.mask?  If it isn't going to be soon, can you please patch 2.2?

[Comment 6 Tuan Van (RETIRED) 2006-01-30 10:58:18 UTC]

2.2.12-r4 commited with gcc4 patch. I shouldn't bump but oh-well. Thanks everyone.