Re: [patch darwin] updated patch fixes PR41296, PR41245 (also re-enable checking removed in PR41224, PR41237)

[IainS <developer at sandoe-acoustics dot co dot uk>]

Yes for darwin8 and darwin9
(I don't have anything earlier install to check - but my guess is  
that they would be OK because of only using stabs).

you can, perhaps, answer for darwin10 - if you follow up my question  
in PR41245.

cheers,
Iain

On 8 Sep 2009, at 14:10, Jack Howarth wrote:

> Iain,
>    Do we really need to special case all of darwin
> to use ld instead of strip or just darwin8?
>                    Jack
>
> On Tue, Sep 08, 2009 at 10:13:03AM +0100, IainS wrote:
> > Hello,
> > further to
> >
> > http://gcc.gnu.org/ml/gcc-patches/2009-09/msg00467.html
> >
> > the attached patch resolves the residual issue with powerpc-apple-
> > darwin8 (--with-dwarf2).
> >
> > bootstrapped on powerpc-apple-darwin8 (stabs and dwarf2)
> > powerpc-apple-darwin9
> > i686-apple-darwin9.
> >
> > object comparison is restored and functional in each case.
> >
> > ---
> > this is the simplest change - one file.
> > One could equally create a "darwin-compare-debug" and alter the  
> > config
> > machinery to select that for darwin.
> >
> > Iain
>
> > Index: contrib/compare-debug
> > ===================================================================
> > --- contrib/compare-debug	(revision 151491)
> > +++ contrib/compare-debug	(working copy)
> > @@ -57,11 +57,20 @@
> >
> >  trap 'rm -f "$1.$suf1" "$2.$suf2"' 0 1 2 15
> >
> > -cp "$1" "$1.$suf1"
> > -strip "$1.$suf1"
> > +sys=$(uname -s)
> > +case $sys in
> > +	Darwin)
> > +	ld -S -x -r -no_uuid "$1" -o "$1.$suf1"
> > +	ld -S -x -r -no_uuid "$2" -o "$2.$suf2"
> > +	;;
> > +	*)
> > +	cp "$1" "$1.$suf1"
> > +	strip "$1.$suf1"
> >
> > -cp "$2" "$2.$suf2"
> > -strip "$2.$suf2"
> > +	cp "$2" "$2.$suf2"
> > +	strip "$2.$suf2"
> > +	;;
> > +esac
> >
> >  if cmp "$1.$suf1" "$2.$suf2"; then
> >    status=0