[Bug 537454] Review Request: eclipse-emf-transaction - A model management layer for managing EMF resources
bugzilla at redhat.com
bugzilla at redhat.com
Sat Nov 21 23:56:40 UTC 2009
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=537454
Lubomir Rintel <lkundrak at v3.sk> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |ASSIGNED
CC| |lkundrak at v3.sk
AssignedTo|nobody at fedoraproject.org |lkundrak at v3.sk
Flag| |fedora-review?
--- Comment #1 from Lubomir Rintel <lkundrak at v3.sk> 2009-11-21 18:56:38 EDT ---
* Package name and version are in accordance with packaging guidelines
* SPEC file clean, legible and well-commented
* License tag correct, license packaged as %doc
* RPMLint happy
* a couple of warnings spelling errors in fc13, all of them false positives
* Filelist okay, for directories it uses and does now it depends on packages
that provide them
* Provides/Require list ok
* Built fine in mock (with dependency repository added)
Objections:
1.) Why do you check stuff out from CVS?
2.) Please don't pretty-format the Description, please use complete sentences.
It does not always look well in package management front-ends.
--
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.
More information about the package-review
mailing list