Bug 466829
Summary: | Review Request: openalchemist - reflexion game | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Xavier Bachelot <xavier> |
Component: | Package Review | Assignee: | Alexey Torkhov <atorkhov> |
Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | atorkhov, fedora-package-review, kevin, notting |
Target Milestone: | --- | Flags: | atorkhov:
fedora-review+
kevin: fedora-cvs+ |
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2008-10-24 23:50:17 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
Xavier Bachelot
2008-10-13 22:13:22 UTC
New version, added a desktop file and icon. Spec URL: http://www.bachelot.org/fedora/SPECS/openalchemist.spec SRPM URL: http://www.bachelot.org/fedora/SRPMS/openalchemist-0.3-2.fc8.src.rpm Correct the following things: - Add license for data to License field. - Add pygtk2 to Requires as openalchemist-config uses it. - Remove duplicate files: leave one of COPYING and CODE-LICENSE. Remove licenses from datadir. New version, changelog : - Modify Summary: tag. - Add graphics license to License: tag. - Add Requires: pygtk2. - Update desktop file. - Remove duplicate license files. Spec URL: http://www.bachelot.org/fedora/SPECS/openalchemist.spec SRPM URL: http://www.bachelot.org/fedora/SRPMS/openalchemist-0.3-3.fc8.src.rpm REVIEW: + rpmlint output is ignorable: openalchemist.i386: W: devel-file-in-non-devel-package /usr/bin/openalchemist-config + The package is named according to the Package Naming Guidelines. + The spec file name matches the base package %{name}, in the format %{name}.spec. + The package meets the Packaging Guidelines. + The package is licensed with a Fedora approved license and meets the Licensing Guidelines. + The License field in the package spec file matches the actual license (GPLv2+ and CC-BY-SA). + File, containing the text of the licenses for the package is included in %doc. + The spec file is written in American English. + The spec file for the package is legible. + The sources used to build the package must matches the upstream source, as provided in the spec URL. 3107b699d63d5ae7cbf65ddc9872c49a openalchemist-0.3-src.tar.gz 3107b699d63d5ae7cbf65ddc9872c49a openalchemist-0.3-src.tar.gz.orig + The package successfully compiles and builds into binary rpms on at least one supported architecture (x86). + All build dependencies are listed in BuildRequires. + No need to deal with locales. + Does not contain shared libraries. + The package does not designed to be relocatable. + A package owns all directories that it creates. + A package does not contain any duplicate files in the %files listing. + Permissions on files are set properly. + The package has a %clean section, which contains rm -rf $RPM_BUILD_ROOT. + The package consistently uses macros, as described in the macros section of Packaging Guidelines. + The package contains code, or permissable content. + Does not contain large documentation files. + Includes only doc files in $doc. + No headers. + No static libraries. + The package does not contain pkgconfig(.pc) files. + The package does not contain library files with a suffix (e.g. libfoo.so.1.1). + No devel packages. + The package does not contain any .la libtool archives. + Includes .desktop file. Properly installed with desktop-file-install. + The package does not own files or directories already owned by other packages. + At the beginning of %install, the package runs rm -rf $RPM_BUILD_ROOT. + All filenames in the package are valid UTF-8. This package is APPROVED. Thanks for the review Alexey. New Package CVS Request ======================= Package Name: openalchemist Short Description: Puzzle game Owners: xavierb Branches: F-8 F-9 EL-4 EL-5 InitialCC: Shouldn't this not include the name "Tetris" in the description? See bug 238526 for some discussion. I think that "Tetris" in that form can be included in description. Problems in other games were in that they were clones and had derived names. This game in not a clone definitely. This case is like "Windows" used in wine description. But, may be, better to remove it to match other games and take away all questions regarding legal status. So, Xavier, please remove it from description. What about something like that (s/Tetris/classic falling block games/) : " OpenAlchemist is a new reflexion game which looks like classic falling block games but where you can take your time. Be a crazy alchemist and try to make new objects from those you get from the sky. " Sounds fine for me. Kevin, could you create CVS for package with updated description? Isn't "reflexion" primarily a British spelling? Actually wikipedia indicates that it's rare now even in British usage, with "reflection" being much more common. http://en.wikipedia.org/wiki/American_and_British_English_spelling_differences The authors of the games are not native english speakers, and neither do I. I can either replace 'reflexion' by 'reflection' or by 'puzzle', just like I did in the Summary: tag. Jason, you're are the only native speaker here, so feel free to rephrase as much as you want, I'll probably be ok with it ;-) Anyway, all of this are details that can be adjusted later, so can you please create the cvs module so I can import ? I'll build when we're all ok with the spelling and grammar of the summary. Thanks for looking into it. cvs done. openalchemist-0.3-4.fc8 has been submitted as an update for Fedora 8. http://admin.fedoraproject.org/updates/openalchemist-0.3-4.fc8 openalchemist-0.3-4.fc9 has been submitted as an update for Fedora 9. http://admin.fedoraproject.org/updates/openalchemist-0.3-4.fc9 openalchemist-0.3-4.fc8 has been pushed to the Fedora 8 stable repository. If problems still persist, please make note of it in this bug report. openalchemist-0.3-4.fc9 has been pushed to the Fedora 9 stable repository. If problems still persist, please make note of it in this bug report. Xavier, one more thought. Shouldn't it have in .desktop file Categories=Game;LogicGame;BlocksGame (i.e. add BlocksGame) as it still falling blocks game? More info on how game categories selected here: https://www.redhat.com/archives/fedora-games-list/2007-March/msg00003.html |