GNU bug report logs - #54933
[patch] Add Liquid War 6

Previous Next

Package: guix-patches;

Reported by: zamfofex <zamfofex <at> twdb.moe>

Date: Thu, 14 Apr 2022 10:49:02 UTC

Severity: normal

Tags: patch

Done: Ludovic Courtès <ludo <at> gnu.org>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: zamfofex <zamfofex <at> twdb.moe>
Subject: bug#54933: closed (Re: bug#54933: [patch] Add Liquid War 6)
Date: Fri, 29 Apr 2022 21:15:02 +0000
[Message part 1 (text/plain, inline)]
Your bug report

#54933: [patch] Add Liquid War 6

which was filed against the guix-patches package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to 54933 <at> debbugs.gnu.org.

-- 
54933: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=54933
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems
[Message part 2 (message/rfc822, inline)]
From: Ludovic Courtès <ludo <at> gnu.org>
To: zamfofex <zamfofex <at> twdb.moe>
Cc: 54933-done <at> debbugs.gnu.org
Subject: Re: bug#54933: [patch] Add Liquid War 6
Date: Fri, 29 Apr 2022 23:14:22 +0200
Hi,

zamfofex <zamfofex <at> twdb.moe> skribis:

> diff --git a/gnu/packages/games.scm b/gnu/packages/games.scm
> index de398c297f..8765f70761 100644
> --- a/gnu/packages/games.scm
> +++ b/gnu/packages/games.scm

Applied with mostly cosmetic changes:

> +    (inputs
> +      (list guile-2.0 zlib expat sqlite ncurses readline curl doxygen python-2
> +            libxslt perl graphviz glu libcaca
> +            (sdl-union (list sdl sdl-image sdl-ttf sdl-mixer))))

I passed it through ‘guix style’ and moved ‘doxygen’ to ‘native-inputs’
as suggested by ‘guix lint’.

> +    (arguments
> +      '(#:configure-flags
> +         (list
> +           "--enable-allinone"
> +           "CFLAGS=-Wno-error"
> +           (string-append "CPPFLAGS="
> +             "-I" (search-input-directory %build-inputs "include/SDL")))))

I used a gexp and ‘this-package-input’here.

> +    (description
> +      (string-append
> +        "Liquid War 6 is a unique multiplayer wargame. Your army is a blob of "
> +        "liquid and you have to try and eat your opponents. Rules are very "
> +        "simple yet original, they have been invented by Thomas Colcombet."))

I changed that to a literal string:

  https://guix.gnu.org/manual/devel/en/html_node/Synopses-and-Descriptions.html

For future patches please take a look at:

  https://guix.gnu.org/manual/devel/en/html_node/Submitting-Patches.html

In particular, please use ‘git format-patch’ and possibly ‘git
send-email’ to send patches.  That ensures the commit log and authorship
are preserved and makes the reviewer’s life easier.  :-)

Thanks!

Ludo’.

[Message part 3 (message/rfc822, inline)]
From: zamfofex <zamfofex <at> twdb.moe>
To: "guix-patches <at> gnu.org" <guix-patches <at> gnu.org>
Subject: [patch] Add Liquid War 6
Date: Thu, 14 Apr 2022 07:00:29 -0300 (BRT)
[Message part 4 (text/plain, inline)]

[liquidwar6.diff (text/x-patch, attachment)]

This bug report was last modified 3 years and 27 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.