[Bug 190267] Review Request: raidem-music - Background music for the game raidem

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: raidem-music - Background music for the game raidem


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190267


wart@xxxxxxxxxx changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
         AssignedTo|bugzilla-sink@xxxxxxxxxxxxx |wart@xxxxxxxxxx
OtherBugsDependingO|163776                      |163779
              nThis|                            |




------- Additional Comments From wart@xxxxxxxxxx  2006-04-30 15:53 EST -------
MUST
====

* Package named appropriately
* License (CC) ok
* Spec file legible and in Am. English
* No BR: needed
* No locales
* No shared libraries
* Not relocatable
* Owns the directory that it creates
* $RPM_BUILD_ROOT cleaned where it should be
* File permissions ok
* No duplicate %files
* Contains no code, but acceptible content (game data)
* No -devel package needed
* No .desktop file needed

SUGGEST
=======
* Slightly modified %description:
  "Music created by Eric Hamilton (dilvie) for the game Raid'em"

The files play fine with xmms.  I was unable to test them with the CVS
version of raidem due to problems with the raidem package itself:
"Error loading base datafiles."

I'll take your word that raidem will load/play these files correctly.

APPROVED


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]