[Bug 427624] Review Request: R-qvalue - Q-value estimation for false discovery rate control

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

 



Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: R-qvalue - Q-value estimation for false discovery rate control


https://bugzilla.redhat.com/show_bug.cgi?id=427624





------- Additional Comments From tibbs@xxxxxxxxxxx  2008-01-07 23:42 EST -------
This does build but I get a huge number of errors building the documentation:

pdfTeX warning: pdflatex (file
/var/lib/texmf/fonts/map/pdftex/updmap/pdftex.map): ambiguous entry for
`ebbx10': font file present but not included, will be treated as font file not
present

and so on.   The resulting document is obviously not correct; compare
http://www.math.uh.edu/~tibbs/qvalue.pdf to 
http://bioconductor.org/packages/2.1/bioc/vignettes/qvalue/inst/doc/qvalue.pdf

I'm really not sure what the problem is, or what to do with this review.  Is
that problem significant enough to block this package?  This is really the only
thing I see that's problematic, but if it's a texlive problem and not a problem
with this package then I'm inclined to approve it, as texlive is breaking other
things in various ways.  What do you think?

I think we've established in the past that they mean LGPLv2+ for the license,
and that upstream is never going to pull their heads out and make even the
simplest of clarifications to the licensing information in their packages or on
their web pages.  So I'm not even going to complain about it.

* source files match upstream:
   90012885da8bc630e146a569da0d46549af6d230a9f852074b2e0f713b1e99e6  
   qvalue_1.12.0.tar.gz
* 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 not included upstream.
* latest version is being packaged.
* BuildRequires are proper.
* %clean is present.
* package builds in mock (rawhide, x86_64).
* package installs properly
* rpmlint has only the two expected complaints for R packages.
* final provides and requires are sane:
   R-qvalue = 1.12.0-1.fc9
  =
   /bin/sh
   R
* %check is present and all tests pass.
* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
* file permissions are appropriate.
* scriptlets are OK (R package registration)
* code, not content.
* %docs are not necessary for the proper functioning of the package.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]