Bug 476406 - Review Request: sympow - Special Values of Symmetric Power Elliptic Curve L-Functions
Summary: Review Request: sympow - Special Values of Symmetric Power Elliptic Curve L-F...
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Jason Tibbitts
QA Contact: Fedora Extras Quality Assurance
Depends On:
TreeView+ depends on / blocked
Reported: 2008-12-14 10:00 UTC by Conrad Meyer
Modified: 2009-03-18 05:59 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2009-03-18 05:59:48 UTC
Type: ---
j: fedora-review+
kevin: fedora-cvs+

Attachments (Terms of Use)

Description Conrad Meyer 2008-12-14 10:00:39 UTC
Spec URL: http://konradm.fedorapeople.org/fedora/SPECS/sympow.spec
SRPM URL: http://konradm.fedorapeople.org/fedora/SRPMS/sympow-1.019-1.fc9.src.rpm
SYMPOW is a program for computing special values of symmetric power
elliptic curve L-functions.

Comment 1 Conrad Meyer 2008-12-14 10:02:39 UTC
Koji scratch build:


Comment 2 Conrad Meyer 2008-12-14 23:45:45 UTC
New URLs:


I just fixed the launcher script to be multilibs-safe.

Comment 3 Jason Tibbitts 2009-03-09 01:07:30 UTC
I don't believe the license of this package is actually BSD; at least, it's not a BSD variant I've seen before.  The third clause is different to me:

Redistribution and use in source and binary forms, with or without
modification, are permitted provided that the following conditions are met:
  * Redistribution of source code must retain the above copyright notice,
 this list of conditions and the following disclaimer.
  * Redistribution in binary form must reproduce the above copyright
 notice, this list of conditions and the following disclaimer in the
 documentation and/or other materials provided with the distribution.
  * If redistribution is done as a part of a compilation that has a more
 restrictive license (such as the GPL), then the fact that SYMPOW has
 a less restrictive license must be made clear to the recipient.
 For example, a line like (include bracketed text if SYMPOW is modified):
  "This compilation includes [a modification of] SYMPOW whose [original]
   code has a less-restrictive license than the entire compilation."
 should appear in a suitable place in the COPYING and/or LICENSE file.

Blocking FE-Legal for an opinion.  I wonder if the third clause might actually render the software GPL-incompatible.  And even then, I'd have to leave it to a lawyer to determine what "more restrictive" really means.

Comment 4 Tom "spot" Callaway 2009-03-12 16:15:42 UTC
Nah, just an odd BSD variant. Free and GPL compatible. Lifting FE-Legal.

Comment 5 Jason Tibbitts 2009-03-13 06:04:27 UTC
* source files match upstream.  sha256sum:
* package meets naming and versioning guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
* summary is OK.
* description is OK.
* dist tag is present.
* build root is OK.
* license field matches the actual license.
* license is open source-compatible.
* license text included in package.
* latest version is being packaged.
* BuildRequires are proper.
* compiler flags are appropriate.
* %clean is present.
* package builds in mock (rawhide, x86_64).
* package installs properly.
* debuginfo package looks complete.
* rpmlint is silent.
* final provides and requires are sane:
   sympow = 1.019-2.fc11
   sympow(x86-64) = 1.019-2.fc11

* no shared libraries are added to the regular linker search paths.
* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
* file permissions are appropriate.
* no generically named files
* code, not content.
* documentation is small, so no -doc subpackage is necessary.
* %docs are not necessary for the proper functioning of the package.
* no headers.
* no pkgconfig files.
* no static libraries.
* no libtool .la files.


Comment 6 Conrad Meyer 2009-03-13 06:10:45 UTC

New Package CVS Request
Package Name: sympow
Short Description: Special Values of Symmetric Power Elliptic Curve L-Functions
Owners: konradm
Branches: F-10

Comment 7 Kevin Fenzi 2009-03-16 02:09:24 UTC
cvs done.

Comment 8 Conrad Meyer 2009-03-18 05:59:48 UTC
Built in koji.


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