Bug 452832 - Review Request: perl-Math-FFT - Perl extension for Fast Fourier Transforms
Review Request: perl-Math-FFT - Perl extension for Fast Fourier Transforms
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
All Linux
medium Severity medium
: ---
: ---
Assigned To: Lubomir Rintel
Fedora Extras Quality Assurance
Depends On:
Blocks: F-Spacewalk
  Show dependency treegraph
Reported: 2008-06-25 08:05 EDT by Miroslav Suchý
Modified: 2009-05-22 12:54 EDT (History)
4 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2008-07-01 07:35:33 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
lkundrak: fedora‑review+
kevin: fedora‑cvs+

Attachments (Terms of Use)

  None (edit)
Description Miroslav Suchý 2008-06-25 08:05:38 EDT
Spec URL: http://miroslav.suchy.cz/fedora/perl-Math-FFT/perl-Math-FFT.spec
This module implements some algorithms for calculating
Fast Fourier Transforms for one-dimensional data sets of size 2^n.

This is my first package, and I'm seeking a sponsor.
Comment 1 Dan Horák 2008-06-25 08:28:53 EDT
Please, no generated specs by cpan2rpm. When you need a spec generator, then you
can use rpmdev-newspec from rpmdevtools package. It has a special mode for Perl
packages and works very well.

See the following links:
Comment 2 Lubomir Rintel 2008-06-25 08:53:42 EDT
Miroslav: You may prefer using cpanspec; the generated SPEC files mostly comply
with the Fedora guidelines, you'll probably have to do just minor adjustments
such as verifying a License, and possibly adding missing dependencies.
Comment 3 Miroslav Suchý 2008-06-25 16:03:00 EDT
OK. I used cpanspec and it indeed produce much better spec (although not
URL are the same:
Spec URL: http://miroslav.suchy.cz/fedora/perl-Math-FFT/perl-Math-FFT.spec

Can you try it now, please?
Comment 4 Lubomir Rintel 2008-06-27 09:46:55 EDT
Miroslav: Thanks for the updated package. Please note that relocatable packages
are deprecated for Fedora [1], so not being relocatable is not an issue at all.

[1] http://fedoraproject.org/wiki/Packaging/Guidelines#Relocatable_packages

RPMLint is quiet.
Spec file is clean legible.

1.) Licensing

Seems like the copyright status of several pieces of code is not known. As per
module's documentation [2], fft4g.c is not necessarily covered by this module's
license. According to original author's [3] website, the file is public domain.
Please do the following:

* Add " and Public Domain" to the License: tag, and accompany it with a comment
about which parts of the code are covered by which license.

* Encourage the upstream author to include verbatim copy of the copyright
disclaimer (as in [3]) in this package's documentation.

[2] http://search.cpan.org/dist/Math-FFT/FFT.pm#COPYRIGHT
[3] http://www.kurims.kyoto-u.ac.jp/~ooura/fft.html

arrays.c file that comes from PGPLOT package is covered by the very same license
as this module's code [4]. You are encouraged to ask this package's upstream to
reflect this fact in their documentation, and please add appropriate comment to
the License: tag in your spec file as well.

[4] http://search.cpan.org/src/KGB/PGPLOT-2.20/LICENSE

2.) Package does not build due to lacking dependencies

Addding BuildRequires: perl(ExtUtils::MakeMaker.pm) might be enough.
Comment 5 Lubomir Rintel 2008-06-28 17:14:17 EDT
To be a bit more clear; only the license tag with the comment and the missing
dependecy are considered blockers with respect to guidelines.
Comment 6 Miroslav Suchý 2008-06-30 07:33:29 EDT
Spec and srpm updated.
Comment 7 Lubomir Rintel 2008-06-30 07:44:36 EDT
License:        (GPL+ or Artistic) and Aritistic and Public Domain

This is still incorrect. There are no bits that are covered just by Artistic
license (that would not be permissible for Fedora anyway). I think this should
say "(GPL+ or Artistic) and Public Domain".

Given it's trivial to fix, this won't warrant a blocker.

Comment 8 Miroslav Suchý 2008-06-30 08:45:44 EDT
Oh, I misinterpreted the arrays.c. 
Comment 9 Miroslav Suchý 2008-06-30 09:07:36 EDT
New Package CVS Request
Package Name: perl-Math-FFT
Short Description: Perl extension for Fast Fourier Transforms
Owners: msuchy
Branches: devel
Cvsextras Commits: yes
Comment 10 Kevin Fenzi 2008-06-30 12:27:30 EDT
cvs done.
Comment 11 Miroslav Suchý 2008-07-01 07:35:33 EDT
Successfully built:
Comment 12 Michael Stahnke 2008-07-09 17:29:05 EDT
Can you please push the updates to EL4, EL5, and possibly F9?  This would allow
the Spacewalk project to start building against this package and remove it from
their separate GIT/yum repo. 
Comment 13 Miroslav Suchý 2008-07-29 07:48:42 EDT
Package Change Request
Package Name: perl-Math-FFT
New Branches: F-9 EL-4 EL-5

Comment 14 Kevin Fenzi 2008-07-29 11:43:39 EDT
cvs done.

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