Spec URL: http://rpm.greysector.net/extras/tachyon.spec SRPM URL: http://rpm.greysector.net/extras/tachyon-0.97-1.src.rpm Description: A portable, high performance parallel ray tracing system supporting MPI and multithreaded implementations. Tachyon is built as a C callable library, which can be used with the included demo programs or within your own application. The distribution also includes a simple scene file parser front-end which reads a few different formats.
Well, * does fedora accept '-fomit-frame-pointer -ffast-math'? Usually these make debugging very difficult. * Does '-m486' make semse?
(In reply to comment #1) > * does fedora accept '-fomit-frame-pointer -ffast-math'? > Usually these make debugging very difficult. > * Does '-m486' make semse? Generally speaking, without having looking into the package: No and no.
-m486 gets overridden by subsequent -march. What's wrong with -fomit-frame-pointer and -ffast-math?
As I said in comment #1, both optimizations make debugging very difficult. * The former one (-fomit-frame-pointer) may make it impossible to debug by gdb. http://sources.redhat.com/gdb/current/onlinedocs/gdb_7.html * -ffast-math equals -fno-math-errno -funsafe-math-optimizations -fno-trapping-math -ffinite-math-only -fno-signaling-nans i.e. 1. this optimization means that you trust that there is no mistake like 'taking root of negative number' or so, which is very risky. 2. I don't know if this program needs accuracy, however, -funsafe-math-optimizations usually decreases precision of floating point calculation.
http://rpm.greysector.net/extras/tachyon.spec http://rpm.greysector.net/extras/tachyon-0.97-2.src.rpm - use only kosher CFLAGS - fix target macro setting
This packages meets : http://fedoraproject.org/wiki/Packaging/Guidelines http://fedoraproject.org/wiki/Packaging/ReviewGuidelines Okay. -------------------------------------------------- This package (tachyon) is APPROVED by me.
Imported and built for devel, FC-5 and FC-6 branches requested. Thanks for the review.
Package Change Request ====================== Package Name: tachyon New Branches: EL-4 EL-5
cvs done.
Package Change Request ====================== Package Name: tachyon New Branches: el6 Owners: rathann
Git done (by process-git-requests).