Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.

Bug 216354

Summary: Review Request: liquidwar - Multiplayer wargame with liquid armies
Product: [Fedora] Fedora Reporter: Hans de Goede <hdegoede>
Component: Package ReviewAssignee: Wart <wart>
Status: CLOSED NEXTRELEASE QA Contact: Fedora Package Reviews List <fedora-package-review>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideFlags: kevin: fedora-cvs+
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2006-11-23 20:30:47 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:
Bug Depends On:    
Bug Blocks: 163779    

Description Hans de Goede 2006-11-19 20:49:50 UTC
Spec URL: http://people.atrpms.net/~hdegoede/liquidwar.spec
SRPM URL: http://people.atrpms.net/~hdegoede/liquidwar-5.6.3-1.fc7.src.rpm
Description:
Liquid War is a unique multiplayer wargame. You control an army of liquid  
and have to try and eat your opponents. A single player mode is available,
but the game is definitely designed to be multiplayer, and has network
support.

Comment 1 Wart 2006-11-21 01:59:19 UTC
GOOD
====
* Package and spec named appropriately
* GPL license ok, license file included
* Source matches upstream:
  a41473c281f214c1ac96ab50f98537c9  liquidwar-5.6.3.tar.gz
* Spec file legible and in Am. English
* Compiles and builds on (FC5, FC6, devel) x (i386, x86_64)
* No -devel subpackage needed
* Large docs are located in -doc subpackage
* Not relocatable
* No locales
* Directory ownership ok
* file permissions ok
* .desktop file and icons installed correctly
* No shared libraries
* Application runs as expected

SHOULD
======
* Consider adding the README file to %doc.  It contains the address of the
  package's mailing list which doesn't appear elsewhere.

* 'make dep' gets run as part of the build, and generates the following
  harmless warning:
gmake[1]: Entering directory `/space/rpmbuild/BUILD/liquidwar-5.6.3/src'
find: .-name: No such file or directory
find: *.c: No such file or directory
cc: no input files
gmake[1]: Leaving directory `/space/rpmbuild/BUILD/liquidwar-5.6.3/src'
  It seems there is a missing space after the '.' in src/Makefile.in line 256:
        @find .\
        -name '*.c' | \

* The man page refers the reader to "/usr/share/doc/liquidwar" for more
  documentation.  This path should be updated to point to the files in the
  -doc subpackage.

* The URL: tag points to a page with a better home page link:
  http://www.ufoot.org/liquidwar/v5

* The metaserver URL in the man page is wrong.  It should be:
  http://www.ufoot.org/liquidwar/v5/metaserver

* An init.d script for starting a liquidwar server at boot time would be nice. 
Perhaps if I find some extra time I'll write one for you.

None of these issues are serious enough to block approval:

APPROVED


Comment 2 Hans de Goede 2006-11-23 20:23:24 UTC
(In reply to comment #1)
Thanks for review and the good feedback!
> 
> SHOULD
> ======
> * Consider adding the README file to %doc.  It contains the address of the
>   package's mailing list which doesn't appear elsewhere.
> 
Done

> * 'make dep' gets run as part of the build, and generates the following
>   harmless warning:
> gmake[1]: Entering directory `/space/rpmbuild/BUILD/liquidwar-5.6.3/src'
> find: .-name: No such file or directory
> find: *.c: No such file or directory
> cc: no input files
> gmake[1]: Leaving directory `/space/rpmbuild/BUILD/liquidwar-5.6.3/src'
>   It seems there is a missing space after the '.' in src/Makefile.in line 256:
>         @find .\
>         -name '*.c' | \
> 
Harmless indeed, ignored.

> * The man page refers the reader to "/usr/share/doc/liquidwar" for more
>   documentation.  This path should be updated to point to the files in the
>   -doc subpackage.
> 
Fixed

> * The URL: tag points to a page with a better home page link:
>   http://www.ufoot.org/liquidwar/v5
> 
Done

> * The metaserver URL in the man page is wrong.  It should be:
>   http://www.ufoot.org/liquidwar/v5/metaserver
> 
Fixed

> * An init.d script for starting a liquidwar server at boot time would be nice. 
> Perhaps if I find some extra time I'll write one for you.
> 

I wouldn't mind receiving this, together with a patch to split out the server in
its own subpackage, as I think if we add an init script the server really should
be in its own subpackage (and we need to think about wether to start it by
default or not)

> None of these issues are serious enough to block approval:
> 
> APPROVED
> 

Imported and build, closing. Thanks again!


Comment 3 Wart 2006-11-24 02:35:25 UTC
(In reply to comment #2)
> > * An init.d script for starting a liquidwar server at boot time would be nice. 
> > Perhaps if I find some extra time I'll write one for you.
> > 
> 
> I wouldn't mind receiving this, together with a patch to split out the server in
> its own subpackage, as I think if we add an init script the server really should
> be in its own subpackage (and we need to think about wether to start it by
> default or not)

It most definitely should not be started by default.  The server admin should
have to run 'chkconfig liquidwar on' manually or use the gui service tools to
enable the service to run at boot time.  Certain system applications (like sshd)
should be started by default, but games should not.

I've got the necessary init/sysconfig/logrotate scripts for liquid war.  Would
you prefer that I attach them to a new bug report, or just check them into cvs
(without requesting a build, of course)?

Comment 4 Hans de Goede 2006-11-24 06:36:27 UTC
Just dropping them in CVS will be fine, a specfile update for this would be very
welcome too :)


Comment 5 Hans de Goede 2007-10-19 11:15:12 UTC
Package Change Request
======================
Package Name: liquidwar
New Branches: F-8



Comment 6 Kevin Fenzi 2007-10-21 16:25:32 UTC
All packages have now been branched for F-8, so this request should be done. 
Feel free to reset fedora-cvs flag if you need anything futher.