Subject: Re: [bug#30889] [PATCH] gnu: Add pipewalker.

From: Ricardo Wurmus <[email-masked]>
Date: Thu, 22 Mar 2018 00:22:11 +0100

Hi Tobias,

> * gnu/packages/games.scm (pipewalker): New public variable.
> ---
>
> Guix,
>
> Here's a little game to waste your time while you should be writing
> patches.

Yay!

> +(define-public pipewalker
> +  (package
> +    (name "pipewalker")
> +    (version "0.9.4")
> +    (source
> +     (origin
> +       (method url-fetch)
> +       (uri (string-append "http://downloads.sourceforge.net/pipewalker/"
> +                           name "-" version ".tar.gz"))

Could you use the sourceforge mirror here?

> +    (inputs
> +     `(("libpng" ,libpng)
> +       ("mesa" ,mesa)
> +       ("sdl" ,sdl)))
> +    (arguments
> +     `(#:configure-flags
> +       (list (string-append "--docdir=" (assoc-ref %outputs "out")
> +                            "/share/doc/" ,name "-" ,version))
> +       #:phases
> +       (modify-phases %standard-phases
> +         (add-after 'configure 'patch-docdir
> +           ;; Makefile.in ignores configure's ‘--docdir=...’ option.  Fix that.
> +           (lambda _
> +             (substitute* "Makefile"
> +               (("(pkgdocdatadir = ).*" _ key)
> +                (string-append key "$(docdir)\n")))
> +             #t)))))

It’s just a minor irritation for me, but I always expect to see the
“arguments” field right below the build system field, because it defines
arguments for the build system.

> +    (home-page "http://pipewalker.sourceforge.net/")
> +    (synopsis "Logical tile puzzle")
> +    (description
> +     "PipeWalker is a simple puzzle game with many diffent themes: connect all
> +computers to one network server, bring water from a source to the taps, etc.
> +The underlying mechanism is always the same: you must turn each tile in the
> +grid in the right direction to combine all components into a single circuit.
> +Every puzzle has a complete solution, although there may be more than
> one.")

Sounds like fun!

-- 
Ricardo

GPG: BCA6 89B6 3655 3801 C3C6  2150 197A 5888 235F ACAC
https://elephly.net