Re: [PATCH] gnu: Add powwow.

[iyzsong@member.fsf.org (宋文武)]

Nils Gillmann <niasterisk@grrlz.net> writes:

> I removed gplv3+ as addressed by ludo' and rekado,
> corrected the url in "website" (added missing ".net").
>
> Here's the new patch, rebased against current master.
Thanks!
>
> From f59e5027d4fb9559e179fb4693617820f6680223 Mon Sep 17 00:00:00 2001
> From: Nils Gillmann <niasterisk@grrlz.net>
> Date: Thu, 3 Mar 2016 12:41:08 +0100
> Subject: [PATCH] gnu: Add powwow.
>
> * gnu/packages/admin.scm (powwow): New variable.
> ---
>  gnu/packages/admin.scm | 25 +++++++++++++++++++++++++
>  1 file changed, 25 insertions(+)
>
> diff --git a/gnu/packages/admin.scm b/gnu/packages/admin.scm
> index b0b2046..1a26fa0 100644
> --- a/gnu/packages/admin.scm
> +++ b/gnu/packages/admin.scm
I think it isn’t for admin…  how about put it in games.scm?
> @@ -9,6 +9,7 @@
>  ;;; Copyright © 2016 Leo Famulari <leo@famulari.name>
>  ;;; Copyright © 2016 Pjotr Prins <pjotr.guix@thebird.nl>
>  ;;; Copyright © 2016 Ricardo Wurmus <rekado@elephly.net>
> +;;; Copyright © 2016 Nils Gillmann <niasterisk@grrlz.net>
>  ;;;
>  ;;; This file is part of GNU Guix.
>  ;;;
> @@ -1496,3 +1497,27 @@ for writing audit records to the disk.  Viewing the logs is done with the
>  @code{ausearch} or @code{aureport} utilities.  Configuring the audit rules is
>  done with the @code{auditctl} utility.")
>      (license license:gpl2+)))
> +
> +(define-public powwow
> +  (package
> +    (name "powwow")
> +    (version "1.2.17")
> +    (source (origin
> +              (method url-fetch)
> +              (uri (string-append "http://www.hoopajoo.net/static/projects/powwow-"
Make it under 80 characters limit?
> +                                  version ".tar.gz"))
> +              (file-name (string-append name "-" version ".tar.gz"))
> +              (sha256
> +               (base32
> +                "1xmsg2y7qcvj67i9ilnih0mvfxcpni7fzrz343x9rdfnkkzf3pp8"))))
> +    (inputs
> +     `(("ncurses" ,ncurses)))
> +    (build-system gnu-build-system)
> +    (home-page "http://www.hoopajoo.net/projects/powwow.html")
> +    (synopsis "POWWOW is a enhanced telnet client which can be used for MUD")
Don’t mention itself in synopsis, I think “MUD and telnet client” should do.
> +    (description
> +     "POWWOW is a client software which can be used for telnet as well as for
> +@dfn{Multi-User Dungeon} (MUD).  Additionally it can serve as a nice client for
> +the chat server psyced with the specific config located at
> +http://lavachat.symlynx.com/unix/")
> +    (license license:gpl2+)))
> -- 
> 2.6.3
Otherwise, look good to me!