Bug 667205 - Review Request: radius-engine - A Lua based real-time 2D graphics game engine
Summary: Review Request: radius-engine - A Lua based real-time 2D graphics game engine
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Jan Kaluža
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 667207 667208
TreeView+ depends on / blocked
 
Reported: 2011-01-04 19:49 UTC by Tom "spot" Callaway
Modified: 2011-01-18 21:36 UTC (History)
3 users (show)

Fixed In Version: radius-engine-0.6-1.fc14
Clone Of:
Environment:
Last Closed: 2011-01-18 21:31:24 UTC
Type: ---
Embargoed:
jkaluza: fedora-review+
j: fedora-cvs+


Attachments (Terms of Use)

Description Tom "spot" Callaway 2011-01-04 19:49:54 UTC
Spec URL: http://www.auroralinux.org/people/spot/review/new/radius-engine-0.6-1.fc14.src.rpm
SRPM URL: http://www.auroralinux.org/people/spot/review/new/radius-engine.spec
Description: 
Radius Engine is a Lua script-based real-time 2D graphics engine designed for
rapidly prototyping games. Built on top of SDL and OpenGL, games made with
Radius Engine are portable to both Windows and Linux.

Comment 1 Tom "spot" Callaway 2011-01-04 19:53:17 UTC
Koji Rawhide Scratch Build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=2701205

Comment 2 Jan Kaluža 2011-01-06 10:24:53 UTC
> $ rpmlint radius-engine-*
> radius-engine-devel.x86_64: W: no-documentation
> 3 packages and 0 specfiles checked; 0 errors, 1 warnings.

> $ rpmlint radius-engine.spec 
> radius-engine.spec: W: no-cleaning-of-buildroot %clean
> radius-engine.spec: W: no-buildroot-tag
> radius-engine.spec: W: no-%clean-section
> 0 packages and 1 specfiles checked; 0 errors, 3 warnings.

These are all okay for F13+. I presume you don't want to have this package in EPEL, right?

Formal review according to Review Guidelines:
Explanation:
[ok] .... the package meets the guideline item
[--] .... the guideline item is not relevant for this package
[ERR] ... the package fails to meet the guideline and must be fixed.
====================

[ok] rpmlint must be run on every package.
[ok] named according to the Package Naming Guidelines.
[ok] The spec file name must match the base package %{name}
[ok] License must be Fedora approved; Licensing Guidelines.
[ok] The License field in the package spec file must match the actual license.
[ok] license file (if any) must packaged in %doc.
[ok] spec file in American English.
[ok] spec legible.
[ok] sources must match the upstream source
[ok] must compile and build.
[--] ExcludeArch if it does not.
[ok] complete and sensible BuildRequires
[--] handling of locales
[ok] ldconfig for dynamic libs
[ok] Packages must NOT bundle copies of system libraries.
[--] rules for relocatable packages
[ok] directory ownership
[ok] no duplicate listing in %files
[ok] sane permissions; %defattr(...)
[ok] consistent macro usage
[ok] code or permissable content
[ok] large doc
[ok] header files
[--] static libs
[ok] .so in -devel
[ok] devel requires base package
[--] remove .la files
[--] GUI app must include a %{name}.desktop and use desktop-file-install
[ok] no owning of other packages' files/dirs
[ok] UTF-8 filenames

Formal review according to Packaging Guidelines:

[ok] naming
[ok] version and release
[ok] Licensing
[ok] no inclusion of pre-built binaries or libraries
[ok] spec legibility
[ok] arch support
[ok] filesystem layout
[ok] changelogs
[ok] tags
[ok] BuildRoot
[ok] Requires
[ok] BuildRequires
[ok] summary and description
[ok] encoding
[ok] compiler flags
[ok] debuginfo
[ok] devel packages
[ok] no duplication of system libraries
[ok] no rpath
[ok] config files
[--] initscripts
[--] desktop files
[--] Icon tag in Desktop Files
[ok] macros (inconsistent usage, as already noted)
[--] handling locale files
[ok] timestamps
[ok] parallel make
[--] scriptlets
[--] conditional deps
[--] relocatable packages
[ok] code vs content
[ok] file and dir ownership
[--] users and groups
[--] web apps
[ok] no conflicts
[ok] no kernel modules
[ok] nothing in /srv
[ok] no bundling
[ok] no fonts bundling
[--] epoch
[ok] symlinks
[--] man pages

I've tested also those two games you've on review with this library and it seems they both work.
To sum it up, it's OK if it's F13+ package.

Comment 3 Tom "spot" Callaway 2011-01-06 13:55:00 UTC
If someone wants this added to EPEL, I will add back the necessary fields, but I have no plans to include it in EPEL at this time, just F13+.

Comment 4 Jan Kaluža 2011-01-07 18:32:22 UTC
Consider the package APPROVED.

Comment 5 Tom "spot" Callaway 2011-01-07 18:49:38 UTC
New Package SCM Request
=======================
Package Name: radius-engine
Short Description: A Lua based real-time 2D graphics game engine
Owners: spot
Branches: f13 f14 devel
InitialCC:

Comment 6 Jason Tibbitts 2011-01-08 02:48:08 UTC
Git done (by process-git-requests).

Comment 7 Fedora Update System 2011-01-10 15:05:46 UTC
radius-engine-0.6-1.fc13 has been submitted as an update for Fedora 13.
https://admin.fedoraproject.org/updates/radius-engine-0.6-1.fc13

Comment 8 Fedora Update System 2011-01-10 15:05:53 UTC
radius-engine-0.6-1.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/radius-engine-0.6-1.fc14

Comment 9 Fedora Update System 2011-01-10 21:26:50 UTC
radius-engine-0.6-1.fc14 has been pushed to the Fedora 14 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update radius-engine'.  You can provide feedback for this update here: https://admin.fedoraproject.org/updates/radius-engine-0.6-1.fc14

Comment 10 Fedora Update System 2011-01-18 21:31:19 UTC
radius-engine-0.6-1.fc13 has been pushed to the Fedora 13 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 11 Fedora Update System 2011-01-18 21:36:05 UTC
radius-engine-0.6-1.fc14 has been pushed to the Fedora 14 stable repository.  If problems still persist, please make note of it in this bug report.


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