[Bug 241052] Review Request: R-Matrix - R module, Classes etc. for dense and sparse matrices and matrix ops

2008-02-12 Thread bugzilla
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-Matrix - R module, Classes etc. for dense and sparse 
matrices and matrix ops


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 241052] Review Request: R-Matrix - R module, Classes etc. for dense and sparse matrices and matrix ops

2008-02-11 Thread bugzilla
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-Matrix - R module, Classes etc. for dense and sparse 
matrices and matrix ops


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-11 20:34 EST ---
New Package CVS Request
===
Package Name: R-Matrix
Short Description: R module, Classes etc. for dense and sparse matrices and
matrix ops
Owners: tmoertel
Branches: F-7 F-8
InitialCC:
Cvsextras Commits: yes


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 241052] Review Request: R-Matrix - R module, Classes etc. for dense and sparse matrices and matrix ops

2008-02-11 Thread bugzilla
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-Matrix - R module, Classes etc. for dense and sparse 
matrices and matrix ops


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 241052] Review Request: R-Matrix - R module, Classes etc. for dense and sparse matrices and matrix ops

2008-02-11 Thread bugzilla
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-Matrix - R module, Classes etc. for dense and sparse 
matrices and matrix ops


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|177841  |
  nThis||
   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-02-11 21:32 EST ---
cvs done.
Removing NEEDSPONSOR.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 241052] Review Request: R-Matrix - R module, Classes etc. for dense and sparse matrices and matrix ops

2008-02-06 Thread bugzilla
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-Matrix - R module, Classes etc. for dense and sparse 
matrices and matrix ops


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-06 22:55 EST ---
Just a quick check-in ping.  :-)

Can we move the process forward somehow?  (Are you waiting on something from 
me?)

Cheers,
Tom


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 241052] Review Request: R-Matrix - R module, Classes etc. for dense and sparse matrices and matrix ops

2008-02-06 Thread bugzilla
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-Matrix - R module, Classes etc. for dense and sparse 
matrices and matrix ops


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-06 23:03 EST ---
Sorry, I never saw the sponsor request come through. You're sponsored now, and
have cvsextras approval. Follow the next steps on the Join link I pasted above.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 241052] Review Request: R-Matrix - R module, Classes etc. for dense and sparse matrices and matrix ops

2008-01-31 Thread bugzilla
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-Matrix - R module, Classes etc. for dense and sparse 
matrices and matrix ops


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 14:31 EST ---
Only items I see:

* Your changelog says you're at -4, but you forgot to increment release to 4.
Just make sure that your %changelog matches. :)

Good:

- rpmlint checks return:
R-Matrix.x86_64: W: incoherent-version-in-changelog 0.999375-4 0.999375-1.fc9
R-Matrix.x86_64: W: one-line-command-in-%post 
/usr/lib/rpm/R-make-search-index.sh
R-Matrix.x86_64: W: one-line-command-in-%postun 
/usr/lib/rpm/R-make-search-index.sh
R-Matrix-devel.x86_64: W: no-documentation
No blockers there, just be careful with matching n-v-r and changelog.

- package meets naming guidelines
- package meets packaging guidelines
- license (GPLv2+) OK, text in %doc, matches source (explanation in spec)
- spec file legible, in am. english
- source matches upstream
- package compiles on devel (x86_64)
- no missing BR
- no unnecessary BR
- no locales
- not relocatable
- owns all directories that it creates
- no duplicate files
- permissions ok
- %clean ok
- macro use consistent
- code, not content
- no need for -docs
- nothing in %doc affects runtime
- no need for .desktop file 
- devel package ok
- no .la files
- devel requires base package n-v-r 

APPROVED (just fix the release before committing).

Get your account here, and follow the rest of the steps:
http://fedoraproject.org/wiki/PackageMaintainers/Join#head-a601c13b0950a89568deafa65f505b4b58ee869b

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 241052] Review Request: R-Matrix - R module, Classes etc. for dense and sparse matrices and matrix ops

2008-01-31 Thread bugzilla
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-Matrix - R module, Classes etc. for dense and sparse 
matrices and matrix ops


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-31 20:20 EST ---
Thanks for the re-review.  I'll fix the changelog.

I already have an account.  I just need cvsextras approval to move forward.   
:-)

Note: This is the point where things got bogged down last time, so maybe
something is wrong with the state of my account (tmoertel).  Please let me
know if you find any problems.

Cheers,
Tom


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 241052] Review Request: R-Matrix - R module, Classes etc. for dense and sparse matrices and matrix ops

2008-01-30 Thread bugzilla
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-Matrix - R module, Classes etc. for dense and sparse 
matrices and matrix ops


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-30 19:36 EST ---
Just checking in:  Do I need to do anything else before review?  (I've linked to
the spec and srpm in the previous comment.)

Cheers,
Tom


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 241052] Review Request: R-Matrix - R module, Classes etc. for dense and sparse matrices and matrix ops

2008-01-26 Thread bugzilla
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-Matrix - R module, Classes etc. for dense and sparse 
matrices and matrix ops


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-26 12:11 EST ---
Great!  I tweaked the spec per the most-recent R-packaging guidelines and also
updated to Matrix version 0.999375-4.  New spec and SRPM:

http://community.moertel.com/rpms/fedora/8/SPECS/R-Matrix.spec
http://community.moertel.com/rpms/fedora/8/SRPMS/R-Matrix-0.999375-1.fc8.src.rpm

Please let me know if you need any changes.


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 241052] Review Request: R-Matrix - R module, Classes etc. for dense and sparse matrices and matrix ops

2008-01-22 Thread bugzilla
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-Matrix - R module, Classes etc. for dense and sparse 
matrices and matrix ops


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-22 07:41 EST ---
Jason, thanks for the ping.

I suspect that momentum on this packaging attempt got lost during the Extras
merge, which occurred around the time of the submission.

I'd like to resume the process.  What's the best way to restart it?

Cheers,
Tom


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 241052] Review Request: R-Matrix - R module, Classes etc. for dense and sparse matrices and matrix ops

2008-01-22 Thread bugzilla
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-Matrix - R module, Classes etc. for dense and sparse 
matrices and matrix ops


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-22 13:19 EST ---
Since it's been quite some time, I'd go ahead and check the package against the
latest R packaging guidelines at http://fedoraproject.org/wiki/Packaging/R, then
put links to the spec and srpm here, and then spot can give it another look. 
hopefully spot's still listening and still willing to sponsor you.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 241052] Review Request: R-Matrix - R module, Classes etc. for dense and sparse matrices and matrix ops

2008-01-22 Thread bugzilla
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-Matrix - R module, Classes etc. for dense and sparse 
matrices and matrix ops


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





--- Additional Comments From [EMAIL PROTECTED]  2008-01-22 20:55 EST ---
Yeah, please make sure the spec meets the R packaging guidelines, then post it
here (if any changes need to be made).

I'll give it another quick pass, and then sponsor you upon completion. :)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 241052] Review Request: R-Matrix - R module, Classes etc. for dense and sparse matrices and matrix ops

2008-01-20 Thread bugzilla
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-Matrix - R module, Classes etc. for dense and sparse 
matrices and matrix ops


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

Version|devel   |rawhide




--- Additional Comments From [EMAIL PROTECTED]  2008-01-20 17:31 EST ---
Did anything ever happen here?  The package is approved and sponsorship was
offered eight months ago, yet I don't see Tom in the cvsextras group and of
course the package is not in the distro.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 241052] Review Request: R-Matrix - R module, Classes etc. for dense and sparse matrices and matrix ops

2007-09-01 Thread bugzilla
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-Matrix - R module, Classes etc. for dense and sparse 
matrices and matrix ops


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

Product|Fedora Extras   |Fedora




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 241052] Review Request: R-Matrix - R module, Classes etc. for dense and sparse matrices and matrix ops

2007-05-25 Thread bugzilla
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-Matrix - R module, Classes etc. for dense and sparse 
matrices and matrix ops


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
   Flag||fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-05-25 12:32 EST ---
There is a very minor issue with this package, but it doesn't hold up approval.

- rpmlint says you've got mixed spaces and tabs, look at line 29. 

Fix that item before requesting builds, and we're good. Nice work with the
-devel package.

Good:

- rpmlint checks return:

W: R-Matrix mixed-use-of-spaces-and-tabs (spaces: line 29, tab: line 5)
W: R-Matrix-devel no-documentation
E: R-Matrix-devel only-non-binary-in-usr-lib

Safe to ignore the Error (all R packages do that), and the no-docs in -devel.

- package meets naming guidelines
- package meets packaging guidelines
- license (Distributable) OK, text in package, matches source (code is a mix of
MIT, GPL, LGPL, documented in spec)
- spec file legible, in am. english
- source matches upstream
- package compiles on devel (x86_64)
- no missing BR
- no unnecessary BR
- no locales
- not relocatable
- owns all directories that it creates
- no duplicate files
- permissions ok
- %clean ok
- macro use consistent
- code, not content
- no need for -docs
- nothing in %doc affects runtime
- no need for .desktop file
- devel package ok
- no .la files 

APPROVED. I'll also sponsor you, thanks for your good work.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 241052] Review Request: R-Matrix - R module, Classes etc. for dense and sparse matrices and matrix ops

2007-05-24 Thread bugzilla
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-Matrix - R module, Classes etc. for dense and sparse 
matrices and matrix ops


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





--- Additional Comments From [EMAIL PROTECTED]  2007-05-24 02:01 EST ---
I have revised the spec at the aforelinked-to location to fix the problems noted
earlier.

Note that the -devel package is built for the same arch as the main pacakge, but
it contains only .c and .h files, so rpmlint complains that it should be built
as noarch.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 241052] Review Request: R-Matrix - R module, Classes etc. for dense and sparse matrices and matrix ops

2007-05-24 Thread bugzilla
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-Matrix - R module, Classes etc. for dense and sparse 
matrices and matrix ops


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





--- Additional Comments From [EMAIL PROTECTED]  2007-05-24 02:09 EST ---
Note: I also updated the package to the recently released upstream version,
0.99875-1.  The corresponding SRPM can be downloaded here:

http://community.moertel.com/rpms/fedora/6/SRPMS/R-Matrix-0.99875-1.src.rpm


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 241052] Review Request: R-Matrix - R module, Classes etc. for dense and sparse matrices and matrix ops

2007-05-23 Thread bugzilla
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-Matrix - R module, Classes etc. for dense and sparse 
matrices and matrix ops


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





--- Additional Comments From [EMAIL PROTECTED]  2007-05-23 15:28 EST ---
Please note that this is my first package submission, and I am seeking a 
sponsor.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 241052] Review Request: R-Matrix - R module, Classes etc. for dense and sparse matrices and matrix ops

2007-05-23 Thread bugzilla
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-Matrix - R module, Classes etc. for dense and sparse 
matrices and matrix ops


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||177841
  nThis||




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 241052] Review Request: R-Matrix - R module, Classes etc. for dense and sparse matrices and matrix ops

2007-05-23 Thread bugzilla
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-Matrix - R module, Classes etc. for dense and sparse 
matrices and matrix ops


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





--- Additional Comments From [EMAIL PROTECTED]  2007-05-23 15:30 EST ---
Couple of minor items:

You probably want to have: BuildRequires: libgfortran-devel (you're linking to 
it)

The BuildRequires: R is correct, but BuildRequires: R-devel may be needed for
other CRAN packages (if they're looking for libR.pc), and it will drag in R as a
dependency. Might not be a bad idea to script it that way to be safe.

You need to mkdir -p %{buildroot}%{_libdir}/R/library before running R CMD
INSTALL into it. :)


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 241052] Review Request: R-Matrix - R module, Classes etc. for dense and sparse matrices and matrix ops

2007-05-23 Thread bugzilla
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-Matrix - R module, Classes etc. for dense and sparse 
matrices and matrix ops


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





--- Additional Comments From [EMAIL PROTECTED]  2007-05-23 15:34 EST ---
Also:

[EMAIL PROTECTED] ~]$ rpmlint
/home/spot/rpmbuild/RPMS/x86_64/R-Matrix-0.9975-1.fc7.x86_64.rpm
/home/spot/rpmbuild/SRPMS/R-Matrix-0.9975-1.fc7.src.rpm
/home/spot/rpmbuild/RPMS/x86_64/R-Matrix-debuginfo-0.9975-1.fc7.x86_64.rpm
W: R-Matrix devel-file-in-non-devel-package
/usr/lib64/R/library/Matrix/include/Matrix.h
W: R-Matrix devel-file-in-non-devel-package
/usr/lib64/R/library/Matrix/include/Matrix_stubs.c
W: R-Matrix devel-file-in-non-devel-package
/usr/lib64/R/library/Matrix/include/cholmod.h
W: R-Matrix incoherent-version-in-changelog 1.0-1 0.9975-1.fc7

The devel files look like they belong in a -devel subpackage. The changelog
entry needs to be corrected, your script seems to be spitting out a generic one
that doesn't match the package version (and is omitting your name).

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 241052] Review Request: R-Matrix - R module, Classes etc. for dense and sparse matrices and matrix ops

2007-05-23 Thread bugzilla
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-Matrix - R module, Classes etc. for dense and sparse 
matrices and matrix ops


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





--- Additional Comments From [EMAIL PROTECTED]  2007-05-23 16:24 EST ---
(In reply to comment #2)
 Couple of minor items:
 
 You probably want to have: BuildRequires: libgfortran-devel (you're linking 
 to it)

libgfortran-devel doesn't exist as such, it is linked automatically 
in when linking with gcc -lgfortran/gfortran. A

BuildRequires: gcc-gfortran

is missing however. 

Also looking at the log, it seems that during the link gcc
is used and there is a -lgfortran. In my recalling this is not 
what should be done, instead gfortran should be used during the
link and -lgfortran should be omitted. This may be for upstream, or
a R bug, however, I don't know exactly.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 241052] Review Request: R-Matrix - R module, Classes etc. for dense and sparse matrices and matrix ops

2007-05-23 Thread bugzilla
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-Matrix - R module, Classes etc. for dense and sparse 
matrices and matrix ops


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





--- Additional Comments From [EMAIL PROTECTED]  2007-05-23 17:26 EST ---
Tom spot Callaway: Is the preferred way of placing devel files in a -devel
subpackage documented anywhere?  (I didn't see anything on the topic in the
Packaging Guidelines.)  If there is a preferred way of doing it, I'd rather not
guess what it is.  :)

Thanks for your help.


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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