This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Depreciate darwin[0-7]


On Sat, Aug 21, 2010 at 03:38:37PM -0700, Mike Stump wrote:
> On Aug 20, 2010, at 8:53 PM, Jack Howarth wrote:
> >  The attached patch depreciates darwin7 and earlier. These darwin releases
> > only support stabs and haven't been properly maintained in quite some
> > time.
> 
> ?  Last time I ran darwin7 it worked just fine and it wasn't that long ago.  I don't believe it works any worse today does it?  In fact, last time I ran darwin6 it worked just fine as I recall.
> 
> > Okay for gcc trunk after adding any required documentation changes to the patch?
> 
> I don't think darwin1 - darwin5 are worth saving anymore, and don't care if they go, so that part of the patch would be fine.  darwin6 worked, last I knew and doesn't pose much a burden.  Likewise I don't think keeping support for 7 in the tree is burdensome, indeed the size of the below patch is rather small.  If someone accidentally breaks support for 6 or 7, it usually is trivial to get it to work again.  So, for example, the recent patches to rip out -lm, those should be inside a target os conditional of 10.4 or later.  If that were missing, should be trivial to add it.  I just don't see the value in removing what amounts to a single line to support darwin7.
> 
> Thoughts?

Mike,
   My original intention was to limit darwin support to those releases which support dwarf2.
While one might be able to compile on darwin6 or darwin7, the actual performance of that
configuration in the testsuite is completely untested. Considering the struggle it is to
keep darwin supported, it seems unwise to hinder current development for support of a
vanishingly small user base on darwin6 and darwin7.
            Jack


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]