Bug 185847 - Review Request: deutex - Doom wad manipulator
Summary: Review Request: deutex - Doom wad manipulator
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Hans de Goede
QA Contact: Fedora Package Reviews List
URL:
Whiteboard:
Depends On:
Blocks: FE-ACCEPT
TreeView+ depends on / blocked
 
Reported: 2006-03-19 19:10 UTC by Wart
Modified: 2007-11-30 22:11 UTC (History)
0 users

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-03-19 23:07:16 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Wart 2006-03-19 19:10:15 UTC
Spec Name or Url: http://www.kobold.org/~wart/fedora/deutex.spec
SRPM Name or Url: http://www.kobold.org/~wart/fedora/deutex-4.4.0-1.src.rpm
Description: 
Doom wad manipulator.  This can be used to extract and replace textures, sounds, sprites, and levels inside of pwads and iwads.  A necessary tool if you want to do anything more than simple level creation.

Comment 1 Hans de Goede 2006-03-19 20:52:33 UTC
I assume this is a cmdline app (and hence the no desktop file)


Comment 2 Hans de Goede 2006-03-19 20:59:38 UTC
Answering my own question, yes it is :)

MUST
====
* rpmlint output clean except for:
W: deutex invalid-license GPL-Compatible
W: deutex invalid-license GPL-Compatible
W: deutex-debuginfo invalid-license GPL-Compatible
This is ok essentially deutex is GPL, but some source-files bear a different 
GPL compatible license.
* Package named correctly
* GPL (+compatible) license OK.
* spec file legible, in Am. English
* Source matches upstream
* Successfully compiles and builds on at least one platform (FC-5 x86_64)
* no locale data, shared libraries, or static libraries
* No excessive Requires: or BR:
* Summary and description ok
* macro use consistent
* Does not create any directories
* Not relocatable
* %doc does not affect runtime
* File permissions ok

Approved (now that went smoothly!)


Comment 3 Wart 2006-03-19 23:07:16 UTC
I tried to be more careful about testing the build on other platforms this time.  :)

Imported and built.  Thanks!


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