emacs-bug-tracker
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#47508: closed ([PATCH] gnu: add instead)


From: GNU bug Tracking System
Subject: bug#47508: closed ([PATCH] gnu: add instead)
Date: Wed, 04 Aug 2021 13:52:02 +0000

Your message dated Wed, 04 Aug 2021 15:50:48 +0200
with message-id 
<4810a8fa8167edd84bfe860acecb1dd9cb81faf8.camel@student.tugraz.at>
and subject line Re: [PATCH] gnu: Add instead.
has caused the debbugs.gnu.org bug report #46378,
regarding [PATCH] gnu: add instead
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs@gnu.org.)


-- 
46378: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=46378
GNU Bug Tracking System
Contact help-debbugs@gnu.org with problems
--- Begin Message --- Subject: [PATCH] gnu: add instead Date: Wed, 31 Mar 2021 02:53:29 +0300
* gnu/packages/games.scm (instead): New variable.
---
 gnu/packages/games.scm | 31 +++++++++++++++++++++++++++++++
 1 file changed, 31 insertions(+)

diff --git a/gnu/packages/games.scm b/gnu/packages/games.scm
index 3f909aad4b..22975cf0d6 100644
--- a/gnu/packages/games.scm
+++ b/gnu/packages/games.scm
@@ -12379,3 +12379,34 @@ game FPS.")
 implemented using ncurses user interface.  An SDL graphical version is also
 available.")
     (license license:gpl3+)))
+
+(define-public instead
+  (package
+    (name "instead")
+    (version "3.3.4")
+    (source
+     (origin
+       (method git-fetch)
+       (uri (git-reference
+             (url "https://github.com/instead-hub/instead";)
+             (commit version)))
+       (file-name (git-file-name name version))
+       (sha256
+        (base32 "09z22i0ny3dhjhj3g1walxg5mdnr3wsx95q1f92z32cabmqkbiyc"))))
+    (build-system cmake-build-system)
+    (arguments
+     `(#:tests? #f))
+    (inputs `(("zlib" ,zlib)
+              ("lua" ,lua)
+              ("sdl2" ,sdl2)
+              ("sdl2-mixer" ,sdl2-mixer)
+              ("sdl2-image" ,sdl2-image)
+              ("sdl2-ttf" ,sdl2-ttf)
+              ("gtk+" ,gtk+)))
+    (native-inputs `(("pkg-config" ,pkg-config)))
+    (home-page "https://instead.hugeping.ru/";)
+    (synopsis "Simple text adventures/visual novels engine")
+    (description "INSTEAD is an INterpreter of Simple TExt ADventure.  Using
+INSTEAD you can create and play games that could be classified as a mix of
+visual novel and text quest.")
+    (license license:expat)))
-- 
2.31.0




--- End Message ---
--- Begin Message --- Subject: Re: [PATCH] gnu: Add instead. Date: Wed, 04 Aug 2021 15:50:48 +0200 User-agent: Evolution 3.34.2
Hi,

I've made a lot of changes to this patch, but given that it has been
around for a while and I didn't want to stall you even further, I've
decided to push the modified version.

Am Mittwoch, den 04.08.2021, 17:29 +0500 schrieb
polyakov@liltechdude.xyz:
> From: Dmitry Polyakov <polyakov@liltechdude.xyz>
> 
> * gnu/local.mk (dist_patch_DATA):
>   Register the patch.
> 
> * gnu/packages/patches/instead-use-games-path.patch:
>   New file.
game-development.scm was not mentioned, also you should mention any
patch before the Makefile it's added to.
> ---
>  gnu/local.mk                                  |  2 +
>  gnu/packages/game-development.scm             | 52
> +++++++++++++++++++
>  .../patches/instead-use-games-path.patch      | 32 ++++++++++++
>  3 files changed, 86 insertions(+)
>  create mode 100644 gnu/packages/patches/instead-use-games-path.patch
> 
> diff --git a/gnu/local.mk b/gnu/local.mk
> index e849480..0c45ae5 100644
> --- a/gnu/local.mk
> +++ b/gnu/local.mk
> @@ -43,6 +43,7 @@
>  # Copyright © 2021 Philip McGrath <philip@philipmcgrath.com>
>  # Copyright © 2021 Arun Isaac <arunisaac@systemreboot.net>
>  # Copyright © 2021 Sharlatan Hellseher <sharlatanus@gmail.com>
> +# Copyright © 2021 Dmitry Polyakov <polyakov@liltechdude.xyz>
>  #
>  # This file is part of GNU Guix.
>  #
> @@ -1261,6 +1262,7 @@ dist_patch_DATA =                               
>               \
>    %D%/packages/patches/imagemagick-WriteTHUMBNAILImage-fix.patch     
> \
>    %D%/packages/patches/inetutils-hurd.patch                  \
>    %D%/packages/patches/inkscape-poppler-0.76.patch           \
> +  %D%/packages/patches/instead-use-games-path.patch          \
>    %D%/packages/patches/inkscape-1.1-fix-build-witch-gcc7.5.patch     
> \
>    %D%/packages/patches/intel-xed-fix-nondeterminism.patch    \
>    %D%/packages/patches/intltool-perl-compatibility.patch     \
> diff --git a/gnu/packages/game-development.scm b/gnu/packages/game-
> development.scm
> index e1593c0..4164b20 100644
> --- a/gnu/packages/game-development.scm
> +++ b/gnu/packages/game-development.scm
> @@ -21,6 +21,7 @@
>  ;;; Copyright © 2020 Timotej Lazar <timotej.lazar@araneo.si>
>  ;;; Copyright © 2020 Giacomo Leidi <goodoldpaul@autistici.org>
>  ;;; Copyright © 2021 Alexandru-Sergiu Marton <brown121407@posteo.ro>
> +;;; Copyright © 2021 Dmitry Polyakov <polyakov@liltechdude.xyz>
>  ;;;
>  ;;; This file is part of GNU Guix.
>  ;;;
> @@ -2115,6 +2116,57 @@ upon which people base their games, ports to
> new platforms, and other
>  projects.")
>        (license license:gpl2))))
>  
> +(define-public instead
> +  (package
> +    (name "instead")
> +    (version "3.3.5")
> +    (build-system cmake-build-system)
> +    (source
> +     (origin
> +       (method git-fetch)
> +       (uri (git-reference
> +             (url "https://github.com/instead-hub/instead";)
> +             (commit version)))
> +       (file-name (git-file-name name version))
> +       (sha256
> +        (base32
> "02j8cw623j51qmr4991i5hsbrzmnp0qfzds8m6nwwr15sjv3hv1g"))
> +       (patches
> +        (search-patches
> +         "instead-use-games-path.patch"))))
I've added a snippet to unbundle zlib.
> +    (arguments
> +     '(#:configure-flags
> +       (list (string-append
> +              "-DLUA_INCLUDE_DIR="
> +              (assoc-ref %build-inputs "luajit") "/include/luajit-
> 2.1/")
> +             "-DWITH_LUAJIT=1"
> +             "-DWITH_GTK3=1")
> +       #:tests? #f))
> +    (inputs
> +     `(("sdl2-images" ,sdl2-image)
> +       ("sdl2-ttf" ,sdl2-ttf)
> +       ("lua" ,lua)
> +       ("pkg-config" ,pkg-config)
> +       ("sdl2-mixer" ,sdl2-mixer)
> +       ("luajit" ,luajit)
> +       ("gtk+",gtk+)))
I've reorganized the imports and added zlib.
> +    (home-page "https://instead3.syscall.ru/";)
> +    (synopsis "Text adventure interpreter")
> +    (description "The STEAD (Simple TExt ADventures) interpreter
> provide
> +functionality to play games that combine visual novels, text
> adventures
> +and classic quests of 1990-s.  STEAD games feature:
> +@enumerate
> +@item Very simple source code of the stories. (based on LUA)
> +@item Graphical game interface
> +@item Supports music and images
> +@item Theme support - a game may change the interface look
> +@end enumerate")
I've shortened the description -- this may be somewhat subjective, but
I find the short one neater.
> +    (license license:gpl2)
License should be Expat, I believe, also I shoved it below the search
path.
> +    (native-search-paths
> +     (list (search-path-specification
> +            (variable "INSTEAD_GAMES_PATH")
> +            (separator #f)                        ;single entry
> +            (files '("share/instead/games")))))))
> +
>  (define-public openvr
>    (package
>      (name "openvr")
> diff --git a/gnu/packages/patches/instead-use-games-path.patch
> b/gnu/packages/patches/instead-use-games-path.patch
> new file mode 100644
> index 0000000..52e96ed
> --- /dev/null
> +++ b/gnu/packages/patches/instead-use-games-path.patch
> @@ -0,0 +1,32 @@
> +From 8b350daa847dd8e131d9e6b5b4434ce68a7903d0 Mon Sep 17 00:00:00
> 2001
> +From: Dmitry Polyakov <polyakov@liltechdude.xyz>
> +Date: Wed, 4 Aug 2021 15:46:33 +0500
> +Subject: [PATCH] [PATCH] Add support for INSTEAD_GAMES_PATH
> +
> +In certain cases, the user needs to determinate by himself where the
> +games will be stored. Using an environment variable is the most
> common
> +way to provide the user with this capability.
I've reworded this to use gender-neutral language.  There are female
and non-binary contributors as well :)
> +---
> + src/main.c | 6 ++++++
> + 1 file changed, 6 insertions(+)
> +
> +diff --git a/src/main.c b/src/main.c
> +index ba0ada1d..b05497b8 100644
> +--- a/src/main.c
> ++++ b/src/main.c
> +@@ -394,6 +394,12 @@ int instead_main(int argc, char *argv[])
> +     setdir(game_cwd);
> +     profile_load(NULL);
> + 
> ++    char *gamespath_env = NULL;
> ++
> ++    if (gamespath_env = getenv("INSTEAD_GAMES_PATH"), gamespath_env
> != NULL) {
> ++      games_sw = strdup(gamespath_env);
> ++    }
> ++
> +     for (i = 1; i < argc; i++) {
> +             if (lua_sw) /* during load profile */
> +                     break;
> +-- 
> +2.32.0
> +

Thanks and have a nice day :)



--- End Message ---

reply via email to

[Prev in Thread] Current Thread [Next in Thread]