Message ID | E1gvNRB-0003UF-5z@rmmprod05.runbox |
---|---|
State | Accepted |
Headers | show |
Series | [bug#34497] Add odamex | expand |
Context | Check | Description |
---|---|---|
cbaines/applying patch | fail | Apply failed |
Hi pkill9, > * gnu/packages/games.scm (odamex): New variable. […] > + (uri (string-append > + "mirror://sourceforge/odamex/Odamex/" > + version > + "/odamex-src-" > + version > + ".tar.gz")) Please put this on fewer lines. > + (sha256 > + (base32 > + "1sh6lqj7vsdmnqz17hw0b6vy7xx6dp41k2sdw99ympsfa2xd1d2j")))) > + (build-system cmake-build-system) > + (arguments `(#:tests? #f)) ;; No tests Please use just one semicolon. > + (inputs > + `(("sdl" ,(@ (gnu packages sdl) sdl)) > + ("sdl-mixer" ,(@ (gnu packages sdl) sdl-mixer)) > + ("zlib" ,(@ (gnu packages compression) zlib)) > + ("libpng" ,(@ (gnu packages image) libpng)) > + ("alsa-lib" ,(@ (gnu packages linux) alsa-lib)))) This is not okay. Please just use the plain variable names. > + (home-page "https://odamex.net/") > + (synopsis "Multiplayer Doom port") > + (description "Odamex is a free, cross-platform modification of the Doom > +engine that allows players to easily join servers dedicated to playing Doom > +online.") Please remove “free” — all software in Guix is Free Software. > + (license license:gpl2))) Looks like this is actually gpl2+ as evidenced by files like this: https://github.com/odamex/odamex/blob/master/server/src/d_netinfo.cpp#L12 Could you please send an updated patch?
From 8613bd3ae3f73cd45fd1b17ee5e9f3d3e23b76fd Mon Sep 17 00:00:00 2001 From: Pkill -9 <pkill9@runbox.com> Date: Sat, 16 Feb 2019 05:57:07 +0000 Subject: [PATCH] gnu: Add odamex. * gnu/packages/games.scm (odamex): 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 6d9e63c4e..9dc19bda1 100644 --- a/gnu/packages/games.scm +++ b/gnu/packages/games.scm @@ -5378,6 +5378,37 @@ Strife, Chex Quest, and fan-created games like Harmony, Hacx and Freedoom.") "file://dumb/licence.txt" "Dumb license, explicitly GPL compatible."))))) +(define-public odamex + (package + (name "odamex") + (version "0.8.0") + (source + (origin + (method url-fetch) + (uri (string-append + "mirror://sourceforge/odamex/Odamex/" + version + "/odamex-src-" + version + ".tar.gz")) + (sha256 + (base32 + "1sh6lqj7vsdmnqz17hw0b6vy7xx6dp41k2sdw99ympsfa2xd1d2j")))) + (build-system cmake-build-system) + (arguments `(#:tests? #f)) ;; No tests + (inputs + `(("sdl" ,(@ (gnu packages sdl) sdl)) + ("sdl-mixer" ,(@ (gnu packages sdl) sdl-mixer)) + ("zlib" ,(@ (gnu packages compression) zlib)) + ("libpng" ,(@ (gnu packages image) libpng)) + ("alsa-lib" ,(@ (gnu packages linux) alsa-lib)))) + (home-page "https://odamex.net/") + (synopsis "Multiplayer Doom port") + (description "Odamex is a free, cross-platform modification of the Doom +engine that allows players to easily join servers dedicated to playing Doom +online.") + (license license:gpl2))) + (define-public fortune-mod (package (name "fortune-mod") -- 2.20.1