Bug 1150

Summary: Review request: west-chamber-kmod - Kernel module (kmod) for west-chamber
Product: Package Reviews Reporter: Caius 'kaio' Chance <kaio>
Component: Review RequestAssignee: Chen Lei <supercyper1>
Status: RESOLVED FIXED    
Severity: normal CC: rpmfusion-package-review, supercyper1
Priority: P5    
Version: Current   
Hardware: All   
OS: GNU/Linux   
URL: http://code.google.com/p/scholarzhang/
namespace:
Bug Depends on:    
Bug Blocks: 4    

Description Caius 'kaio' Chance 2010-04-05 16:09:06 CEST
Spec: http://kaio.fedorapeople.org/pkgs/west-chamber-kmod.spec
SRPM: http://kaio.fedorapeople.org/pkgs/west-chamber-kmod-0.0.1-3.20100405svn.fc12.src.rpm

West-chamber is extensions named after the famous Chinese ancient friction -
Romance of the West Chamber for iptables.

This is not eligible to be included in Fedora because this is a kernel module.

$ rpmlint SRPMS/west-chamber-kmod-0.0.1-3.20100405svn.fc12.src.rpm RPMS/x86_64/kmod-west-chamber-*
west-chamber-kmod.src: W: spelling-error %description -l en_US iptables -> potables, portables, timetables
west-chamber-kmod.src: W: invalid-url Source0: http://scholarzhang.googlecode.com/files/west-chamber-20100405.tar.bz2 HTTP Error 404: Not Found
kmod-west-chamber.x86_64: W: spelling-error Summary(en_US) Metapackage -> Meta package, Meta-package, Prepackage
kmod-west-chamber.x86_64: W: no-documentation
kmod-west-chamber-2.6.32.10-90.fc12.x86_64.x86_64: W: spelling-error Summary(en_US) fc -> cf, dc, f
kmod-west-chamber-2.6.32.10-90.fc12.x86_64.x86_64: W: summary-not-capitalized C west-chamber kernel module(s) for 2.6.32.10-90.fc12.x86_64
kmod-west-chamber-2.6.32.10-90.fc12.x86_64.x86_64: W: spelling-error %description -l en_US fc -> cf, dc, f
kmod-west-chamber-2.6.32.10-90.fc12.x86_64.x86_64: W: unstripped-binary-or-object /lib/modules/2.6.32.10-90.fc12.x86_64/extra/west-chamber/xt_CUI.ko
kmod-west-chamber-2.6.32.10-90.fc12.x86_64.x86_64: W: unstripped-binary-or-object /lib/modules/2.6.32.10-90.fc12.x86_64/extra/west-chamber/xt_gfw.ko
kmod-west-chamber-2.6.32.10-90.fc12.x86_64.x86_64: W: unstripped-binary-or-object /lib/modules/2.6.32.10-90.fc12.x86_64/extra/west-chamber/xt_ZHANG.ko
kmod-west-chamber-2.6.32.10-90.fc12.x86_64.x86_64: W: no-documentation
kmod-west-chamber-2.6.32.10-90.fc12.x86_64.x86_64: E: kernel-modules-not-in-kernel-packages /lib/modules/2.6.32.10-90.fc12.x86_64/extra/west-chamber/xt_CUI.ko
kmod-west-chamber-2.6.32.10-90.fc12.x86_64.x86_64: E: kernel-modules-not-in-kernel-packages /lib/modules/2.6.32.10-90.fc12.x86_64/extra/west-chamber/xt_gfw.ko
kmod-west-chamber-2.6.32.10-90.fc12.x86_64.x86_64: E: kernel-modules-not-in-kernel-packages /lib/modules/2.6.32.10-90.fc12.x86_64/extra/west-chamber/xt_ZHANG.ko
kmod-west-chamber-2.6.32.10-90.fc12.x86_64.x86_64: E: kernel-modules-not-in-kernel-packages /lib/modules/2.6.32.10-90.fc12.x86_64/extra/west-chamber
3 packages and 0 specfiles checked; 4 errors, 11 warnings.

Spellings are correct.
URL is correct which I could download without problem.
Summary is capitalized.
Binary package has binary is correct.
Documentation is available on wiki of project page.
Kernel modules are not default ones so located in other directories.

This is my first RPM Fusion package.

I am a Fedora sponsored packager.
Comment 1 Chen Lei 2010-04-06 05:05:54 CEST
 formal review here:

 +:ok, =:needs attention, -:needs fixing

MUST Items:
[+] MUST: rpmlint must be run on every package.
<<output if not already posted>>
[+] MUST: The package must be named according to the Package Naming Guidelines.
[+] MUST: The spec file name must match the base package %{name}
[+] MUST: The package must meet the Packaging Guidelines. [FIXME?: covers this
list and more]
[+] MUST: The package must be licensed with a Fedora approved license and meet
the Licensing Guidelines.
[+] MUST: The License field in the package spec file must match the actual
license.
[+] MUST: If (and only if) the source package includes the text of the
license(s) in its own file, then that file, containing the text of the
license(s) for the package must be included in %doc.
[+] MUST: The spec file must be written in American English.
[+] MUST: The spec file for the package MUST be legible.
[+] MUST: The sources used to build the package must match the upstream source,
as provided in the spec URL.
<<md5sum checksum>>38d1503a1df5d8b5fa6823e3e0ed4982
[+] MUST: The package must successfully compile and build into binary rpms on
at least one supported architecture.
[+] MUST: If the package does not successfully compile, build or work on an
architecture, then those architectures should be listed in the spec in
ExcludeArch.
[+] MUST: All build dependencies must be listed in BuildRequires
[+] MUST: A package must own all directories that it creates. If it does not
create a directory that it uses, then it should require a package which does
create that directory.
[+] MUST: Each package must have a %clean section, which contains rm -rf
%{buildroot} (or $RPM_BUILD_ROOT).
[N/A] MUST: The package must contain code, or permissible content. This is
described in detail in the code vs. content section of Packaging Guidelines.

================
*** APPROVED ***
================ 
Comment 2 Chen Lei 2010-04-06 05:10:36 CEST
Note:

Please set "%define buildforkernels current" instead of "%define buildforkernels newest" the first time you import a module into rpmfusion.

See  http://rpmfusion.org/Packaging/KernelModules/Kmods2#head-998f6964b3188f6a2ac6e757ff14f0511b79ffb5
Comment 3 Caius 'kaio' Chance 2010-04-06 15:10:01 CEST
Package CVS request
======================
Package Name: west-chamber-kmod
Short Description: Kernel module (kmod) for west-chamber
Owners: kaio supercyper
Branches: F-12 F-13
----------------------
License tag: free