After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 524625 - patch to compile pan with gcc 4.3
patch to compile pan with gcc 4.3
Status: RESOLVED FIXED
Product: Pan
Classification: Other
Component: general
pre-1.0 betas
Other Linux
: Normal blocker
: 0.133
Assigned To: Charles Kerr
Pan QA Team
Depends on:
Blocks:
 
 
Reported: 2008-03-27 11:46 UTC by Duncan
Modified: 2008-07-05 05:53 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
gcc 4.3 patch for pan (1.38 KB, patch)
2008-03-27 11:57 UTC, Duncan
none Details | Review

Description Duncan 2008-03-27 11:46:24 UTC
Here's a patch to pan so it compiles with gcc 4.3, as originally posted on the developer list  by Daniel Rahn (OpenSuSE), with a slight modification combining a couple separate patches to the same file at the same place into one, as Gentoo's epatch didn't like it as originally posted.  If he bugged it, I don't see it, so I thought I'd do so so it doesn't get lost.

Patch to be attached as it doesn't appear I can attach it on original bug filing...

Duncan
Comment 1 Duncan 2008-03-27 11:57:04 UTC
Created attachment 108106 [details] [review]
gcc 4.3 patch for pan

The above patch cures the errors, but there's still a lot of warnings about deprecated headers such as what follows.  You may wish to clear these while you are at it, and not have to worry about 4.4 so much.  There's info in the gcc 4.3 porting guide at

http://gcc.gnu.org/gcc-4.3/porting_to.html

Here's one of the remaining warnings:

In file included from /usr/lib/gcc/x86_64-pc-linux-gnu/4.3.0/include/g++-v4/ext/hash_set:64,
                 from ../../pan/general/quark.h:32,
                 from my-tree.cc:25:
/usr/lib/gcc/x86_64-pc-linux-gnu/4.3.0/include/g++-v4/backward/backward_warning.h:32:2: warning: #warning This file includes at least one deprecated or antiquated header which may be removed without further notice at a future date. Please use a non-deprecated interface with equivalent functionality instead. For a listing of replacement headers and interfaces, consult the file backward_warning.h. To disable this warning use -Wno-deprecated.
Comment 2 Duncan 2008-03-27 14:56:15 UTC
FWIW, the downstream Gentoo bug, with a slightly different patch it appears, is

http://bugs.gentoo.org/show_bug.cgi?id=211670
Comment 3 Petr Kovar 2008-04-13 16:55:43 UTC
Confirming the bug and changing the severity to blocker. Hope it gets some attention.
Comment 4 Charles Kerr 2008-07-04 16:08:51 UTC
svn revision 342 contains a number of changes along these lines.  It's currently building for me on RH9, and I'm not sure which (if any) of the pieces of this patch still need to be applied.

Could you do a fresh checkout and see if any of these compile errors (not warnings) persist?
Comment 5 Duncan 2008-07-04 19:47:48 UTC
Confirmed FIXED_IN_SVN with r345. =8^)

Actually, all four patches Gentoo was carrying against 0.132, my local live-svn ebuild being recently rebased on it, are no longer needed. =8^)

Thanks!
Duncan