[Bug 1063042] Review Request: rubygem-em-websocket - EventMachine based WebSocket server

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

 



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



--- Comment #2 from Eduardo Mayorga <e@xxxxxxxxxxxxxxxx> ---
OK - Package meets naming and packaging guidelines
OK - Spec file matches base package name.
OK - Spec has consistant macro usage.
OK - Meets Packaging Guidelines.
OK - License
OK - License field in spec matches
OK - License file included in package
OK - Spec in American English
OK - Spec is legible.
OK - Sources match upstream md5sum:
b6255d70a4afa8222d5c6c5a05ef12f4  em-websocket-0.5.0.gem
b6255d70a4afa8222d5c6c5a05ef12f4  em-websocket-0.5.0.gem.new
OK - Package needs ExcludeArch
OK - BuildRequires correct
OK - Package is code or permissible content.
OK - Doc subpackage needed/used.
OK - Packages %doc files don't affect runtime.
OK - Package obey's FHS standard

FAILS - Package compiles and builds on at least one arch.
OK - Package has no duplicate files in %files.
OK - Package doesn't own any directories other packages own.
OK - Package owns all the directories it creates.
FAILS - No rpmlint output.
FAILS - final provides and requires are sane:
(include output of for i in *rpm; do echo $i; rpm -qp --provides $i; echo =;
rpm -qp --requires $i; echo; done
manually indented after checking each line.  I also remove the rpmlib junk and
anything provided by glibc.)

SHOULD Items:

FAILS - Should build in mock.
FAILS - Should build on all supported archs
FAILS - Should function as described.
OK - Should have dist tag
OK - Should package latest version

Issues:

1. Group tag is deprecated since F17, you shouldn't use it anymore.
2. The package doesn't build in mock (check %check section).
3. Provides tag is missing. You need this:
Provides: rubygem(%{gem_name}) = %{version}
3. em-http-request seems to be a Require, and it isn't in Fedora
4. You must remove binary extension sources and build leftovers. Add this:
# Remove the binary extension sources and build leftovers.
rm -f %{buildroot}/%{gem_instdir}/Rakefile
rm -f %{buildroot}/%{gem_instdir}/em-websocket.gemspec

rpmlint output:

$ rpmlint rubygem-em-websocket-0.5.0-1.fc19.src.rpm
rubygem-em-websocket.src: E: invalid-spec-name
rubygem-em-websocket.src:56: W: mixed-use-of-spaces-and-tabs (spaces: line 56,
tab: line 51)
1 packages and 0 specfiles checked; 1 errors, 1 warnings.

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