Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 199747 - Review Request: pingus - Guide the penguins safely home before they drop of the cliff
Summary: Review Request: pingus - Guide the penguins safely home before they drop of t...
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Paul F. Johnson
QA Contact: Fedora Package Reviews List
URL:
Whiteboard:
Depends On: 199630
Blocks: FE-ACCEPT
TreeView+ depends on / blocked
 
Reported: 2006-07-21 18:22 UTC by Hans de Goede
Modified: 2007-11-30 22:11 UTC (History)
0 users

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-07-25 11:32:28 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Hans de Goede 2006-07-21 18:22:10 UTC
Spec URL: http://people.atrpms.net/~hdegoede/pingus.spec
SRPM URL: http://people.atrpms.net/~hdegoede/pingus-0.7.0-0.1.20060721.src.rpm
Description:
A game inspired by the game Lemmings. You take command in the game of a bunch
of small penguins and have to guide them around in levels. Since the penguins
walk on their own, the player can only influence them by giving them commands,
like build a bridge, dig a hole or redirect all penguins in the other
direction. The goal of each level is to reach the exit, for which multiple
combination of commands are necessary. The game is presented in a 2D site view.


---

Note this packages needs ClanLib, review bug 199630

Comment 1 Paul F. Johnson 2006-07-21 19:40:01 UTC
Okay, from a very quick glance over...

needs BR get-text

I'll check it over more closely tonight


Comment 2 Hans de Goede 2006-07-21 20:35:32 UTC
Darn, I spend hours getting the gettext support to work and still forgot to add
the BR. I will add that together with any other nescesarry changes once a full
review is done, or if some other changes come up before that.


Comment 3 Paul F. Johnson 2006-07-23 10:33:33 UTC
Once I add the BR for get-text, everything is happy under mock and rpmlint comes
up clean.

The icon installs correctly under games.

One concern is that the website only has version 0.6.0 as the stable release and
not 0.7.0 as available here. I couldn't find it in the development area either.

If you can make the BR and version number changes, this is good to go.

Comment 4 Hans de Goede 2006-07-23 10:40:07 UTC
If you look at the first screen when you start pingus it will welcome you to
pingus 0.7.0 . As you can see in the spec this is a svn snapshot, which is also
reflected in the release: 0.1.20060721 . The current svn trunk is destined to 
become 0.7.0 some day (hopefully soon), so calling it 0.6.x is wrong, since it
is not 0.6.x.


Comment 5 Paul F. Johnson 2006-07-23 10:52:36 UTC
Not a problem, it's just the BR and out the door it can go!

Comment 6 Hans de Goede 2006-07-23 17:46:38 UTC
New spec with fixed BR here:
Spec URL: http://people.atrpms.net/~hdegoede/pingus.spec

Notice I didn't upload a new SRPM as its huge and my uplink is quite slow.


Comment 7 Paul F. Johnson 2006-07-24 09:04:42 UTC
I could get the original spec file version to compile happily under mock, but
the current one gives me an error that clanlib-devel cannot be found. Is this an
SELinux related problem?

Comment 8 Hans de Goede 2006-07-24 13:55:40 UTC
?? Are you sure, ClanLib has recently been build and hasn't hit the repo yet
(atleast not my mirror) also clanlib is only available in the development repo
(waiting for CVS branching). So I don't find it strange it doesn't build in
mock, I do find it strange it once did build :)



Comment 9 Paul F. Johnson 2006-07-24 14:11:32 UTC
I have a local repo for this sort of thing which is why it built originally.

Comment 10 Hans de Goede 2006-07-24 14:42:41 UTC
Is ClanLib still in this repo?


Comment 11 Paul F. Johnson 2006-07-24 14:54:40 UTC
From memory, yes. I'll need to check though (it was last time I looked)

Comment 12 Paul F. Johnson 2006-07-24 21:43:12 UTC
D'oh, the cache was chomped. I'm happy for this to be released now.

APPROVED

Comment 13 Hans de Goede 2006-07-25 11:32:28 UTC
Thanks! Imported and Build, closing.

p.s.

For the next review, when you approve a package you should change the blocker
bug from FE-REVIEW to FE-ACCEPT . I've done that for the 2 ClanLib's and I'll do
it for this one.



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