Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=912686

Stanislav Ochotnicky <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected]

--- Comment #2 from Stanislav Ochotnicky <[email protected]> ---
So few issues first:
- buildroot is no longer needed
- Source0 is unversioned so it can change and it's going to be hard to notice.
Contacting upstream about properly versioning their releases would be a good
idea
- related to above to ensure stable tarball whenever it's recreated it would be
better to use SVN directly and export exact revision & make a tarball out of it
(with comment explaining the steps)
- java/lib directory contains bundled libraries (icu4j, junit) these need to be
removed during %prep and proper BR added

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=HfAsMUKHCO&a=cc_unsubscribe
_______________________________________________
package-review mailing list
[email protected]
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to