[Bug 706565] Review Request: sheepdog - The Sheepdog Distributed Storage System for KVM/QEMU

[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=706565

--- Comment #4 from Steven Dake <sdake@xxxxxxxxxx> 2011-05-22 12:16:08 EDT ---
Summary of issues needing correction in MUST requirements of package review:

not sure if this is a significant issue or not - I don't understand why rpmlint
is complaining.

MUST (1 warning): rpmlint must be run on the source rpm and all binary rpms the
build produces. The output should be posted in the review.[1] 

[sdake@beast Downloads]$ rpmlint -v sheepdog-0.2.3-1.fc14.src.rpm
sheepdog.src: I: checking
sheepdog.src: W: name-repeated-in-summary C Sheepdog
sheepdog.src: I: checking-url http://www.osrg.net/sheepdog (timeout 10 seconds)
sheepdog.src: I: checking-url
http://downloads.sourceforge.net/project/sheepdog/sheepdog/0.2.3/sheepdog-0.2.3.tar.gz
(timeout 10 seconds)
1 packages and 0 specfiles checked; 0 errors, 1 warnings.

MUST(FAIL): The License field in the package spec file must match the actual
license. [3]

The package license is GPLv2, not GPLv2+.  Looking through the source files, I
don't see any references to GPLv3 - if you can point me at them, I'll make sure
they get fixed upstream.


MUST (NEEDHELP): If the package does not successfully compile, build or work on
an architecture, then those architectures should be listed in the spec in
ExcludeArch. Each architecture listed in ExcludeArch MUST have a bug filed in
bugzilla, describing the reason that the package does not compile/build/work on
that architecture. The bug number MUST be placed in a comment, next to the
corresponding ExcludeArch line. [8]

I am not sure how to verify this.  Last I recall, we turned off builds of
dependent packages (corosync) on some architectures in Fedora, which would make
building this difficult on these arches :).  I will get back to you on this
point Monday.

-- 
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.
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review


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