Bug 920861

Summary: Review Request: gnome-tetravex - GNOME Tetravex game
Product: [Fedora] Fedora Reporter: Tanner Doshier <doshitan>
Component: Package ReviewAssignee: Kalev Lember <kalevlember>
Status: CLOSED NEXTRELEASE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 19CC: kalevlember, notting, package-review
Target Milestone: ---Flags: kalevlember: fedora-review+
gwync: fedora-cvs+
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-04-30 17:10:00 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Tanner Doshier 2013-03-12 22:13:33 UTC
Spec URL: http://doshitan.com/tmp/gnome-tetravex/gnome-tetravex.spec
SRPM URL: http://doshitan.com/tmp/gnome-tetravex/gnome-tetravex-3.7.90-1.fc19.src.rpm
Description: This is the GNOME Tetravex game

gnome-tetravex.x86_64: E: incorrect-fsf-address /usr/share/doc/gnome-tetravex-3.7.90/COPYING
3 packages and 0 specfiles checked; 1 errors, 0 warnings.

Comment 1 Tanner Doshier 2013-04-02 22:45:52 UTC
- Update to 3.8.0
- Add high contrast icons

Spec URL: http://doshitan.com/tmp/gnome-tetravex/gnome-tetravex.spec
SRPM URL: http://doshitan.com/tmp/gnome-tetravex/gnome-tetravex-3.8.0-1.fc19.src.rpm

Comment 2 Kalev Lember 2013-04-14 00:44:09 UTC
Taking for review.

Koji scratch build:

Comment 3 Kalev Lember 2013-04-14 00:53:52 UTC
Fedora review gnome-tetravex-3.8.0-1.fc19.src.rpm 2013-04-13

+ OK
! needs attention

rpmlint output:
$ rpmlint gnome-tetravex \
          gnome-tetravex-debuginfo-3.8.0-1.fc19.x86_64.rpm \
gnome-tetravex.x86_64: W: obsolete-not-provided gnome-games-gnotravex
gnome-tetravex.x86_64: E: incorrect-fsf-address /usr/share/doc/gnome-tetravex-3.8.0/COPYING
3 packages and 0 specfiles checked; 1 errors, 1 warnings.

+ Rpmlint warnings / errors are harmless and can be ignored
+ The package is named according to Fedora packaging guidelines
+ The spec file name matches the base package 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

  As with lightsoff, the package doesn't have proper license headers in the
  source code, but we have the information in About->License to go with.

+ 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:
  dcf2828c746f451396f7a8d60939198a  gnome-tetravex-3.8.0.tar.xz
  dcf2828c746f451396f7a8d60939198a  Download/gnome-tetravex-3.8.0.tar.xz
+ The package builds in koji
n/a ExcludeArch bugs filed
+ BuildRequires look sane
+ The spec file handles locales properly
n/a ldconfig in %post and %postun
+ Package does not bundle copies of system libraries
n/a Package isn't relocatable
+ Package owns all the directories it creates
+ No duplicate files in %files
+ 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
n/a -devel must require the fully versioned base
+ Packages should not contain libtool .la files
+ Proper .desktop file handling
+ Doesn't own files or directories already owned by other packages
+ Filenames are valid UTF-8

Looks good!


Comment 4 Tanner Doshier 2013-04-14 20:58:26 UTC
Using upstream's description blurb.

New Package SCM Request
Package Name: gnome-tetravex
Short Description: Complete the puzzle by matching numbered tiles
Owners: doshitan
Branches: f19

Comment 5 Gwyn Ciesla 2013-04-15 13:56:24 UTC
Git done (by process-git-requests).

Comment 6 Tanner Doshier 2013-04-30 17:10:00 UTC
Forgot to close the bug. Imported and built in rawhide and f19.