Bug 176109 - Review Request: pytraffic
Review Request: pytraffic
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
All Linux
medium Severity medium
: ---
: ---
Assigned To: John Mahowald
Fedora Package Reviews List
Depends On:
  Show dependency treegraph
Reported: 2005-12-19 10:43 EST by Michel Van den Bergh
Modified: 2007-11-30 17:11 EST (History)
0 users

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2007-05-22 14:59:31 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)

  None (edit)
Description Michel Van den Bergh 2005-12-19 10:43:20 EST
Spec Name or Url: http://alpha.uhasselt.be/Research/Algebra/Members/pytraffic/pytraffic.spec
SRPM Name or Url: 
Description: A screen version of the well known sliding block puzzle "Rush Hour".
Comment 1 John Mahowald 2005-12-21 17:21:03 EST
Use the python-abi in Requires like the template from the fedora-rpmdevtools

fedora-qa script found:
* Scriptlets: missing "gtk-update-icon-cache" in %postun (wiki:
ScriptletSnippets) (in subpackage pytraffic)
* Source 0 is not available
  (wiki: QAChecklist item 2)
* Scriptlets: missing "gtk-update-icon-cache" in %postun (wiki:
ScriptletSnippets) (in subpackage pytraffic)
Comment 2 Michel Van den Bergh 2005-12-23 02:36:57 EST
Thanks for the comments! I addressed these in a new version.

New spec file:

Comment 3 Michel Van den Bergh 2005-12-25 09:55:15 EST
I wonder if there are any more comments? This is my first package so
if it would be approved for inclusion I would need sponsoring...

Merry Christmas!
Comment 4 Michel Van den Bergh 2006-01-25 05:07:14 EST
Comment 5 John Mahowald 2006-01-25 14:03:35 EST
I can't sponsor, unfortunately.

Have you considered splitting off a noarch data subpackage for most of the arch
independent data under /usr/share? Then make the main package depend on it.
Comment 6 Michel Van den Bergh 2006-01-27 04:37:09 EST
>I can't sponsor, unfortunately.

Too bad... I hope somebody picks it up. I get lots of positive comments about
pytraffic from people who like the puzzle game Rush Hour.

I thought about putting the themes in a noarch package but it seems you cannot
make an arch and a noarch package from the same spec file... Splitting the spec
files seems inconvenient for now.

I made a new version of the spec file since I found out that the installer
should not strip binaries (rpm does this automatically).

Comment 7 Aurelien Bompard 2006-03-05 11:40:00 EST
John, you should be able to sponsor people now, aren't you ?
Comment 8 John Mahowald 2006-03-06 00:45:38 EST
Yes, now I can.

- package meets naming guidelines
- package meets packaging guidelines
- license (GPL) OK, text in %doc, matches source
- spec file legible, in am. english
- source matches upstream
- package compiles on devel (x86_64)
- no missing BR
- no unnecessary BR
- no locales
- not relocatable
- owns all directories that it creates
- no duplicate files
- permissions ok
- %clean ok
- macro use consistent
- code, not content
- no need for -docs
- nothing in %doc affects runtime
- .desktop file 

APPROVED, apply for cvsextras and I'll sponsor.
Comment 9 Michel Van den Bergh 2006-03-06 12:12:26 EST

I'm a bit busy with work right now but I will continue the procedure as soon as
I can. 
Comment 11 Jason Tibbitts 2007-05-22 14:59:31 EDT
So it's been well over a year now.  I think it's high time that this bug is closed.

Note You need to log in before you can comment on or make changes to this bug.