* Re: [Patch Darwin/PR49992 2/2] remove ranlib special-casing from the darwin port.
@ 2011-11-10 14:29 Tristan Gingold
2011-11-13 14:31 ` Iain Sandoe
0 siblings, 1 reply; 4+ messages in thread
From: Tristan Gingold @ 2011-11-10 14:29 UTC (permalink / raw)
To: Iain Sandoe; +Cc: GCC Patches, Arnaud Charlet
Hi Iain,
Late reply from gcc-patches archive.
You can simplify further m-lib-tgt-specific-darwin.adb as the default indexer options is nothing:
Index: mlib-tgt-specific-darwin.adb
===================================================================
--- mlib-tgt-specific-darwin.adb (revision 180458)
+++ mlib-tgt-specific-darwin.adb (working copy)
@@ -36,8 +36,6 @@
-- Non default subprograms
- function Archive_Indexer_Options return String_List_Access;
-
procedure Build_Dynamic_Library
(Ofiles : Argument_List;
Options : Argument_List;
@@ -62,15 +60,6 @@
Shared_Options : constant Argument_List :=
(1 => Shared_Libgcc'Access);
- -----------------------------
- -- Archive_Indexer_Options --
- -----------------------------
-
- function Archive_Indexer_Options return String_List_Access is
- begin
- return new String_List'(1 => new String'("-c"));
- end Archive_Indexer_Options;
-
---------------------------
-- Build_Dynamic_Library --
---------------------------
@@ -175,7 +164,6 @@
end Is_Archive_Ext;
begin
- Archive_Indexer_Options_Ptr := Archive_Indexer_Options'Access;
Build_Dynamic_Library_Ptr := Build_Dynamic_Library'Access;
DLL_Ext_Ptr := DLL_Ext'Access;
Dynamic_Option_Ptr := Dynamic_Option'Access;
I have tested this chunk together with the one you provided for Makefile.in:
Index: gcc/ada/gcc-interface/Makefile.in
===================================================================
--- gcc/ada/gcc-interface/Makefile.in (revision 179962)
+++ gcc/ada/gcc-interface/Makefile.in (working copy)
@@ -2179,7 +2179,6 @@ ifeq ($(strip $(filter-out darwin%,$(osys))),)
EH_MECHANISM=-gcc
GNATLIB_SHARED = gnatlib-shared-darwin
- RANLIB = ranlib -c
GMEM_LIB = gmemlib
LIBRARY_VERSION := $(LIB_VERSION)
soext = .dylib
and I didn't find regressions in our test suite (on x86_64-darwin).
I think you just need to rewrite the ChangeLog entry to get the OK from an Ada maintainer.
Tristan.
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [Patch Darwin/PR49992 2/2] remove ranlib special-casing from the darwin port.
2011-11-10 14:29 [Patch Darwin/PR49992 2/2] remove ranlib special-casing from the darwin port Tristan Gingold
@ 2011-11-13 14:31 ` Iain Sandoe
2011-11-13 15:19 ` Iain Sandoe
0 siblings, 1 reply; 4+ messages in thread
From: Iain Sandoe @ 2011-11-13 14:31 UTC (permalink / raw)
To: Arnaud Charlet; +Cc: Tristan Gingold, GCC Patches
[-- Attachment #1: Type: text/plain, Size: 876 bytes --]
On 10 Nov 2011, at 13:44, Tristan Gingold wrote:
> You can simplify further m-lib-tgt-specific-darwin.adb as the
> default indexer options is nothing:
thanks - I wasn't confident to remove it entirely, but that is a much
better solution.
> I have tested this chunk together with the one you provided for
> Makefile.in:
> and I didn't find regressions in our test suite (on x86_64-darwin).
likewise on *-darwin9 (re-tested on ppc-darwin with the revised patch
and the other changes in trunk).
> I think you just need to rewrite the ChangeLog entry to get the OK
> from an Ada maintainer.
see revised patch/Changelog.
Ok for trunk/4.6?
Iain
===
2011-xx-yy Tristan Gingold <...
Iain Sandoe <...
gcc/ada:
PR target/49992
* mlib-tgt-specific-darwin.adb (Archive_Indexer_Options): Remove.
* gcc-interface/Makefile.in (darwin): Likewise.
[-- Attachment #2: 181319-pr49992-part2-revised.txt --]
[-- Type: text/plain, Size: 1764 bytes --]
Index: gcc/ada/mlib-tgt-specific-darwin.adb
===================================================================
--- gcc/ada/mlib-tgt-specific-darwin.adb (revision 181311)
+++ gcc/ada/mlib-tgt-specific-darwin.adb (working copy)
@@ -36,8 +36,6 @@ package body MLib.Tgt.Specific is
-- Non default subprograms
- function Archive_Indexer_Options return String_List_Access;
-
procedure Build_Dynamic_Library
(Ofiles : Argument_List;
Options : Argument_List;
@@ -62,15 +60,6 @@ package body MLib.Tgt.Specific is
Shared_Options : constant Argument_List :=
(1 => Shared_Libgcc'Access);
- -----------------------------
- -- Archive_Indexer_Options --
- -----------------------------
-
- function Archive_Indexer_Options return String_List_Access is
- begin
- return new String_List'(1 => new String'("-c"));
- end Archive_Indexer_Options;
-
---------------------------
-- Build_Dynamic_Library --
---------------------------
@@ -175,7 +164,6 @@ package body MLib.Tgt.Specific is
end Is_Archive_Ext;
begin
- Archive_Indexer_Options_Ptr := Archive_Indexer_Options'Access;
Build_Dynamic_Library_Ptr := Build_Dynamic_Library'Access;
DLL_Ext_Ptr := DLL_Ext'Access;
Dynamic_Option_Ptr := Dynamic_Option'Access;
Index: gcc/ada/gcc-interface/Makefile.in
===================================================================
--- gcc/ada/gcc-interface/Makefile.in (revision 181311)
+++ gcc/ada/gcc-interface/Makefile.in (working copy)
@@ -2182,7 +2182,6 @@ ifeq ($(strip $(filter-out darwin%,$(osys))),)
EH_MECHANISM=-gcc
GNATLIB_SHARED = gnatlib-shared-darwin
- RANLIB = ranlib -c
GMEM_LIB = gmemlib
LIBRARY_VERSION := $(LIB_VERSION)
soext = .dylib
[-- Attachment #3: Type: text/plain, Size: 3 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [Patch Darwin/PR49992 2/2] remove ranlib special-casing from the darwin port.
2011-11-13 14:31 ` Iain Sandoe
@ 2011-11-13 15:19 ` Iain Sandoe
0 siblings, 0 replies; 4+ messages in thread
From: Iain Sandoe @ 2011-11-13 15:19 UTC (permalink / raw)
To: Arnaud Charlet; +Cc: Tristan Gingold, GCC Patches
On 13 Nov 2011, at 09:56, Iain Sandoe wrote:
>
> On 10 Nov 2011, at 13:44, Tristan Gingold wrote:
>> You can simplify further m-lib-tgt-specific-darwin.adb as the
>> default indexer options is nothing:
>
> thanks - I wasn't confident to remove it entirely, but that is a
> much better solution.
>
>> I have tested this chunk together with the one you provided for
>> Makefile.in:
>
>
>> and I didn't find regressions in our test suite (on x86_64-darwin).
>
> likewise on *-darwin9 (re-tested on ppc-darwin with the revised
> patch and the other changes in trunk).
>
>> I think you just need to rewrite the ChangeLog entry to get the OK
>> from an Ada maintainer.
>
> see revised patch/Changelog.
>
> Ok for trunk/4.6?
> Iain
>
> ===
Uhhh that'll teach me to cut&paste (well.. maybe not ;-))
> 2011-xx-yy Tristan Gingold <...
> Iain Sandoe <...
>
> gcc/ada:
PR target/49992
* mlib-tgt-specific-darwin.adb (Archive_Indexer_Options): Remove.
* gcc-interface/Makefile.in (darwin): Remove ranlib special-casing
for Darwin.
> <181319-pr49992-part2-revised.txt>
>
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* [Patch Darwin/PR49992 2/2] remove ranlib special-casing from the darwin port.
@ 2011-10-14 10:19 Iain Sandoe
0 siblings, 0 replies; 4+ messages in thread
From: Iain Sandoe @ 2011-10-14 10:19 UTC (permalink / raw)
To: GCC Patches; +Cc: Arnaud Charlet, Mike Stump
As per the PR audit trail, there is no reason to retain this in the
building of GCC.
As for its use as a general option in tool-builds;
With current darwin toolsets it has the potential to cause issues when
using convenience libs containing common.
OK for trunk?
Iain
gcc/ada:
PR target/49992
* mlib-tgt-specific-darwin.adb: Remove ranlib special case.
* gcc-interface/Makefile.in (darwin): Likewise.
Index: gcc/ada/mlib-tgt-specific-darwin.adb
===================================================================
--- gcc/ada/mlib-tgt-specific-darwin.adb (revision 179962)
+++ gcc/ada/mlib-tgt-specific-darwin.adb (working copy)
@@ -68,7 +68,7 @@ package body MLib.Tgt.Specific is
function Archive_Indexer_Options return String_List_Access is
begin
- return new String_List'(1 => new String'("-c"));
+ return new String_List'(1 => new String'(""));
end Archive_Indexer_Options;
---------------------------
Index: gcc/ada/gcc-interface/Makefile.in
===================================================================
--- gcc/ada/gcc-interface/Makefile.in (revision 179962)
+++ gcc/ada/gcc-interface/Makefile.in (working copy)
@@ -2179,7 +2179,6 @@ ifeq ($(strip $(filter-out darwin%,$(osys))),)
EH_MECHANISM=-gcc
GNATLIB_SHARED = gnatlib-shared-darwin
- RANLIB = ranlib -c
GMEM_LIB = gmemlib
LIBRARY_VERSION := $(LIB_VERSION)
soext = .dylib
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2011-11-13 11:34 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-11-10 14:29 [Patch Darwin/PR49992 2/2] remove ranlib special-casing from the darwin port Tristan Gingold
2011-11-13 14:31 ` Iain Sandoe
2011-11-13 15:19 ` Iain Sandoe
-- strict thread matches above, loose matches on Subject: below --
2011-10-14 10:19 Iain Sandoe
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).