Bug 731820

Summary: Review Request: kcm-qt-graphicssystem - Easily configure the standard Qt graphics system
Product: [Fedora] Fedora Reporter: Rex Dieter <rdieter>
Component: Package ReviewAssignee: Kalev Lember <kalevlember>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: kalevlember, kevin, notting, package-review
Target Milestone: ---Flags: kalevlember: fedora-review+
gwync: fedora-cvs+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: kcm-qt-graphicssystem-1.3-1.fc16 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2011-09-07 03:18:44 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:
Bug Depends On:    
Bug Blocks: 656997    

Comment 1 Kalev Lember 2011-08-20 07:33:37 UTC
Taking for review.

Koji scratch build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=3288557

> Requires: kdebase-runtime
Is this a plugin to one of the apps in kdebase-runtime? If so, the requires
should include %{?_isa} to make the dependency arch-dependant.

Comment 2 Kalev Lember 2011-08-20 07:45:11 UTC
Fedora review kcm-qt-graphicssystem-1.2-1.fc15.src.rpm  2011-08-20

+ OK
! needs attention

koji scratch build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=3288557

$ rpmlint kcm-qt-graphicssystem-1.2-1.fc16.src.rpm \
          kcm-qt-graphicssystem-1.2-1.fc16.x86_64.rpm \
          kcm-qt-graphicssystem-debuginfo-1.2-1.fc16.x86_64.rpm
kcm-qt-graphicssystem.src:2: W: mixed-use-of-spaces-and-tabs (spaces: line 2, tab: line 2)
3 packages and 0 specfiles checked; 0 errors, 1 warnings.

! rpmlint warning is harmless, but would be nice to make it happy
+ The package is named according to Fedora packaging guidelines
+ The spec file name matches the package base name
+ The package meets the Packaging Guidelines
+ The package is licensed with a Fedora approved license and meets the
  Licensing Guidelines.
+ The license field in the spec file matches the actual license
+ The package contains the license file (COPYING)
+ Spec file is written in American English
+ Spec file is legible
+ Upstream sources match sources in the srpm. md5sum:
  aef872cb2e892891a813026b0a8156f5  129817-kcm-qt-graphicssystem-1.2.tar.xz
  aef872cb2e892891a813026b0a8156f5  Download/129817-kcm-qt-graphicssystem-1.2.tar.xz
+ The package builds in koji
n/a ExcludeArch bugs filed
+ BuildRequires look sane
n/a The spec file MUST handle locales properly
n/a ldconfig in %post and %postun
+ Package does not bundle copies of system libraries
n/a Package isn't relocatable
+ No duplicated files in %files section
+ Permissions are properly set
+ Consistent use of macros
+ The package must contain code or permissible content
n/a Large documentation files should go in -doc subpackage
+ Files marked %doc should not affect package
n/a Header files should be in -devel
n/a Static libraries should be in -static
n/a Library files that end in .so must go in a -devel package
  It's a plugin package, not applicable.
n/a -devel must require the fully versioned base
+ Packages should not contain libtool .la files
n/a Packages containing GUI apps must include %{name}.desktop file
+ Directory ownership sane
+ Filenames are valid UTF-8


Remaining issues:
 - there are trailing tabs on lines 2 and 8 which rpmlint is complaining about
 - should probably use %{?_isa} to tighten the kdebase-runtime requires

These are all minor issues and can all be fixed before importing the package.

APPROVED

Comment 3 Rex Dieter 2011-08-20 20:34:56 UTC
New Package SCM Request
=======================
Package Name: kcm-qt-graphicssystem
Short Description: Easily configure the standard Qt graphics system
Owners: rdieter
Branches: f15 f16
InitialCC:

Comment 4 Gwyn Ciesla 2011-08-20 21:20:56 UTC
Git done (by process-git-requests).

Comment 5 Fedora Update System 2011-08-21 00:00:06 UTC
kcm-qt-graphicssystem-1.2-1.fc16 has been submitted as an update for Fedora 16.
https://admin.fedoraproject.org/updates/kcm-qt-graphicssystem-1.2-1.fc16

Comment 6 Fedora Update System 2011-08-22 18:34:22 UTC
kcm-qt-graphicssystem-1.2-1.fc16 has been pushed to the Fedora 16 testing repository.

Comment 7 Fedora Update System 2011-08-26 21:19:17 UTC
kcm-qt-graphicssystem-1.3-1.fc16 has been submitted as an update for Fedora 16.
https://admin.fedoraproject.org/updates/kcm-qt-graphicssystem-1.3-1.fc16

Comment 8 Fedora Update System 2011-09-07 03:18:39 UTC
kcm-qt-graphicssystem-1.3-1.fc16 has been pushed to the Fedora 16 stable repository.  If problems still persist, please make note of it in this bug report.