[Bug 233256] Review Request: asc-music - Background music for the game asc
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 [EMAIL PROTECTED] changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE --- Additional Comments From [EMAIL PROTECTED] 2007-04-22 07:27 EST --- Thanks for the review! Imported and build, closing. -- 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@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 233256] Review Request: asc-music - Background music for the game asc
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 [EMAIL PROTECTED] changed: What|Removed |Added Flag||fedora-cvs? --- Additional Comments From [EMAIL PROTECTED] 2007-04-21 08:05 EST --- New Package CVS Request === Package Name: asc-music Short Description: Background music for the game asc Owners:[EMAIL PROTECTED] Branches: devel InitialCC: empty -- 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@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 233256] Review Request: asc-music - Background music for the game asc
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 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ -- 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@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 233256] Review Request: asc-music - Background music for the game asc
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 --- Additional Comments From [EMAIL PROTECTED] 2007-04-20 08:08 EST --- 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). -- 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@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 233256] Review Request: asc-music - Background music for the game asc
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 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Additional Comments From [EMAIL PROTECTED] 2007-04-20 15:34 EST --- It is good enough for me. 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@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 233256] Review Request: asc-music - Background music for the game asc
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 [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |ASSIGNED -- 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@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 233256] Review Request: asc-music - Background music for the game asc
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
[EMAIL PROTECTED] changed:
What|Removed |Added
Priority|normal |medium
--- Additional Comments From [EMAIL PROTECTED] 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.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 233256] Review Request: asc-music - Background music for the game asc
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 [EMAIL PROTECTED] changed: What|Removed |Added Severity|normal |medium [EMAIL PROTECTED] changed: What|Removed |Added AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] Flag||fedora-review? -- 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@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
