Bug 613067
Summary: | Review Request:spice - Implements the SPICE protocol | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Gerd Hoffmann <kraxel> |
Component: | Package Review | Assignee: | Matthias Clasen <mclasen> |
Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | fedora-package-review, mclasen, notting, supercyper1 |
Target Milestone: | --- | Flags: | mclasen:
fedora-review+
kevin: fedora-cvs+ |
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2010-07-13 12:39:07 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
Gerd Hoffmann
2010-07-09 15:57:38 UTC
Note that this package depends on celt051 (bug #612979) and spice-protocol (bug #612943). Builds fine in mock. rpmlint output: rpmlint /var/lib/mock/fedora-rawhide-x86_64/result/*.rpm spice.src: W: name-repeated-in-summary C SPICE spice-client.x86_64: W: no-manual-page-for-binary spicec 5 packages and 0 specfiles checked; 0 errors, 2 warnings. Looking at the license situation: - the included COPYING is LGPL - the majority of sources seem to say LGPL, some say GPL, and then there's eg server/jpeg_encoder.h which looks BSD - the spec file says GPL package name: ok spec file name: ok packaging guidelines: small cleanups are possible if the package is only for F13+: - remove BuildRoot - remove initial rm -rf in %install - ditch %clean things that need fixing: - ExclusiveArch: should have a comment explaining why the package only works on some arches and ideally a bug ref. See https://fedoraproject.org/wiki/Packaging/Guidelines#Architecture_Build_Failures - the CFLAGS munging needs justification in a comment. Why remove -Wall ? and in particular, why remove _FORTIFY_SOURCE ? See https://fedoraproject.org/wiki/Packaging/Guidelines#Compiler_flags - if libspice-server.a must be packaged, it needs to go into a -static subpackage. See https://fedoraproject.org/wiki/Packaging/Guidelines#Packaging_Static_Libraries license: ok license field/license file: see previous comment spec file language: ok spec file readable: ok buildable: ok ExcludeArch: see above BuildRequires: ok locale handling: ok ldconfig: ok system libraries: ok relocatable: ok directory ownership: ok duplicate files: ok file permissions: ok. (pedants prefer the 4-argument form of %defattr) macro use: ok permissable content: ok large docs: ok %doc content: ok headers: ok shared libs: ok static libs: see above devel deps: ok libtool archives: ok gui apps: ok file ownership: ok utf8 filenames: ok Created bug #613529 for the portability issues which make spice x86 only. Updated packages + specfile uploaded to http://kraxel.fedorapeople.org/review/spice/ Setting CFLAGS as you do now is probably a nop, looking at the definition of %configure: %configure \ CFLAGS="${CFLAGS:-%optflags}" ; export CFLAGS ; \ CXXFLAGS="${CXXFLAGS:-%optflags}" ; export CXXFLAGS ; \ ... but that is not a big deal. The rest looks fine now. Approved New Package CVS Request ======================= Package Name: spice Short Description: Implements the SPICE protocol Owners: kraxel alexl Branches: InitialCC: CVS done (by process-cvs-requests.py). %changelog * Mon Jul 12 2010 Gerd Hoffmann <kraxel> - 0.5.2-3 - %configure handles CFLAGS automatically, no need to fiddle with %{optflags} manually. should be changed to * Mon Jul 12 2010 Gerd Hoffmann <kraxel> - 0.5.2-3 - %%configure handles CFLAGS automatically, no need to fiddle with %%{optflags} manually. See http://koji.fedoraproject.org/koji/buildinfo?buildID=183372 rawhide builds are done. |