Bug 253322 - Review Request: tomsfastmath - A fast public domain, open source, large integer arithmetic library
Review Request: tomsfastmath - A fast public domain, open source, large integ...
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
All Linux
medium Severity medium
: ---
: ---
Assigned To: Nobody's working on this, feel free to take it
Fedora Extras Quality Assurance
Depends On:
Blocks: FE-DEADREVIEW 253324
  Show dependency treegraph
Reported: 2007-08-17 16:41 EDT by Noah Kantrowitz
Modified: 2008-01-27 17:16 EST (History)
4 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2008-01-27 17:16:07 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)

  None (edit)
Description Noah Kantrowitz 2007-08-17 16:41:09 EDT
Spec URL: http://dev.laptop.org/~coderanger/rpm/tomsfastmath/tomsfastmath.spec
SRPM URL: http://dev.laptop.org/~coderanger/rpm/tomsfastmath/tomsfastmath-0.12-1.fc8.src.rpm
Description: TomsFastMath is a fast public domain, open source, large integer arithmetic library written in portable ISO C. It is a port of LibTomMath with optional support for inline assembler multipliers. It has the following features.
Comment 1 Jason Tibbitts 2007-11-07 21:04:25 EST
I just did a quick build of this and found some issues:

The debuginfo package seems broken; it doesn't contain any source.

The proper set of CFLAGS aren't used.  You can get them in with
  export CFLAGS="%{optflags}"
at the very beginning of %build, but that still doesn't solve the broken
debuginfo problem.  Plus this package adds some additional options like -O3
which are generally frowned upon.  Setting IGNORE_SPEED does get rid of those,
but of course you should benchmark to see if it actually makes a difference.

The -devel package doesn't own /usr/include/tomsfastmath/.

It would be nice to package the changes.txt file as documentation, probably in
the -devel package.
Comment 2 Mamoru TASAKA 2007-12-15 11:53:05 EST
Comment 3 Michael Stone 2007-12-15 15:06:47 EST
Noah just passed this bug over to me, so I'll reply with a new spec and srpm as
soon as I've learned enough to fix the packaging bugs Jason found. Thanks for
checking in.
Comment 4 Mamoru TASAKA 2008-01-17 09:29:50 EST
Again ping?
Comment 5 Jason Tibbitts 2008-01-27 15:06:33 EST
I think it's time to close this one.  Unless, of course, Mamoru would like to
keep it open or whoever is driving this now would like to give us an updated
package for review.
Comment 6 Michael Stone 2008-01-27 16:47:18 EST
Closing it sounds wise to me. If I'm ever in a position where I'm able to
maintain the package, I'll reopen it. Please select whatever resolution seems

Note You need to log in before you can comment on or make changes to this bug.