Bug 452805 - (ember) Review Request: ember - WorldForge client
Review Request: ember - WorldForge client
Status: CLOSED NEXTRELEASE
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Tom "spot" Callaway
Fedora Extras Quality Assurance
:
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2008-06-25 04:01 EDT by Alexey Torkhov
Modified: 2008-09-15 08:18 EDT (History)
7 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2008-09-12 03:26:28 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
tcallawa: fedora‑review+
huzaifas: fedora‑cvs+


Attachments (Terms of Use)

  None (edit)
Description Alexey Torkhov 2008-06-25 04:01:57 EDT
Spec URL: http://purple.worldforge.org/~alex/fedora/ember.spec
SRPM URL: http://purple.worldforge.org/~alex/fedora/ember-0.5.2-1.fc10.src.rpm
Description: Ember is a client for MMORPGs using the Worldforge system.
It uses the Ogre 3D engine with CEGUI.

This package builds only under rawhide (in mock) because it requires cegui-0.6.0 and libwfut.
All patches (including changed tarball because of non-free pack) are reported upstream and would be fixed in next version.
Comment 1 Alexey Torkhov 2008-07-09 16:47:27 EDT
Spec URL: http://purple.worldforge.org/~alex/fedora/ember.spec
SRPM URL: http://purple.worldforge.org/~alex/fedora/ember-0.5.3-1.fc10.src.rpm

Update to 0.5.3. All patches are now merged. License upgraded to GPLv3+.
Comment 3 Tom "spot" Callaway 2008-09-11 13:10:11 EDT
REVIEW
==========

Good:

- rpmlint checks return:
ember.x86_64: W: conffile-without-noreplace-flag /etc/ember/ogre.cfg
ember.x86_64: W: conffile-without-noreplace-flag /etc/ember/resources.cfg
ember.x86_64: W: conffile-without-noreplace-flag /etc/ember/terrain.cfg

Documented why you didn't do this, safe to ignore.

- package meets naming guidelines
- package meets packaging guidelines
- license (GPLv3+) OK, text in %doc, matches source
- spec file legible, in am. english
- source matches upstream (f5d94ac46131bcf0bb7788e311331aac523287b8)
- package compiles on devel (x86_64, koji scratch)
- 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 ok

Nice job with this package, APPROVED.
Comment 4 Alexey Torkhov 2008-09-11 16:26:46 EDT
Many thanks for review!

New Package CVS Request
=======================
Package Name: ember
Short Description: 3D client for WorldForge
Owners: atorkhov wart
Branches: F-9
InitialCC:
Comment 5 Huzaifa S. Sidhpurwala 2008-09-11 23:22:57 EDT
cvs done

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