diff mbox series

[bug#50952] gnu: Add naev.

Message ID 87mtnso5a5.fsf@nicolasgoaziou.fr
State Accepted
Headers show
Series [bug#50952] gnu: Add naev. | expand

Checks

Context Check Description
cbaines/comparison success View comparision
cbaines/git branch success View Git branch
cbaines/applying patch fail View Laminar job
cbaines/issue success View issue
cbaines/comparison success View comparision
cbaines/git branch success View Git branch
cbaines/applying patch fail View Laminar job
cbaines/issue success View issue
cbaines/comparison success View comparision
cbaines/git branch success View Git branch
cbaines/applying patch fail View Laminar job
cbaines/issue success View issue
cbaines/comparison success View comparision
cbaines/git branch success View Git branch
cbaines/applying patch fail View Laminar job
cbaines/issue success View issue
cbaines/comparison success View comparision
cbaines/git branch success View Git branch
cbaines/applying patch fail View Laminar job
cbaines/issue success View issue

Commit Message

Nicolas Goaziou Oct. 1, 2021, 8:32 p.m. UTC 
Hello,

Here comes a patch for the Naev game.

Regards,

Comments

Ludovic Courtès Oct. 30, 2021, 5:08 p.m. UTC | #1 
Hi,

Nicolas Goaziou <mail@nicolasgoaziou.fr> skribis:

>>From 00610d682ebf6653144d63e4cd09e9d896f3bf5c Mon Sep 17 00:00:00 2001
> From: Nicolas Goaziou <mail@nicolasgoaziou.fr>
> Date: Fri, 1 Oct 2021 22:29:19 +0200
> Subject: [PATCH] gnu: Add naev.
>
> * gnu/packages/games.scm (naev): New variable.

[...]

> +    (build-system meson-build-system)
> +    (arguments
> +     ;; XXX: Do not add debugging symbols, which cause the build to fail.

That sounds weird.  Perhaps ‘--buildtype=release’ does something beyond
removing ‘-g’, which would explain that it doesn’t fail?

Otherwise LGTM!

Ludo’.
Nicolas Goaziou Nov. 2, 2021, 8:33 p.m. UTC | #2 
Hello,

Ludovic Courtès <ludo@gnu.org> writes:

>> Subject: [PATCH] gnu: Add naev.
>>
>> * gnu/packages/games.scm (naev): New variable.
>
> [...]
>
>> +    (build-system meson-build-system)
>> +    (arguments
>> +     ;; XXX: Do not add debugging symbols, which cause the build to fail.
>
> That sounds weird.  Perhaps ‘--buildtype=release’ does something beyond
> removing ‘-g’, which would explain that it doesn’t fail?

I couldn't spot anything suspicious in the code base. However, upstream
mention that "--buildtype=release" should be used for a regular
installation.

> Otherwise LGTM!

Pushed. Thank you!

Regards,
diff mbox series

Patch

From 00610d682ebf6653144d63e4cd09e9d896f3bf5c Mon Sep 17 00:00:00 2001
From: Nicolas Goaziou <mail@nicolasgoaziou.fr>
Date: Fri, 1 Oct 2021 22:29:19 +0200
Subject: [PATCH] gnu: Add naev.

* gnu/packages/games.scm (naev): New variable.
---
 gnu/packages/games.scm | 60 ++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 60 insertions(+)

diff --git a/gnu/packages/games.scm b/gnu/packages/games.scm
index dabbbe6ea1..b3ea7de5dd 100644
--- a/gnu/packages/games.scm
+++ b/gnu/packages/games.scm
@@ -7998,6 +7998,66 @@  ncurses for text display.")
     (home-page "http://frotz.sourceforge.net")
     (license license:gpl2+)))
 
+(define-public naev
+  (package
+    (name "naev")
+    (version "0.8.2")
+    (source
+     (origin
+       (method git-fetch)
+       (uri (git-reference
+             (url "https://github.com/naev/naev")
+             (commit (string-append "v" version))))
+       (file-name (git-file-name name version))
+       (sha256
+        (base32 "02rk2fv2nhx5xsi0cariisamab3dpncwps4q3i3ki0y27xpwxzfx"))))
+    (build-system meson-build-system)
+    (arguments
+     ;; XXX: Do not add debugging symbols, which cause the build to fail.
+     `(#:configure-flags (list "--buildtype=release")
+       #:tests? #f            ;sole test fails with a missing "/dev/dri" error
+       #:meson ,meson-0.55))
+    (native-inputs
+     `(("gettext" ,gettext-minimal)
+       ("pkg-config" ,pkg-config)))
+    (inputs
+     `(("freetype" ,freetype)
+       ("glpk" ,glpk)
+       ("libpng" ,libpng)
+       ("libvorbis" ,libvorbis)
+       ("libwebp" ,libwebp)
+       ("libxml2" ,libxml2)
+       ("luajit" ,luajit)
+       ("openal" ,openal)
+       ("openblas" ,openblas)
+       ("physfs" ,physfs)
+       ("sdl" ,(sdl-union (list sdl2 sdl2-image sdl2-mixer)))
+       ("suitesparse" ,suitesparse)))
+    (home-page "https://naev.org/")
+    (synopsis "Game about space exploration, trade and combat")
+    (description
+     "Naev is a 2d action/rpg space game that combines elements from
+the action, RPG and simulation genres.  You pilot a spaceship from
+a top-down perspective, and are more or less free to do what you want.
+As the genre name implies, you’re able to trade and engage in combat
+at will.  Beyond that, there’s an ever-growing number of story-line
+missions, equipment, and ships; even the galaxy itself grows larger
+with each release.  For the literacy-inclined, there are large amounts
+of lore accompanying everything from planets to equipment.")
+    (license (list license:gpl3
+                   license:public-domain
+                   license:expat        ;edtaa3func.c
+                   license:bsd-2        ;distance_field.c
+                   license:bsd-3        ;perlin.c
+                   ;; Assets.
+                   license:silofl1.1
+                   license:gpl2+
+                   license:cc0
+                   license:cc-by3.0
+                   license:cc-by-sa3.0
+                   license:cc-by4.0
+                   license:cc-by-sa4.0))))
+
 (define-public frotz-dumb-terminal
   (package
     (name "frotz-dumb-terminal")
-- 
2.33.0