[Bug 584988] Review Request: R-lme4 - Linear mixed-effects models using S4 classes

[bugzilla at redhat.com]

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


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

--- Comment #2 from josef radinger <cheese at nosuchhost.net> 2010-04-30 15:05:02 EDT ---
(In reply to comment #1)
> Having a quick look at it:
> 
> - Fix version
>   The version of this package is 0.999375.33 not 0.999375 cf 
>   http://fedoraproject.org/wiki/Packaging:R#R_version
agreed

> - Fix %changelog
>   Please update the format of the changelog, the version-release should be on
> the same line as the changelog entree.
according to http://fedoraproject.org/wiki/Packaging:Guidelines#Changelogs
allowed.

> - Fix Requires
>   You might want to have Requires: R-core >= %{Rversion} instead of just R
agreed

> - Fix %check
> Although, this show that there are dependencies missing for this package:
> > Packages required but not available:
> >  mlmRev MEMSS    
those are only suggestions for the check and mlmRev has a dependency on R-lme4
-> circular dependency. should have noted that in the spec.

new SRPM:
http://www.nosuchhost.net/~cheese/fedora/packages/devel/R-lme4-0.999375.33-4.fc12.src.rpm

-- 
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.