Bug 817684 - building against ice requires -fpermissive
Summary: building against ice requires -fpermissive
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-30 21:06 UTC by Matthew Woehlke
Modified: 2012-06-26 00:40 UTC (History)
1 user (show)

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


Attachments (Terms of Use)
patches from arch linux (11.74 KB, patch)
2012-04-30 21:06 UTC, Matthew Woehlke
no flags Details | Diff

Description Matthew Woehlke 2012-04-30 21:06:08 UTC
Created attachment 581280 [details]
patches from arch linux

Description of problem:
Due to some missing use of 'this->' in ICE public headers, building code that uses ICE (and building ICE itself, for that matter) requires -fpermissive when using gcc 4.7.

Version-Release number of selected component (if applicable):
ice-3.4.2.12

How reproducible:
Always

Additional info:
I've attached a patch containing a number of changes, including ones to fix this issue, taken from arch linux. (These are taken from 'ice-for-clang.patch' in http://aur.archlinux.org/packages/ze/zeroc-ice/zeroc-ice.tar.gz.) With these, ICE itself appears to build successfully without -fpermissive and to likewise be usable without the same.

Comment 1 Fedora Update System 2012-05-28 08:35:19 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 2 Fedora Update System 2012-05-29 10:25:43 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 3 Fedora Update System 2012-05-29 15:38:07 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 4 Matthew Woehlke 2012-05-29 17:12:28 UTC
I'm still verifying, but I'm guessing the patch set in -14 is a superset of the patches here.

However, I would still suggest removing -fpermissive (which I see is still present) from ICE's build, at least as a validation measure.

Comment 5 Fedora Update System 2012-06-12 13:57:48 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 6 Fedora Update System 2012-06-26 00:40:08 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.