Bug 584988
Summary: | Review Request: R-lme4 - Linear mixed-effects models using S4 classes | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | josef radinger <cheese> |
Component: | Package Review | Assignee: | Pierre-YvesChibon <pingou> |
Status: | CLOSED WONTFIX | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | fedora-package-review, notting, pingou |
Target Milestone: | --- | ||
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2010-11-16 14:42:25 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
josef radinger
2010-04-22 20:26:49 UTC
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 - Fix %changelog Please update the format of the changelog, the version-release should be on the same line as the changelog entree. - Fix Requires You might want to have Requires: R-core >= %{Rversion} instead of just R - Fix %check Although, this show that there are dependencies missing for this package: > Packages required but not available: > mlmRev MEMSS (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 (In reply to comment #2) > (In reply to comment #1) > > - 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. Please see if you can submit then for review & integration. We normally package both REQUIRES and SUGGESTS. See what is the strongest dependency between R-lme4 and R-mlmRev and keep that one (commenting indeed in the spec for the other one). Since you are already sponsored I will review the package I already looked at. I will try to do it in the coming days. you mean i shall create additional packages (the SUGGESTS) and decide which buildrequire really needs to be there to correctly build? eg: buildrequires from A ->B, but not from B->A (In reply to comment #4) > you mean i shall create additional packages (the SUGGESTS) and decide which > buildrequire really needs to be there to correctly build? yes, this is normally the way I/we do it. > eg: > buildrequires from A ->B, but not from B->A Then you can leave %check in one, and remove it (with explanation in the spec) from the other one. hm you also miss the BR tex(latex) here tex(latex) fixed, the rest still to come ping ? Shall we close or do you still want to go ? I am closing this, almost two months without reply. |