#1862: Configure.pl should actually test for gcc -fvisibility=hidden support
-----------------------+----------------------------------------------------
Reporter: doughera | Owner: jkeenan
Type: bug | Status: assigned
Priority: normal | Milestone: 2.11
Component: configure | Version: 2.10.0
Severity: medium | Keywords:
Lang: | Patch: applied
Platform: |
-----------------------+----------------------------------------------------
Changes (by jkeenan):
* patch: => applied
Comment:
Replying to [comment:2 doughera]:
>
> Almost -- I think you'd have to delete the test in line 300, since
sym_export will already be defined by that point for gcc >= 4.0 -- but
yes, this should work. Thanks for cooking up the patch.
I modified the patch per Andy's suggestion. I then configured in two
ways: first with gcc-3.3, second with gcc-4.something. In the first case
'visibility' was not found in ''lib/Parrot/Config/Generated.pm''; in the
second case it was. So I think this demonstrates the patch achieved its
end.
Committed to master in commit 3b80d03. Will keep the patch open until the
weekend for comments or complaints.
Thank you very much.
kid51
--
Ticket URL: <https://trac.parrot.org/parrot/ticket/1862#comment:3>
Parrot <https://trac.parrot.org/parrot/>
Parrot Development
_______________________________________________
parrot-tickets mailing list
[email protected]
http://lists.parrot.org/mailman/listinfo/parrot-tickets
