Bug 235636
Summary: | Review Request: pipepanic - A pipe connecting game | ||||||
---|---|---|---|---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Andrea Musuruane <musuruan> | ||||
Component: | Package Review | Assignee: | Hans de Goede <hdegoede> | ||||
Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Package Reviews List <fedora-package-review> | ||||
Severity: | medium | Docs Contact: | |||||
Priority: | medium | ||||||
Version: | rawhide | CC: | hdegoede | ||||
Target Milestone: | --- | Flags: | hdegoede:
fedora-review+
jwboyer: 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: | 2007-05-05 12:43:22 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: | |||||||
Attachments: |
|
Description
Andrea Musuruane
2007-04-08 17:51:19 UTC
Hi, I'd be happy to review this if you help me out and review a game for me: Escape: https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=235588 Data for Escape: https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=235589 Andrea, the same goes for me, I would be happy to exchange a review for a review of any of these: * asc-music - Background music for the game asc - https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=233256 - requires asc * vegastrike - 3D OpenGL spaceflight simulator - https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=233782 * vegastrike-data - Data files for Vega Strike - https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=233783 * ballz - Platform game with some puzzle elements - https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=236798 I haven't looked yet, but if this game points / uses Lucas Trademark PipeDream anywhere, then the use of that TM in docs / credits / etc, must be removed. For more on this see the pipenightdreams review and spec file, and also: https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=224627 (In reply to comment #3) > I haven't looked yet, but if this game points / uses Lucas Trademark PipeDream > anywhere, then the use of that TM in docs / credits / etc, must be removed. The few docs shipped do not have such references. MUST: ===== * rpmlint output is clean * Package and spec file named appropriately * Packaged according to packaging guidelines * License ok * spec file is legible and in Am. English. * Source matches upstream * Compiles and builds on devel x86_64 * BR: ok * No locales * No shared libraries, ldconfig not needed * Not relocatable 0 Package owns / or requires all dirs * No duplicate files & Permissions ok * %clean & macro usage OK * Contains code and permissable content * %doc does not affect runtime, and isn't large enough to warrent a sub package * no -devel package needed * .desktop file as required and properly installed MUST FIX ======== * /usr/share/pipepanic is not owned, change: %{_datadir}/%{name}/* under files to just %{_datadir}/%{name} Should FIX ========== * Drop " that is playable on Linux PCs and the Sharp Zaurus." from %description ofcourse it runs on linux, its a Fedora package, and that it also runs on the Zaurus is of 0 interest. * Set a window title, currently the window is untitled. I'll attach a patch for this. Notice that this patch also needs an icon file in bmp format installed as /usr/share/pipepanic/icon.bmp This file can be generated by the following obscure ImageMagick command, and yes the using of convert twice and the | is necessary: convert PipepanicIcon32.png bmp3:- | convert - -fill '#FF00FF' -opaque black -colors 256 -compress none bmp3:icon.bmp Created attachment 153891 [details]
PATCH: set window title and icon
Spec URL: http://www.webalice.it/musuruan/RPMS/reviews/pipepanic.spec SRPM URL: http://www.webalice.it/musuruan/RPMS/reviews/pipepanic-0.1.3-2.src.rpm Changelog: - Fixed package ownership of its datadir - Changed description - Added a patch by Hans de Goede to set a window title and icon Looks good now, approved! New Package CVS Request ======================= Package Name: pipepanic Short Description: A pipe connecting game Owners: musuruan Branches: FC-5 FC-6 InitialCC: Thanks for the review. Built successfully. Closing. |