Bug 233256 - Review Request: asc-music - Background music for the game asc
Review Request: asc-music - Background music for the game asc
Status: CLOSED NEXTRELEASE
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Andrea Musuruane
Fedora Package Reviews List
:
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2007-03-21 06:41 EDT by Hans de Goede
Modified: 2007-11-30 17:11 EST (History)
0 users

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2007-04-22 07:27:39 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
musuruan: fedora‑review+
jwboyer: fedora‑cvs+


Attachments (Terms of Use)

  None (edit)
Description Hans de Goede 2007-03-21 06:41:46 EDT
Spec URL: http://people.atrpms.net/~hdegoede/asc-music.spec
SRPM URL: http://people.atrpms.net/~hdegoede/asc-music-1.0-1.src.rpm
Description:
Music created by Michael Kievernagel for the game Advanced Strategic Command
(asc).

Note that if you have run asc before installing the music you must remove the
asc cache file: $HOME/.asc/asc.cache, otherwise asc will not find the music.
Comment 1 Andrea Musuruane 2007-04-19 09:56:40 EDT
 - 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.

Comment 2 Hans de Goede 2007-04-20 08:08:33 EDT
AFAIK the music is an integral part of ASC, its just distributed separately from
the source so that people don't have a huge download each new release.

Since its in integral part, its under the same license.

"Proof" of it being in integral part is tihs ./configure message when building
the asc package:

configure: WARNING:
The following music files could not be found: data/music/frontiers.mp3
data/music/time_to_strike.mp3 data/music/machine_wars.mp3  .
The music files are not distributed together with the source code because
of their size. ASC can run without the music files being present, this message
is just to inform you about the existence of the music.

Notice how they give the size as the only reason for the separate distribution,
they don't say anything about it being under a different license.

---

If this is not good enough I can mail upstream to ask. I'm afraid google is of
no use in locating the composer of the music himself. (I already tried to find
his contact info before submitting this for review).

Comment 3 Andrea Musuruane 2007-04-20 15:34:09 EDT
It is good enough for me.

APPROVED.
Comment 4 Hans de Goede 2007-04-21 08:05:17 EDT
New Package CVS Request
=======================
Package Name:      asc-music
Short Description: Background music for the game asc
Owners:            j.w.r.degoede@hhs.nl
Branches:          devel
InitialCC:         <empty>

Comment 5 Hans de Goede 2007-04-22 07:27:39 EDT
Thanks for the review!

Imported and build, closing.

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