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

Paulo Andrade <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
              Flags|fedora-review?              |fedora-review+



--- Comment #4 from Paulo Andrade <[email protected]> ---
Hi Gil,

  There are files under Apache and BSD license, so,
license probably should be changed to:
CDDL and ASL 2.0 and BSD
See for example:
sxc-sxc-0.8/sxc-runtime/src/main/java/org/metatype/sxc/util/Base64.java
sxc-sxc-0.8/sxc-xpath/src/test/java/org/metatype/sxc/xpath/StreaxPerformance.java

  Please comment about:
# https://gil.fedorapeople.org/dependency-report-plugin-1.0.3-1.fc19.src.rpm
# BuildRequires: mvn(org.codehaus.swizzle:dependency-report-plugin)
Does it mean it is an optional build requires dependency?

  An issue I see in most mvn related packages is
every subpackage having a %license line, because there
is no clean information about a single common subpackage
that all others require. It looks weird that every
subpackage have the line:
%license LICENSE.txt

Please correct the license tag before pushing to git,
otherwise I consider the package approved.

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