https://bugzilla.redhat.com/show_bug.cgi?id=1196925



--- Comment #5 from Alec Leamas <[email protected]> ---
Looks basically OK. Issues:

  -  Package requires other packages for directories it uses.
     Note: No known owner of /usr/share/maven-poms/jsemver
     Seems like a bug in xmvn-install (?). For the time being the
     easy fix might be to claim the directory using %dir.
  -  Please  remove all class and .jar files in %prep to make sure no
     prebuilt binaries are used in the build process. This is not
     strictly required by the GL in this case since the upstream is
     "clean" in this sense, but it's a god habit to avoid surprises
     when upstream is updated.
  -  PLease dont add the docs to both the -javadoc and the main package [1]

[1]:
https://fedoraproject.org/wiki/Packaging:Guidelines?rd=Packaging/Guidelines#DuplicateFiles

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list
[email protected]
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to