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=600529 Pierre-YvesChibon <pingou@xxxxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |pingou@xxxxxxxxxxxx --- Comment #1 from Pierre-YvesChibon <pingou@xxxxxxxxxxxx> 2010-06-04 19:01:34 EDT --- As a quick look at the spec: - The URL is not correct - The BuildRequires to tetex-latex has to be updated - The Requires for post and postun can be removed - you might want to change the Requires from R to R-core - You have a double dependency on R in your BuildRequires - The comment on the %install is incorrect (also in %files) The rest look correct, I will do a deeper pre-review (as I can not sponsor you) later. -- 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@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review