Bug 816988 - Please include upstream patch for GCC 4.7
Summary: Please include upstream patch for GCC 4.7
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: ice
Version: 17
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Haïkel Guémar
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-04-27 12:02 UTC by Mary Ellen Foster
Modified: 2012-06-26 00:40 UTC (History)
1 user (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2012-06-26 00:40:21 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)
Modified version of upstream patch (43.51 KB, patch)
2012-05-11 13:59 UTC, Mary Ellen Foster
no flags Details | Diff

Description Mary Ellen Foster 2012-04-27 12:02:36 UTC
Description of problem:
ZeroC upstream has provided a patch that fixes compilation with GCC 4.7 and other recent compilers:
    http://www.zeroc.com/forums/attachments/patches/968d1329271168-patch-compiling-ice-clang-gcc4-7-ice_for_clang.txt

Please apply this patch to the Fedora packages. If not, you get the bug described here on Fedora 17:
    http://www.zeroc.com/forums/bug-reports/5709-problem-ami-gcc-4-7-a.html#post24868

Version-Release number of selected component (if applicable):
ice-3.4.2-12.fc17

How reproducible:
Every time

Steps to Reproduce:
(See above forum posting)

Comment 1 Mary Ellen Foster 2012-05-09 14:35:25 UTC
I see that the upCast related patch was apparently applied in version 3.4.2-11. Regardless, I still get failures building on Fedora 17. :(

Comment 2 Mary Ellen Foster 2012-05-11 13:59:26 UTC
Created attachment 583855 [details]
Modified version of upstream patch

Here's a slightly modified version of the upstream patch from http://www.zeroc.com/forums/patches/5647-patch-compiling-ice-clang-gcc4-7-a.html -- basically, it adds __linux to the list of #ifdefs in three places, and is otherwise unchanged. With this patch, I no longer see any problems with GCC 4.7 on Fedora 17.

This should replace the existing Patch1, Patch7, and Patch8 from the srpm.

I also note that the current SRPM contains a Patch9 which is not currently being applied as part of the build -- is that a problem?

Comment 3 Mary Ellen Foster 2012-05-21 09:12:00 UTC
Ping -- any chance of updated packages? At the moment, ice is actually broken on F17 (due to this and to bug 812156) ...

Comment 4 Haïkel Guémar 2012-05-21 09:15:59 UTC
I'm working on that, though ice build time doesn't help. :]

Comment 5 Fedora Update System 2012-05-28 08:35:40 UTC
ice-3.4.2-13.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/ice-3.4.2-13.fc17

Comment 6 Fedora Update System 2012-05-29 10:26:34 UTC
Package ice-3.4.2-13.fc17:
* should fix your issue,
* was pushed to the Fedora 17 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing ice-3.4.2-13.fc17'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/FEDORA-2012-8533/ice-3.4.2-13.fc17
then log in and leave karma (feedback).

Comment 7 Fedora Update System 2012-05-29 15:38:20 UTC
ice-3.4.2-14.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/ice-3.4.2-14.fc17

Comment 8 Fedora Update System 2012-06-12 13:58:07 UTC
ice-3.4.2-15.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/ice-3.4.2-15.fc17

Comment 9 Fedora Update System 2012-06-26 00:40:21 UTC
ice-3.4.2-15.fc17 has been pushed to the Fedora 17 stable repository.  If problems still persist, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.