* [Patch Darwin/PR49992 1/2] remove ranlib special-casing from the darwin port.
@ 2011-10-14 10:13 Iain Sandoe
2011-10-22 13:47 ` PING * : " Iain Sandoe
0 siblings, 1 reply; 3+ messages in thread
From: Iain Sandoe @ 2011-10-14 10:13 UTC (permalink / raw)
To: GCC Patches; +Cc: Ralf Wildenhues, Mike Stump
As per the PR audit trail, there is no reason to retain this special-
casing for Darwin.
(given that current GCC is not build-able using Darwin toolsets of
the vintage that required the case).
Mike has OK'd this off-list - but, since Ralf commented on the
previous version, I'd like to give him the opportunity to comment here.
OK for trunk?
Iain
* configure.ac: Remove ranlib special case for Darwin port.
* gcc/configure.ac: Likewise.
* configure: Regenerate.
* gcc/configure: Regenerate.
Index: configure.ac
===================================================================
--- configure.ac (revision 179962)
+++ configure.ac (working copy)
@@ -2274,10 +2274,6 @@ case "${target}" in
extra_arflags_for_target=" -X32_64"
extra_nmflags_for_target=" -B -X32_64"
;;
- *-*-darwin[[3-9]]*)
- # ranlib before Darwin10 requires the -c flag to look at common
symbols.
- extra_ranlibflags_for_target=" -c"
- ;;
esac
alphaieee_frag=/dev/null
Index: gcc/configure.ac
===================================================================
--- gcc/configure.ac (revision 179962)
+++ gcc/configure.ac (working copy)
@@ -829,17 +829,7 @@ esac
gcc_AC_PROG_LN_S
ACX_PROG_LN($LN_S)
AC_PROG_RANLIB
-case "${host}" in
-*-*-darwin*)
- # By default, the Darwin ranlib will not treat common symbols as
- # definitions when building the archive table of contents. Other
- # ranlibs do that; pass an option to the Darwin ranlib that makes
- # it behave similarly.
- ranlib_flags="-c"
- ;;
-*)
- ranlib_flags=""
-esac
+ranlib_flags=""
AC_SUBST(ranlib_flags)
gcc_AC_PROG_INSTALL
^ permalink raw reply [flat|nested] 3+ messages in thread
* PING * : [Patch Darwin/PR49992 1/2] remove ranlib special-casing from the darwin port.
2011-10-14 10:13 [Patch Darwin/PR49992 1/2] remove ranlib special-casing from the darwin port Iain Sandoe
@ 2011-10-22 13:47 ` Iain Sandoe
2011-10-28 15:52 ` PING 2 " Iain Sandoe
0 siblings, 1 reply; 3+ messages in thread
From: Iain Sandoe @ 2011-10-22 13:47 UTC (permalink / raw)
To: GCC Patches; +Cc: Ralf Wildenhues, Mike Stump
On 14 Oct 2011, at 10:36, Iain Sandoe wrote:
> As per the PR audit trail, there is no reason to retain this special-
> casing for Darwin.
> (given that current GCC is not build-able using Darwin toolsets of
> the vintage that required the case).
>
> Mike has OK'd this off-list - but, since Ralf commented on the
> previous version, I'd like to give him the opportunity to comment
> here.
> OK for trunk?
> Iain
>
> * configure.ac: Remove ranlib special case for Darwin port.
> * gcc/configure.ac: Likewise.
> * configure: Regenerate.
> * gcc/configure: Regenerate.
>
> Index: configure.ac
> ===================================================================
> --- configure.ac (revision 179962)
> +++ configure.ac (working copy)
> @@ -2274,10 +2274,6 @@ case "${target}" in
> extra_arflags_for_target=" -X32_64"
> extra_nmflags_for_target=" -B -X32_64"
> ;;
> - *-*-darwin[[3-9]]*)
> - # ranlib before Darwin10 requires the -c flag to look at common
> symbols.
> - extra_ranlibflags_for_target=" -c"
> - ;;
> esac
>
> alphaieee_frag=/dev/null
> Index: gcc/configure.ac
> ===================================================================
> --- gcc/configure.ac (revision 179962)
> +++ gcc/configure.ac (working copy)
> @@ -829,17 +829,7 @@ esac
> gcc_AC_PROG_LN_S
> ACX_PROG_LN($LN_S)
> AC_PROG_RANLIB
> -case "${host}" in
> -*-*-darwin*)
> - # By default, the Darwin ranlib will not treat common symbols as
> - # definitions when building the archive table of contents. Other
> - # ranlibs do that; pass an option to the Darwin ranlib that makes
> - # it behave similarly.
> - ranlib_flags="-c"
> - ;;
> -*)
> - ranlib_flags=""
> -esac
> +ranlib_flags=""
> AC_SUBST(ranlib_flags)
>
> gcc_AC_PROG_INSTALL
>
>
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* PING 2 : [Patch Darwin/PR49992 1/2] remove ranlib special-casing from the darwin port.
2011-10-22 13:47 ` PING * : " Iain Sandoe
@ 2011-10-28 15:52 ` Iain Sandoe
0 siblings, 0 replies; 3+ messages in thread
From: Iain Sandoe @ 2011-10-28 15:52 UTC (permalink / raw)
To: GCC Patches; +Cc: Ralf Wildenhues, Mike Stump
Since this is approved by Mike, if there is no further comment by
Monday, I plan to apply it.
On 22 Oct 2011, at 08:36, Iain Sandoe wrote:
>
> On 14 Oct 2011, at 10:36, Iain Sandoe wrote:
>
>> As per the PR audit trail, there is no reason to retain this
>> special-casing for Darwin.
>> (given that current GCC is not build-able using Darwin toolsets of
>> the vintage that required the case).
>>
>> Mike has OK'd this off-list - but, since Ralf commented on the
>> previous version, I'd like to give him the opportunity to comment
>> here.
>> OK for trunk?
>> Iain
>>
>> * configure.ac: Remove ranlib special case for Darwin port.
>> * gcc/configure.ac: Likewise.
>> * configure: Regenerate.
>> * gcc/configure: Regenerate.
>>
>> Index: configure.ac
>> ===================================================================
>> --- configure.ac (revision 179962)
>> +++ configure.ac (working copy)
>> @@ -2274,10 +2274,6 @@ case "${target}" in
>> extra_arflags_for_target=" -X32_64"
>> extra_nmflags_for_target=" -B -X32_64"
>> ;;
>> - *-*-darwin[[3-9]]*)
>> - # ranlib before Darwin10 requires the -c flag to look at
>> common symbols.
>> - extra_ranlibflags_for_target=" -c"
>> - ;;
>> esac
>>
>> alphaieee_frag=/dev/null
>> Index: gcc/configure.ac
>> ===================================================================
>> --- gcc/configure.ac (revision 179962)
>> +++ gcc/configure.ac (working copy)
>> @@ -829,17 +829,7 @@ esac
>> gcc_AC_PROG_LN_S
>> ACX_PROG_LN($LN_S)
>> AC_PROG_RANLIB
>> -case "${host}" in
>> -*-*-darwin*)
>> - # By default, the Darwin ranlib will not treat common symbols as
>> - # definitions when building the archive table of contents. Other
>> - # ranlibs do that; pass an option to the Darwin ranlib that makes
>> - # it behave similarly.
>> - ranlib_flags="-c"
>> - ;;
>> -*)
>> - ranlib_flags=""
>> -esac
>> +ranlib_flags=""
>> AC_SUBST(ranlib_flags)
>>
>> gcc_AC_PROG_INSTALL
>>
>>
>>
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2011-10-28 15:41 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-10-14 10:13 [Patch Darwin/PR49992 1/2] remove ranlib special-casing from the darwin port Iain Sandoe
2011-10-22 13:47 ` PING * : " Iain Sandoe
2011-10-28 15:52 ` PING 2 " 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).