[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: rpms/sextractor/FC-4 sextractor.spec,1.2,1.3



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 freenet de> 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.



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]