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"
Paul.
--
fedora-extras-list mailing list
fedora-extras-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/fedora-extras-list
