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

2010-11-17 Thread bugzilla
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

Pierre-YvesChibon pin...@pingoured.fr changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||WONTFIX
   Flag|needinfo?(che...@nosuchhost |
   |.net)   |
Last Closed||2010-11-16 09:42:25

--- Comment #9 from Pierre-YvesChibon pin...@pingoured.fr 2010-11-16 09:42:25 
EST ---
I am closing this, almost two months without reply.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


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

2010-09-21 Thread bugzilla
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

Pierre-YvesChibon pin...@pingoured.fr changed:

   What|Removed |Added

   Flag||needinfo?(che...@nosuchhost
   ||.net)

--- Comment #8 from Pierre-YvesChibon pin...@pingoured.fr 2010-09-21 14:11:45 
EDT ---
ping ? Shall we close or do you still want to go ?

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


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

2010-05-06 Thread bugzilla
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 #6 from Pierre-YvesChibon pin...@pingoured.fr 2010-05-06 14:08:54 
EDT ---
hm you also miss the BR tex(latex) here

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


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

2010-05-03 Thread bugzilla
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 #4 from josef radinger che...@nosuchhost.net 2010-05-03 07:46:00 
EDT ---
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

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


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

2010-05-03 Thread bugzilla
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 #5 from Pierre-YvesChibon pin...@pingoured.fr 2010-05-03 08:03:50 
EDT ---
(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.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


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

2010-04-30 Thread bugzilla
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 che...@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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


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

2010-04-30 Thread bugzilla
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

Pierre-YvesChibon pin...@pingoured.fr changed:

   What|Removed |Added

 AssignedTo|nob...@fedoraproject.org|pin...@pingoured.fr

--- Comment #3 from Pierre-YvesChibon pin...@pingoured.fr 2010-04-30 16:59:09 
EDT ---
(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.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


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

2010-04-28 Thread bugzilla
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

Pierre-YvesChibon pin...@pingoured.fr changed:

   What|Removed |Added

 CC||pin...@pingoured.fr

--- Comment #1 from Pierre-YvesChibon pin...@pingoured.fr 2010-04-28 05:49:03 
EDT ---
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

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review