[Bug 233256] Review Request: asc-music - Background music for the game asc

[bugzilla at redhat.com]

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: asc-music - Background music for the game asc


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


bugzilla at redhat.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|normal                      |medium




------- Additional Comments From musuruan at gmail.com  2007-04-19 09:56 EST -------
 - Package meets naming and packaging guidelines. OK
 - Spec file matches base package name. OK
 - Spec has consistant macro usage. OK
 - Meets Packaging Guidelines. OK
 - License. SEE BELOW
 - License field in spec matches. SEE BELOW
 - License file included in package. SEE BELOW
 - Spec in American English. OK
 - Spec is legible. OK
 - Sources match upstream md5sum: N/A transcoded from original files

 - Package needs ExcludeArch. N/A
 - BuildRequires correct. OK
 - Spec handles locales/find_lang. N/A
 - Package is relocatable and has a reason to be. N/A
 - Package has %defattr and permissions on files is good. OK
 - Package has a correct %clean section. OK
 - Package has correct buildroot. OK
 - Package is code or permissible content. OK
 - Doc subpackage needed/used. N/A
 - Packages %doc files don't affect runtime. OK

 - Headers/static libs in -devel subpackage. N/A
 - Spec has needed ldconfig in post and postun. N/A
 - .pc files in -devel subpackage/requires pkgconfig. N/A
 - .so files in -devel subpackage. N/A
 - -devel package Requires: %{name} = %{version}-%{release}. N/A
 - .la files are removed. N/A

 - Package is a GUI app and has a .desktop file. N/A

 - Package compiles and builds on at least one arch. OK fc6/i386
 - Package has no duplicate files in %files. OK
 - Package doesn't own any directories other packages own. OK
 - Package owns all the directories it creates. OK
 - No rpmlint output. OK
 - final provides and requires are sane:
	asc-music-1.0-1.noarch.rpm
	asc-music = 1.0-1
	=
	asc

SHOULD Items:

 - Should build in mock. OK fc6/i386
 - Should build on all supported archs. OK package is noarch
 - Should function as described. OK
 - Should have sane scriptlets. N/A
 - Should have subpackages require base package with fully versioned depend. NO,
but OK
 - Should have dist tag. NO, but OK
 - Should package latest version. OK

Issues:

1. Let's try to clear out under what license the audio files are released. 
 
   I read the following on the upstream site: "ASC is distributed under 
   the terms of the Gnu General Public License (the license Linux 
   uses too), which basically means the game is free, the complete source 
   code is available and you can do everything with it as long as it 
   remains free."

   There is no indication on download section of what license is used for 
   the audio files.

   The best would be to have such a statement from the author and include 
   it in your source package.

   BTW, the debian package also state that this is GPL'ed, but again, 
   without any further explaination.



-- 
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.