Subject: Re: [bug#29808] [PATCH] gnu: Add Pioneer.
From: Eric Bavier <[email-masked]>
To: Leo Famulari <[email-masked]>
Cc: [email-masked]
Date: Wed, 27 Dec 2017 08:20:52 -0600
List-Id: <debbugs-submit.debbugs.gnu.org>

On Tue, 26 Dec 2017 14:20:02 -0500
Leo Famulari <[email-masked]> wrote:

> On Thu, Dec 21, 2017 at 05:42:50PM -0600, [email-masked] wrote:
> > From: Eric Bavier <[email-masked]>
> > 
> > * gnu/packages/games.scm (pioneer): New variable.  
> 
> Cool, I love this type of game :)

Me too :)

> 
> > +    (arguments
> > +     `(#:tests? #f                      ;tests are broken
> > +       #:configure-flags (list "--with-external-liblua"
> > +                               (string-append "PIONEER_DATA_DIR="
> > +                                              %output "/share/games/pioneer"))  
> 
> This is just for the game's "assets", right? That is, it doesn't need to
> be written to for things like saving the game?

Right, afaict nothing is written there during gameplay.

> 
> > +       #:phases (modify-phases %standard-phases
> > +                  (add-before 'configure 'bootstrap
> > +                    (lambda _ (zero? (system "./bootstrap"))))  
> 
> Lately we have been doing these bootstrap phases directly after
> unpacking:
> 
> https://lists.gnu.org/archive/html/guix-devel/2017-07/msg00101.html

Good catch.  Thanks.

Fixed and pushed in aac1aa75f4fede7db9fb5be1af344c985fd526f4

`~Eric