[Bug 510255] Review Request: cobertura - a Java tool for calculating the test coverage

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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





--- Comment #3 from Victor G. Vasilyev <victor.vasilyev@xxxxxxx>  2009-08-08 22:20:27 EDT ---
Kevin,

Thanks for review!

(In reply to comment #2)
> Issues:
> 
> 1. Looking at the files here, it looks like the License should be:
> ASL 1.1 and GPLv2+
> The ant parts seem to be under the ASL1.1, and the rest is GPLv2 or later.
> Unless there is something I am missing there... 
Of course, you are right.
http://cobertura.sourceforge.net/license.html 
I've admired by the juridical/technical trick of copyright holder, and forgot
about my first wrong assumption. Now, it is corrected.

BTW I hope, the spec file won't be considered as a thing that links software
with incompatible licenses. See here:
http://cobertura.svn.sourceforge.net/viewvc/cobertura/trunk/cobertura/src/net/sourceforge/cobertura/ant/package.html?view=markup

The second release is prepared for review:
Spec URL: http://victorv.fedorapeople.org/files/cobertura.spec
SRPM URL: http://victorv.fedorapeople.org/files/cobertura-1.9-2.fc12.src.rpm

Changes:
* The license tag is changed

The rpmlint tool shows 0 errors, 0 warnings against SRPM and all RPMs.

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

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]