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=773043

--- Comment #3 from Alexander Kurtakov <akurt...@redhat.com> 2012-01-12 
06:47:42 EST ---
Initial issues found:
1. There is manually added license file. We never do this in Fedora please drop
it.
2. post/postun update_maven_depmap are not needed
3. rm -fr %{buildroot} in install iss not needed
4. rpm version is 2.2.3 but the link for the source points to 2.2.4 
5. please use the canonical URL tag not Url
6. non-standard-group Development/Libraries/Java - there is no such rpm group,
probably better to use Development/Libraries
7. please use your full name in the changelog

-- 
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

Reply via email to