From: ludo@gnu.org (Ludovic Courtès)
To: Christopher Allan Webber <cwebber@dustycloud.org>
Cc: guix-devel@gnu.org
Subject: Re: [PATCH] add kobodeluxe
Date: Mon, 17 Oct 2016 16:50:29 +0200 [thread overview]
Message-ID: <87zim381xm.fsf@gnu.org> (raw)
In-Reply-To: <87h98bag8p.fsf@dustycloud.org> (Christopher Allan Webber's message of "Sun, 16 Oct 2016 20:58:30 -0500")
Hello!
Welcome, Stephen!
Christopher Allan Webber <cwebber@dustycloud.org> skribis:
> From c8b122d8c7cf7a574bbbac66963ac355c64effc5 Mon Sep 17 00:00:00 2001
> From: Stephen Webber <webber.sl@gmail.com>
> Date: Sun, 16 Oct 2016 15:04:35 -0500
> Subject: [PATCH] gnu: Add kobodeluxe.
>
> * gnu/packages/games.scm (kobodeluxe): New variable.
> * gnu/packages/patches/kobodeluxe_paths.patch: New file.
> * gnu/packages/patches/kobodeluxe-spelling.patch: New file.
> * gnu/packages/patches/kobodeluxe_enemies-pipe-decl.patch: New file.
> * gnu/packages/patches/kobodeluxe_const_charp_conversion.patch: New file.
> * gnu/packages/patches/kobodeluxe_manpage-minus-not-hyphen.patch: New file.
> * gnu/packages/patches/kobodeluxe-midicon-segmentation-fault.patch: New file.
> * gnu/packages/patches/kobodeluxe_graphics_window-signed-char.patch: New file.
> * gnu/local.mk (dist_patch_DATA): Register it.
Overall looks good to me! I have minor/cosmetic suggestions, but
otherwise this is ready for commit IMO.
> + %D%/packages/patches/kobodeluxe_paths.patch
> + %D%/packages/patches/kobodeluxe-spelling.patch
> + %D%/packages/patches/kobodeluxe_enemies-pipe-decl.patch
> + %D%/packages/patches/kobodeluxe_const_charp_conversion.patch
> + %D%/packages/patches/kobodeluxe_manpage-minus-not-hyphen.patch
> + %D%/packages/patches/kobodeluxe-midicon-segmentation-fault.patch
> + %D%/packages/patches/kobodeluxe_graphics_window-signed-char.patch
Maybe use only hyphens in file names for consistency.
> + (synopsis "Shooter with space station destruction")
> + (description
> + "Kobo Deluxe is an enhanced version of Akira Higuchi's XKobo for Un*x
> +systems with X11.")
Maybe s/XKobo.*/XKobo graphical game/.
That way ‘guix package -s game’ will return it. :-)
> --- /dev/null
> +++ b/gnu/packages/patches/kobodeluxe-spelling.patch
> @@ -0,0 +1,25 @@
> +Description: two spelling fixes
> +Author: Damyan Ivanov <dmn@debian.org>
Maybe this patch can be omitted; hopefully it’ll be fixed in the next
release.
> +++ b/gnu/packages/patches/kobodeluxe_paths.patch
> @@ -0,0 +1,40 @@
> +# Disable reading of configs, graphics, and data from the current directory.
> +# So it's safe to run kobodeluxe from anywhere w/o worry about an attacker
> +# feeding it bad data.
Could you indicate the origin or status of this patch?
Chris: feel free to push an updated version!
Stephen, you picked a tricky package as your first patch but it’s
perfect! I’m sure you had a good mentor too. ;-)
Ludo’.
next prev parent reply other threads:[~2016-10-17 14:50 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-10-16 20:47 [PATCH] add kobodeluxe Stephen Webber
2016-10-16 23:22 ` Christopher Allan Webber
2016-10-17 1:58 ` Christopher Allan Webber
2016-10-17 14:50 ` Ludovic Courtès [this message]
2016-10-17 2:06 ` Kei Kebreau
2016-10-17 2:19 ` Stephen Webber
2016-10-17 14:14 ` Christopher Allan Webber
2016-10-17 16:41 ` Kei Kebreau
2016-10-18 3:49 ` Stephen Webber
2016-10-19 1:36 ` Christopher Allan Webber
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87zim381xm.fsf@gnu.org \
--to=ludo@gnu.org \
--cc=cwebber@dustycloud.org \
--cc=guix-devel@gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/guix.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.