[Patch Darwin/PR49992 2/2] remove ranlib special-casing from the darwin port.

[Patch Darwin/PR49992 2/2] remove ranlib special-casing from the darwin port.

[Iain Sandoe]

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

PING * : [Patch Darwin/PR49992 2/2] remove ranlib special-casing from the darwin port.

[Iain Sandoe]

On 14 Oct 2011, at 10:37, Iain Sandoe wrote:

> 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
>
>

PING 2 : [Patch Darwin/PR49992 2/2] remove ranlib special-casing from the darwin port.

[Iain Sandoe]

This is unreviewed for 2 weeks.

I am sure that this issue will be affecting Ada on Darwin10/11 with  
the latest toolchains.

It might be subtle without LTO - OTOH when LTO is engaged it breaks  
things completely.


On 22 Oct 2011, at 08:37, Iain Sandoe wrote:

>
> On 14 Oct 2011, at 10:37, Iain Sandoe wrote:
>
>> 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
>>
>>
>

Re: PING 2 : [Patch Darwin/PR49992 2/2] remove ranlib special-casing from the darwin port.

[Mike Stump]

On Oct 28, 2011, at 8:41 AM, Iain Sandoe wrote:
> This is unreviewed for 2 weeks.

Odd, usually the Ada people are fairly responsive.  If they want me to weigh in, I approve of the concept behind the work.

Re: PING 2 : [Patch Darwin/PR49992 2/2] remove ranlib special-casing from the darwin port.

[Arnaud Charlet]

Le 28/10/2011 17:41, Iain Sandoe a écrit :
> This is unreviewed for 2 weeks.
>
> I am sure that this issue will be affecting Ada on Darwin10/11 with 
> the latest toolchains.

It's actually under discussion and is pretty subtle, so delicate. Thanks 
for your patience.

Arno

Re: PING * : [Patch Darwin/PR49992 2/2] remove ranlib special-casing from the darwin port.

[Arnaud Charlet]

The new version of the patch as suggested by Tristan is OK to commit,
thanks.

Arno

Re: [Patch Darwin/PR49992 2/2] remove ranlib special-casing from the darwin port.

[Iain Sandoe]

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>
>
>

Re: [Patch Darwin/PR49992 2/2] remove ranlib special-casing from the darwin port.

[Iain Sandoe]

[-- 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 --]

Re: [Patch Darwin/PR49992 2/2] remove ranlib special-casing from the darwin port.

[Tristan Gingold]

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.