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

--- Comment #3 from Alexander Kurtakov <[email protected]> 2010-11-01 
07:32:57 EDT ---
(In reply to comment #2)
> === Issues ===
> 0. drop gcj
Fixed.
> 1. rpmlint issues
Fixed.
> 2. Buildroot
Removed.
> 3. License: Upstream (http://www.jdom.org/docs/faq.html#a0030) says, it's 'an
> Apache-style open source license'. Please check the license tag (not sure 
> which
> version or if we have an appropriate tag at all)
Changed to ASL
> 4. Check BRs/Rs for jpackage-utils (and java/java-devel)
Fixed
> 5. Check javadoc Rs for jpackage-utils
Fixed.
> 6. global -> define
Removed.
> 7. R(post) and R(postun) jpackage-utils missing (for the update_maven_depmap
> macro)
Fixed.
> 8. The pom file is installed correctly, but specified as additional source 
> from
> maven repo. Please use the one included in the source tarball or leave a
> comment why that's impossible.
I can't find a pom included
> 
> === Final Notes ===
> 1. A short comment about the patches would be nice.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
_______________________________________________
package-review mailing list
[email protected]
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to