diff mbox

update default OS for debugging on darwin

Message ID EB761ADB-8288-442A-8692-B58582C5BDA8@comcast.net
State New
Headers show

Commit Message

Mike Stump Feb. 7, 2011, 9:05 p.m. UTC
Not a big issue, but when debugging, I want a default that more closely matches my system, without having to give extra arguments when running cc1.


2011-02-07  Mike Stump  <mikestump@comcast.net>

	* config/darwin.opt (mmacosx-version-min): Update default OS
	version.

Comments

Jack Howarth Feb. 8, 2011, 1:30 a.m. UTC | #1
On Mon, Feb 07, 2011 at 01:05:22PM -0800, Mike Stump wrote:
> Not a big issue, but when debugging, I want a default that more closely matches my system, without having to give extra arguments when running cc1.
> 
> 
> 2011-02-07  Mike Stump  <mikestump@comcast.net>
> 
> 	* config/darwin.opt (mmacosx-version-min): Update default OS
> 	version.
> 
> Index: config/darwin.opt
> ===================================================================
> --- config/darwin.opt	(revision 169897)
> +++ config/darwin.opt	(revision 169898)
> @@ -217,12 +217,13 @@
>  Target Report Var(darwin_fix_and_continue)
>  Generate code suitable for fast turn around debugging
>  
> -; The Init here is for the convenience of GCC developers, so that
> -; cc1 and cc1plus don't crash if no -mmacosx-version-min is passed.  The
> -; driver will always pass a -mmacosx-version-min, so in normal use
> -; the Init is never used.
> +; The Init here is for the convenience of GCC developers, so that cc1
> +; and cc1plus don't crash if no -mmacosx-version-min is passed.  The
> +; driver will always pass a -mmacosx-version-min, so in normal use the
> +; Init is never used.  Useful for setting the OS on which people
> +; ususally debug.
>  mmacosx-version-min=
> -Target Joined Report Var(darwin_macosx_version_min) Init("10.1")
> +Target Joined Report Var(darwin_macosx_version_min) Init("10.6")
>  The earliest MacOS X version on which this program will run
>  
>  mone-byte-bool

Mike,
   Speaking of OS versions, can you take a look again at...

http://gcc.gnu.org/ml/gcc-patches/2010-08/msg01703.html

I think it is long past time to start moving the ancient darwin
releases into the unsupported target list. Again, that second hunk
removes  *-darwin1[0-9]* from the powerpc-*-darwin* case since
darwin10 was never released for powerpc.
                  Jack
Mike Stump Feb. 8, 2011, 6:49 a.m. UTC | #2
On Feb 7, 2011, at 5:30 PM, Jack Howarth wrote:
>   Speaking of OS versions, can you take a look again at...
> 
> http://gcc.gnu.org/ml/gcc-patches/2010-08/msg01703.html

> I think it is long past time to start moving the ancient darwin
> releases into the unsupported target list. Again, that second hunk
> removes  *-darwin1[0-9]* from the powerpc-*-darwin* case since
> darwin10 was never released for powerpc.

But I run ppc on darwin10...  :-(  I would have to think about this, which if the patch is supposed to save me from from thinking about older system, is counter productive.  So, I'd propose I not think about this and we leave things as is.  We aren't saving much by adding a line to error out.
Iain Sandoe Feb. 8, 2011, 8:22 a.m. UTC | #3
On 8 Feb 2011, at 06:49, Mike Stump wrote:

> On Feb 7, 2011, at 5:30 PM, Jack Howarth wrote:
>>  Speaking of OS versions, can you take a look again at...
>>
>> http://gcc.gnu.org/ml/gcc-patches/2010-08/msg01703.html
>
>> I think it is long past time to start moving the ancient darwin
>> releases into the unsupported target list. Again, that second hunk
>> removes  *-darwin1[0-9]* from the powerpc-*-darwin* case since
>> darwin10 was never released for powerpc.
>
> But I run ppc on darwin10...  :-(  I would have to think about this,  
> which if the patch is supposed to save me from from thinking about  
> older system, is counter productive.  So, I'd propose I not think  
> about this and we leave things as is.  We aren't saving much by  
> adding a line to error out.

besides which, since we now have a compiler which is Darwin64  
compatible,
  perhaps the puredarwin folks might be interested in building for  
powerpc. (perhaps not ;-))
cheers
Iain
diff mbox

Patch

Index: config/darwin.opt
===================================================================
--- config/darwin.opt	(revision 169897)
+++ config/darwin.opt	(revision 169898)
@@ -217,12 +217,13 @@ 
 Target Report Var(darwin_fix_and_continue)
 Generate code suitable for fast turn around debugging
 
-; The Init here is for the convenience of GCC developers, so that
-; cc1 and cc1plus don't crash if no -mmacosx-version-min is passed.  The
-; driver will always pass a -mmacosx-version-min, so in normal use
-; the Init is never used.
+; The Init here is for the convenience of GCC developers, so that cc1
+; and cc1plus don't crash if no -mmacosx-version-min is passed.  The
+; driver will always pass a -mmacosx-version-min, so in normal use the
+; Init is never used.  Useful for setting the OS on which people
+; ususally debug.
 mmacosx-version-min=
-Target Joined Report Var(darwin_macosx_version_min) Init("10.1")
+Target Joined Report Var(darwin_macosx_version_min) Init("10.6")
 The earliest MacOS X version on which this program will run
 
 mone-byte-bool