On Sat, 2006-07-22 at 13:27 +0100, Paul Howarth wrote:
> On Sat, 2006-07-22 at 05:04 +0200, Ralf Corsepius wrote:
> > On Fri, 2006-07-21 at 15:12 -0500, Jason L Tibbitts III wrote:
> > > My apologies; I'm resending this because I mistyped the address of the
> > > SExtractor maintainer.
> > >
> > > >>>>> "RC" == Ralf Corsepius <rc040203@xxxxxxxxxx> writes:
> > >
> > > RC> On Fri, 2006-07-21 at 10:24 -0700, Sergio Pascual wrote:
> > > >> %build -%configure +%configure CFLAGS="${CFLAGS} -funroll-loops
> > > >> -fomit-frame-pointer -O1
> > > RC> REVERT this change IMMEDIATELY.
> > >
> > > RC> You are breaking debug infos.
> > >
> > > Yes, this is not good.
> > During a review, this would be a BLOCKER and would cause a package not
> > to be accepted.
> >
> > I am not willing to let maintainers get away with such stuff
> > post-review.
> >
> > > For reference, the bug which prompted this is
> > > here:
> > >
> > > https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=199700
> > >
> > > The problem is that the code simply does not function with the default
> > > Fedora optimization flags. I am not familiar with the software and
> > > have no idea whether this points to a GCC problem or just poorly
> > > written code. Someone should investigate the minimum change to the
> > > stock Fedora cflags which permit this software to work properly, and
> > > then investigate why the behavior differs.
> > >
> > > Is there a standard method for overriding a single flag in
> > > %{optflags}?
> >
> > The way he does it is the way how things are supposed to work.
>
> A similar approach is used is openais:
> (http://cvs.fedora.redhat.com/viewcvs/devel/openais/openais.spec?view=markup)
>
> # -O3 required for performance reasons
> CFLAGS="$(echo '%{optflags}' | sed -e 's/-O[0-9]*//') -O3"
> make CFLAGS="$CFLAGS"
Bummer. This is even worse.
Again, the way this package's maintainer set CFLAGS is OK, but what he
does is embarrassing. Seems as if he doesn't know what he is doing.
I am going to file a request to packaging committee to ban -O3 and
-fomit-frame-pointer
Ralf
--
fedora-extras-list mailing list
fedora-extras-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/fedora-extras-list
