unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Cyrill Schenkel <cyrill.schenkel@gmail.com>
To: Guix-devel <guix-devel@gnu.org>
Subject: Re: [PATCH] gnu: Add cmatrix.
Date: Tue, 01 Jul 2014 13:07:00 +0200	[thread overview]
Message-ID: <87fvilqqwb.fsf@gmail.com> (raw)
In-Reply-To: <87egy5z83r.fsf@gnu.org> ("Ludovic \=\?utf-8\?Q\?Court\=C3\=A8s\=22'\?\= \=\?utf-8\?Q\?s\?\= message of "Tue, 01 Jul 2014 12:28:08 +0200")

[-- Attachment #1: Type: text/plain, Size: 962 bytes --]

Hi

ludo@gnu.org (Ludovic Courtès) writes:

> Also, what about putting it in games.scm instead?

Done.

>> +    (build-system gnu-build-system)
>> +    (arguments
>> +     '(#:phases
>> +       (alist-cons-before
>> +        'patch-source-shebangs 'autoreconf
>> +        (lambda args
>> +          (system* "autoreconf" "-fi"))
>> +        %standard-phases)))
>
> This shouldn’t be needed because the tarballs includes ./configure,
> Makefile.in, etc.

Actually it is needed because the configure which comes with the tarball
was generated with a very old version of autotools and doesn't support
the passing of variables (like SHELL). 

>> +    (synopsis "CMatrix simulates the display from \"The Matrix\"")
>
> In general the synopsis shouldn’t repeat the package name, so:
>
>   Simulate the display from \"The Matrix\"

Done.

>
> Could you send an updated patch?

An updated patch is attached.

Regards
Cyrill


[-- Attachment #2: 0001-gnu-Add-cmatrix.patch --]
[-- Type: text/x-diff, Size: 3043 bytes --]

From f0da309bfd8187d8d63386bb908c462fa12fc6f9 Mon Sep 17 00:00:00 2001
From: Cyrill Schenkel <cyrill.schenkel@gmail.com>
Date: Tue, 1 Jul 2014 13:02:53 +0200
Subject: [PATCH] gnu: Add cmatrix.

* gnu/packages/games.scm (cmatrix): New variable.
---
 gnu/packages/games.scm | 36 +++++++++++++++++++++++++++++++++++-
 1 file changed, 35 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/games.scm b/gnu/packages/games.scm
index 8a2b2df..946f56b 100644
--- a/gnu/packages/games.scm
+++ b/gnu/packages/games.scm
@@ -2,6 +2,7 @@
 ;;; Copyright © 2013 John Darrington <jmd@gnu.org>
 ;;; Copyright © 2014 David Thompson <dthompson2@worcester.edu>
 ;;; Copyright © 2014 Eric Bavier <bavier@member.fsf.org>
+;;; Copyright © 2014 Cyrill Schenkel <cyrill.schenkel@gmail.com>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -23,6 +24,7 @@
   #:use-module (guix packages)
   #:use-module (guix download)
   #:use-module (gnu packages)
+  #:use-module (gnu packages autotools)
   #:use-module (gnu packages boost)
   #:use-module (gnu packages gettext)
   #:use-module (gnu packages gl)
@@ -32,13 +34,15 @@
   #:use-module (gnu packages guile)
   #:use-module (gnu packages libcanberra)
   #:use-module (gnu packages image)
+  #:use-module (gnu packages ncurses)
   #:use-module (gnu packages python)
   #:use-module (gnu packages readline)
   #:use-module (gnu packages xorg)
   #:use-module (gnu packages pkg-config)
   #:use-module (gnu packages sqlite)
   #:use-module (gnu packages sdl)
-  #:use-module (guix build-system gnu))
+  #:use-module (guix build-system gnu)
+  #:use-module (guix build gnu-build-system))
 
 (define-public gnubg
   (package
@@ -228,3 +232,33 @@ into stereotyped or otherwise humorous dialects.  The filters are provided as
 a C library, so they can easily be integrated into other programs.")
     (license gpl2+)))
  
+(define-public cmatrix
+  (package
+    (name "cmatrix")
+    (version "1.2a")
+    (source
+     (origin
+       (method url-fetch)
+       (uri (string-append "http://www.asty.org/cmatrix/dist/cmatrix-" version
+                           ".tar.gz"))
+       (sha256
+        (base32
+         "0k06fw2n8nzp1pcdynhajp5prba03gfgsbj91bknyjr5xb5fd9hz"))))
+    (build-system gnu-build-system)
+    (arguments
+     '(#:phases
+       (alist-cons-before
+        'patch-source-shebangs 'autoreconf
+        (lambda args
+          (system* "autoreconf" "-fi"))
+        %standard-phases)))
+    (inputs `(("ncurses" ,ncurses)
+              ("autoconf" ,autoconf)
+              ("automake" ,automake)))
+    (home-page "http://wwww.asty.org/cmatrix")
+    (synopsis "Simulate the display from \"The Matrix\"")
+    (description "CMatrix simulates the display from \"The Matrix\" and is
+based on the screensaver from the movie's website.  It works with terminal
+settings up to 132x300 and can scroll lines all at the same rate or
+asynchronously and at a user-defined speed.")
+    (license gpl2+)))
-- 
2.0.0


  reply	other threads:[~2014-07-01 11:07 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-07-01 10:05 [PATCH] gnu: Add cmatrix Cyrill Schenkel
2014-07-01 10:28 ` Ludovic Courtès
2014-07-01 11:07   ` Cyrill Schenkel [this message]
2014-07-01 12:11     ` Ludovic Courtès

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

  List information: https://guix.gnu.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87fvilqqwb.fsf@gmail.com \
    --to=cyrill.schenkel@gmail.com \
    --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 public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).