[PATCH] gnu: Add minetest-advtrains.

  • Done
  • quality assurance status badge
Details
4 participants
  • Ludovic Courtès
  • Maxime Devos
  • Xinglu Chen
  • Vivien Kraus
Owner
unassigned
Submitted by
Vivien Kraus
Severity
normal
V
V
Vivien Kraus wrote on 6 Sep 2021 10:53
(address . guix-patches@gnu.org)
87a6kqm6f3.fsf@planete-kraus.eu
Dear Guix,

I’d like to have the advtrains minetest mod in Guix, what do you think?

Best regards,

Vivien
From 2a0abac482a9a40a1263419766d23a169ec61e89 Mon Sep 17 00:00:00 2001
From: Vivien Kraus <vivien@planete-kraus.eu>
Date: Sun, 5 Sep 2021 15:21:35 +0200
Subject: [PATCH] gnu: add minetest-advtrains

* gnu/packages/minetest.scm (minetest-advtrains): New variable.
* gnu/packages/minetest.scm (minetest-basic-trains): New variable.
---
gnu/packages/minetest.scm | 61 +++++++++++++++++++++++++++++++++++++++
1 file changed, 61 insertions(+)

Toggle diff (71 lines)
diff --git a/gnu/packages/minetest.scm b/gnu/packages/minetest.scm
index 5453f4d16a..13ea6036f5 100644
--- a/gnu/packages/minetest.scm
+++ b/gnu/packages/minetest.scm
@@ -624,3 +624,64 @@ track of important locations.")
license:cc-by4.0 license:cc-by-sa3.0 license:public-domain
license:cc0 license:fdl1.2+))
(properties `((upstream-name . "RealBadAngel/unified_inventory")))))
+
+(define-public minetest-advtrains
+ (package
+ (name "minetest-advtrains")
+ (version "2.3.1")
+ (source
+ (origin
+ (method git-fetch)
+ (uri (git-reference
+ (url "https://git.bananach.space/advtrains.git")
+ (commit "release-2.3.1")))
+ (sha256
+ (base32 "1ijqlchh269jpvmgmdmdvy3nsnk0bszkvvcqk6vaysvxam695ggw"))
+ (file-name (git-file-name name version))))
+ (build-system minetest-mod-build-system)
+ (home-page "http://advtrains.de/")
+ (synopsis
+ "Adds good-looking, realistic trains with realistic rails.")
+ (description
+ "This mod features realistic trains and various equipment for railways, with a focus on automated train operation.
+
+Main features:
+
+- Almost-realistic tracks with actual curves and switches
+- Railway signals, controllable by various means
+- ATC: Simple, command-like automatic train control
+- LuaATC: Powerful Lua-scripted automatic train operation(requires some programming knowledge)*
+- An Interlocking system, featuring track sections, routes and automatic stopping before signals")
+ (license
+ (list license:cc-by-sa3.0 license:agpl3))
+ (properties
+ `((upstream-name . "orwell/advtrains")))))
+
+(define-public minetest-basic-trains
+ (package
+ (name "minetest-basic-trains")
+ (version "1.0.1")
+ (source
+ (origin
+ (method git-fetch)
+ (uri (git-reference
+ (url "http://git.bananach.space/basic_trains.git/")
+ (commit
+ "d44c410f7c2a7202ee68b66fc50febae89e0c5dc")))
+ (sha256
+ (base32
+ "0vvzndj48kgdz2bfgivfm217sbmc2lmxpp2mispcy7byn4i26prx"))
+ (file-name (git-file-name name version))))
+ (build-system minetest-mod-build-system)
+ (propagated-inputs
+ `(("minetest-advtrains" ,minetest-advtrains)))
+ (home-page
+ "http://advtrains.de/wiki/doku.php?id=usage:trains:basic_trains")
+ (synopsis
+ "Collection of basic trains for the Advanced Trains mod. Formerly included in main advtrains modpack (until 2.3.0)")
+ (description
+ "This modpack contains the trains which were the \"default\" trains in advtrains up to version 2.2.1.")
+ (license
+ (list license:cc-by-sa3.0 license:agpl3))
+ (properties
+ `((upstream-name . "orwell/basic_trains")))))
--
2.33.0
M
M
Maxime Devos wrote on 6 Sep 2021 13:02
720fc481c64b06abd22b9b3004488052efcb48dc.camel@telenet.be

M
M
Maxime Devos wrote on 6 Sep 2021 13:03
682dea6d53dd6435db1ab060ae0c5fdabda19750.camel@telenet.be
Maxime Devos schreef op ma 06-09-2021 om 13:02 [+0200]:
Toggle quote (2 lines)
>

Please ignore, I accidentally pushed 'Sent' before writing
anything.
-----BEGIN PGP SIGNATURE-----

iI0EABYKADUWIQTB8z7iDFKP233XAR9J4+4iGRcl7gUCYTX1nBccbWF4aW1lZGV2
b3NAdGVsZW5ldC5iZQAKCRBJ4+4iGRcl7r/MAP0Upke1pfH02qkUtcBJnXPKl5rm
iGDVJYSDxxZaBgfSAwD+JRePi4gOWLFHY1pXDGhGJNwayrP5oUdMQTXl5VllkAU=
=M3TM
-----END PGP SIGNATURE-----


M
M
Maxime Devos wrote on 6 Sep 2021 13:17
031b52f03cac2218f9250882e5b8d3fe56cc7f49.camel@telenet.be
Vivien Kraus via Guix-patches via schreef op ma 06-09-2021 om 10:53 [+0200]:
Toggle quote (4 lines)
> Guix,
>
> I’d like to have the advtrains minetest mod in Guix, what do you think?

Did the minetest importer work well for you?

Toggle quote (5 lines)
> +- Almost-realistic tracks with actual curves and switches
> +- Railway signals, controllable by various means
> +- ATC: Simple, command-like automatic train control
> +- LuaATC: Powerful Lua-scripted automatic train operation(requires some programming knowledge)*

Package descriptions are in texinfo, not markdown. You'll need
to make the necessary adjustments. Being proper Texinfo is necessary

(It would be nice if guix could automatically convert the markup ...)

Toggle quote (3 lines)
> + (license
> + (list license:cc-by-sa3.0 license:agpl3))

it's agpl3+ actually. I now that the ContentDB page (and hence, the importer)
says it's AGPL-3.0-only, but ContentDB only very recently distinguishes between
AGPL-3.0-only and AGPL-3.0-or-later. I sent a bug report upstream.

Greetings,
Maxime.
-----BEGIN PGP SIGNATURE-----

iI0EABYKADUWIQTB8z7iDFKP233XAR9J4+4iGRcl7gUCYTX4wBccbWF4aW1lZGV2
b3NAdGVsZW5ldC5iZQAKCRBJ4+4iGRcl7tvGAQCZZKk4a5aH1ilpnBM11dtqexeg
09HqDF9D1FUmSS7JAwD/Wtd1KE8+ue/++MGIkgh3S+SFZSOloi/cDxoi/NOUnwI=
=WzUH
-----END PGP SIGNATURE-----


V
V
Vivien Kraus wrote on 6 Sep 2021 13:56
b4c0387688c3197484f96ea46f2c800b64b77faf.camel@planete-kraus.eu
Hello Maxime,

Le lundi 06 septembre 2021 à 13:17 +0200, Maxime Devos a écrit :
Toggle quote (1 lines)
> Did the minetest importer work well for you?
I noticed 3 minor things:
1. The advtrain dependency on basic-trains was duplicated, I had to
remove one occurence;
2. The version number came prefixed with a "v" that I had to remove;
3. It was not able to get the commit ID and the hash for minetest-
advtrains, I had to fill it in myself (it was set to #f). Presumably
because of the exotic tag name.

That’s not counting the markdown and license issues, these are out of
scope indeed.

Toggle quote (2 lines)
> Package descriptions are in texinfo, not markdown. You'll need
> to make the necessary adjustments.
I updated the patch to account for an @itemize instead of dash
enumerations, I think it was the only thing missing.

Toggle quote (4 lines)
> According to
> http://git.bananach.space/basic_trains.git/tree/license.txt,
> it's agpl3+ actually.

You’re right, advtrains too:
I updated the patch.

Also, when I run ./etc/indent-code.el gnu/packages/minetest, there are
some re-indentations that happen for othe minetest packages. Thus, I’m
not sure what indentation and general style I should adopt. Hopefully
this problem will be fixed by guix style :)

Vivien
From 36b3170366205e761c57665f19dba1a581ca9053 Mon Sep 17 00:00:00 2001
From: Vivien Kraus <vivien@planete-kraus.eu>
Date: Sun, 5 Sep 2021 15:21:35 +0200
Subject: [PATCH] gnu: add minetest-advtrains

* gnu/packages/minetest.scm (minetest-advtrains): New variable.
* gnu/packages/minetest.scm (minetest-basic-trains): New variable.
---
gnu/packages/minetest.scm | 68 +++++++++++++++++++++++++++++++++++++++
1 file changed, 68 insertions(+)

Toggle diff (78 lines)
diff --git a/gnu/packages/minetest.scm b/gnu/packages/minetest.scm
index 5453f4d16a..80151b8de6 100644
--- a/gnu/packages/minetest.scm
+++ b/gnu/packages/minetest.scm
@@ -624,3 +624,71 @@ track of important locations.")
license:cc-by4.0 license:cc-by-sa3.0 license:public-domain
license:cc0 license:fdl1.2+))
(properties `((upstream-name . "RealBadAngel/unified_inventory")))))
+
+(define-public minetest-advtrains
+ (package
+ (name "minetest-advtrains")
+ (version "2.3.1")
+ (source
+ (origin
+ (method git-fetch)
+ (uri (git-reference
+ (url "https://git.bananach.space/advtrains.git")
+ (commit "release-2.3.1")))
+ (sha256
+ (base32 "1ijqlchh269jpvmgmdmdvy3nsnk0bszkvvcqk6vaysvxam695ggw"))
+ (file-name (git-file-name name version))))
+ (build-system minetest-mod-build-system)
+ (home-page "http://advtrains.de/")
+ (synopsis
+ "Adds good-looking, realistic trains with realistic rails.")
+ (description
+ "This mod features realistic trains and various equipment for railways, with a focus on automated train operation.
+
+Main features:
+
+@itemize
+@item
+almost-realistic tracks with actual curves and switches;
+@item
+railway signals, controllable by various means;
+@item
+ATC: simple, command-like automatic train control;
+@item
+LuaATC: Powerful Lua-scripted automatic train operation (requires some programming knowledge);
+@item
+an interlocking system, featuring track sections, routes and automatic stopping before signals.
+@end itemize")
+ (license
+ (list license:cc-by-sa3.0 license:agpl3+))
+ (properties
+ `((upstream-name . "orwell/advtrains")))))
+
+(define-public minetest-basic-trains
+ (package
+ (name "minetest-basic-trains")
+ (version "1.0.1")
+ (source
+ (origin
+ (method git-fetch)
+ (uri (git-reference
+ (url "http://git.bananach.space/basic_trains.git/")
+ (commit
+ "d44c410f7c2a7202ee68b66fc50febae89e0c5dc")))
+ (sha256
+ (base32
+ "0vvzndj48kgdz2bfgivfm217sbmc2lmxpp2mispcy7byn4i26prx"))
+ (file-name (git-file-name name version))))
+ (build-system minetest-mod-build-system)
+ (propagated-inputs
+ `(("minetest-advtrains" ,minetest-advtrains)))
+ (home-page
+ "http://advtrains.de/wiki/doku.php?id=usage:trains:basic_trains")
+ (synopsis
+ "Collection of basic trains for the Advanced Trains mod. Formerly included in main advtrains modpack (until 2.3.0)")
+ (description
+ "This modpack contains the trains which were the \"default\" trains in advtrains up to version 2.2.1.")
+ (license
+ (list license:cc-by-sa3.0 license:agpl3+))
+ (properties
+ `((upstream-name . "orwell/basic_trains")))))
--
2.33.0
M
M
Maxime Devos wrote on 7 Sep 2021 12:20
f43eb8064e9e5e06c29a87194faaf7e787ee9721.camel@telenet.be
Vivien Kraus schreef op ma 06-09-2021 om 13:56 [+0200]:
Toggle quote (8 lines)
> Hello Maxime,
>
> Le lundi 06 septembre 2021 à 13:17 +0200, Maxime Devos a écrit :
> > Did the minetest importer work well for you?
> I noticed 3 minor things:
> 1. The advtrain dependency on basic-trains was duplicated, I had to
> remove one occurence;

I can reproduce this with ‘guix import minetest --recursive orwell/basic_trains’.
Presumably the issue is that advtrains is listed twice in the dependencies
on ContentDB, as ‘advtrains’ and ‘advtrains_train_track’. I'll send a patch
to remove duplicates.

Toggle quote (2 lines)
> 2. The version number came prefixed with a "v" that I had to remove;

The importer uses the ‘release title’ as version number. Apparently
the release titles for orwell/advtrains include a "v" prefix. I'll
send a patch to remove the prefix.

Toggle quote (4 lines)
> 3. It was not able to get the commit ID and the hash for minetest-
> advtrains, I had to fill it in myself (it was set to #f). Presumably
> because of the exotic tag name.

FWIW, the minetest importer doesn't look at git tag names, it just copies
the commit from ContentDB. And in case of advtrains, the commit is
not available there, so the importer sets it to #false.

Toggle quote (7 lines)
> [...]
>
> Also, when I run ./etc/indent-code.el gnu/packages/minetest, there are
> some re-indentations that happen for othe minetest packages. Thus, I’m
> not sure what indentation and general style I should adopt. Hopefully
> this problem will be fixed by guix style :)

The indentation from "./etc/indent-code.el" should be good. Feel free
to precedede the ‘gnu: Add minetest-advtrains’ patch with a patch fixing
the indentation.

Greetings,
Maime.
-----BEGIN PGP SIGNATURE-----

iI0EABYKADUWIQTB8z7iDFKP233XAR9J4+4iGRcl7gUCYTc9ChccbWF4aW1lZGV2
b3NAdGVsZW5ldC5iZQAKCRBJ4+4iGRcl7mc6AP9PWWXF38dKNjLmkU5p8jfj0lbt
cgH/8bVl1QgCCXxcCAD+KcUdUg+A6AgweGaHeK3a/8Msl7QiZNldgbgHWDwHowc=
=shVi
-----END PGP SIGNATURE-----


V
V
Vivien Kraus wrote on 7 Sep 2021 12:45
e7485e6dbbafc68831f37ebd09062f85fd734896.camel@planete-kraus.eu
Hello Maxime,

Le mardi 07 septembre 2021 à 12:20 +0200, Maxime Devos a écrit :
Toggle quote (6 lines)
> The indentation from "./etc/indent-code.el" should be good. Feel
> free
> to precedede the ‘gnu: Add minetest-advtrains’ patch with a patch
> fixing
> the indentation.

Since the indentation only changed for 2 packages, I made 2 commits,
one for each package. This amounts to 3 commits (they should commute):
1. Fix indentation for minetest-data;
2. Fix indentation for minetest-mesecons;
3. Add minetest-advtrains.

Vivien
From 791ad5843d8175d569132697aff264c633d2b112 Mon Sep 17 00:00:00 2001
From: Vivien Kraus <vivien@planete-kraus.eu>
Date: Sun, 5 Sep 2021 15:21:35 +0200
Subject: [PATCH 3/3] gnu: add minetest-advtrains

* gnu/packages/minetest.scm (minetest-advtrains): New variable.
* gnu/packages/minetest.scm (minetest-basic-trains): New variable.
---
gnu/packages/minetest.scm | 68 +++++++++++++++++++++++++++++++++++++++
1 file changed, 68 insertions(+)

Toggle diff (78 lines)
diff --git a/gnu/packages/minetest.scm b/gnu/packages/minetest.scm
index 9e48d5c69c..ad0b70195d 100644
--- a/gnu/packages/minetest.scm
+++ b/gnu/packages/minetest.scm
@@ -624,3 +624,71 @@ track of important locations.")
license:cc-by4.0 license:cc-by-sa3.0 license:public-domain
license:cc0 license:fdl1.2+))
(properties `((upstream-name . "RealBadAngel/unified_inventory")))))
+
+(define-public minetest-advtrains
+ (package
+ (name "minetest-advtrains")
+ (version "2.3.1")
+ (source
+ (origin
+ (method git-fetch)
+ (uri (git-reference
+ (url "https://git.bananach.space/advtrains.git")
+ (commit "release-2.3.1")))
+ (sha256
+ (base32 "1ijqlchh269jpvmgmdmdvy3nsnk0bszkvvcqk6vaysvxam695ggw"))
+ (file-name (git-file-name name version))))
+ (build-system minetest-mod-build-system)
+ (home-page "http://advtrains.de/")
+ (synopsis
+ "Adds good-looking, realistic trains with realistic rails.")
+ (description
+ "This mod features realistic trains and various equipment for railways, with a focus on automated train operation.
+
+Main features:
+
+@itemize
+@item
+almost-realistic tracks with actual curves and switches;
+@item
+railway signals, controllable by various means;
+@item
+ATC: simple, command-like automatic train control;
+@item
+LuaATC: Powerful Lua-scripted automatic train operation (requires some programming knowledge);
+@item
+an interlocking system, featuring track sections, routes and automatic stopping before signals.
+@end itemize")
+ (license
+ (list license:cc-by-sa3.0 license:agpl3+))
+ (properties
+ `((upstream-name . "orwell/advtrains")))))
+
+(define-public minetest-basic-trains
+ (package
+ (name "minetest-basic-trains")
+ (version "1.0.1")
+ (source
+ (origin
+ (method git-fetch)
+ (uri (git-reference
+ (url "http://git.bananach.space/basic_trains.git/")
+ (commit
+ "d44c410f7c2a7202ee68b66fc50febae89e0c5dc")))
+ (sha256
+ (base32
+ "0vvzndj48kgdz2bfgivfm217sbmc2lmxpp2mispcy7byn4i26prx"))
+ (file-name (git-file-name name version))))
+ (build-system minetest-mod-build-system)
+ (propagated-inputs
+ `(("minetest-advtrains" ,minetest-advtrains)))
+ (home-page
+ "http://advtrains.de/wiki/doku.php?id=usage:trains:basic_trains")
+ (synopsis
+ "Collection of basic trains for the Advanced Trains mod. Formerly included in main advtrains modpack (until 2.3.0)")
+ (description
+ "This modpack contains the trains which were the \"default\" trains in advtrains up to version 2.2.1.")
+ (license
+ (list license:cc-by-sa3.0 license:agpl3+))
+ (properties
+ `((upstream-name . "orwell/basic_trains")))))
--
2.33.0
From 2c86184c6ad043aa87bbddc10ed3c4f63e6e690e Mon Sep 17 00:00:00 2001
From: Vivien Kraus <vivien@planete-kraus.eu>
Date: Tue, 7 Sep 2021 12:28:35 +0200
Subject: [PATCH 2/3] gnu: minetest-mesecons: Fix indentation.

* gnu/packages/minetest.scm (minetest-mesecons): Fix indentation.
---
gnu/packages/minetest.scm | 50 +++++++++++++++++++--------------------
1 file changed, 25 insertions(+), 25 deletions(-)

Toggle diff (69 lines)
diff --git a/gnu/packages/minetest.scm b/gnu/packages/minetest.scm
index 6f4682ba47..9e48d5c69c 100644
--- a/gnu/packages/minetest.scm
+++ b/gnu/packages/minetest.scm
@@ -304,37 +304,37 @@ and a variety of other stuff.")
;; The release on ContentDB does not have its own version number.
(let ((commit "db5879706d04d3480bc4863ce0c03fa73e5f10c7")
(revision "0"))
- (package
- (name "minetest-mesecons")
- (version (git-version "1.2.1" revision commit))
- (source
- (origin
- (method git-fetch)
- (uri (git-reference
- (url "https://github.com/minetest-mods/mesecons")
- (commit commit)))
- (sha256
- (base32 "04m9s9l3frw1lgki41hgvjsw2zkrvfv0sy750b6j12arzb3lv645"))
- (file-name (git-file-name name version))))
- (build-system minetest-mod-build-system)
- (home-page "https://mesecons.net")
- (synopsis
- "Digital circuitry for Minetest, including wires, buttons and lights")
- (description
- "Mesecons is a mod for Minetest implementing various items related
+ (package
+ (name "minetest-mesecons")
+ (version (git-version "1.2.1" revision commit))
+ (source
+ (origin
+ (method git-fetch)
+ (uri (git-reference
+ (url "https://github.com/minetest-mods/mesecons")
+ (commit commit)))
+ (sha256
+ (base32 "04m9s9l3frw1lgki41hgvjsw2zkrvfv0sy750b6j12arzb3lv645"))
+ (file-name (git-file-name name version))))
+ (build-system minetest-mod-build-system)
+ (home-page "https://mesecons.net")
+ (synopsis
+ "Digital circuitry for Minetest, including wires, buttons and lights")
+ (description
+ "Mesecons is a mod for Minetest implementing various items related
to digital circuitry, such as wires, buttons, lights and programmable
controllers. Among other things, there are also pistons, solar panels,
pressure plates and note blocks.
Mesecons has a similar goal to Redstone in Minecraft, but works in its own way,
with different rules and mechanics.")
- ;; LGPL for code, CC-BY-SA for textures.
- ;; The README.md and COPYING.txt disagree about the "+" in license:lgpl3+.
- ;; For now, assume README.md is correct. Upstream has been asked to
- ;; correct the inconsistency:
- ;; <https://github.com/minetest-mods/mesecons/issues/575>.
- (license (list license:lgpl3+ license:cc-by-sa3.0))
- (properties `((upstream-name . "Jeija/mesecons"))))))
+ ;; LGPL for code, CC-BY-SA for textures.
+ ;; The README.md and COPYING.txt disagree about the "+" in license:lgpl3+.
+ ;; For now, assume README.md is correct. Upstream has been asked to
+ ;; correct the inconsistency:
+ ;; <https://github.com/minetest-mods/mesecons/issues/575>.
+ (license (list license:lgpl3+ license:cc-by-sa3.0))
+ (properties `((upstream-name . "Jeija/mesecons"))))))
(define-public minetest-mineclone
(package
--
2.33.0
From a97f8395a685b688b240f28089f368b9065d1d61 Mon Sep 17 00:00:00 2001
From: Vivien Kraus <vivien@planete-kraus.eu>
Date: Tue, 7 Sep 2021 12:27:59 +0200
Subject: [PATCH 1/3] gnu: minetest-data: Fix indentation.

* gnu/packages/minetest.scm (minetest-data): Fix indentation.
---
gnu/packages/minetest.scm | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)

Toggle diff (28 lines)
diff --git a/gnu/packages/minetest.scm b/gnu/packages/minetest.scm
index 5453f4d16a..6f4682ba47 100644
--- a/gnu/packages/minetest.scm
+++ b/gnu/packages/minetest.scm
@@ -154,8 +154,8 @@ in different ways.")
(source (origin
(method git-fetch)
(uri (git-reference
- (url "https://github.com/minetest/minetest_game")
- (commit version)))
+ (url "https://github.com/minetest/minetest_game")
+ (commit version)))
(file-name (git-file-name name version))
(sha256
(base32
@@ -172,8 +172,8 @@ in different ways.")
"/share/minetest/games/minetest_game")))
(mkdir-p install-dir)
(copy-recursively
- (assoc-ref %build-inputs "source")
- install-dir)
+ (assoc-ref %build-inputs "source")
+ install-dir)
#t))))
(synopsis "Main game data for the Minetest game engine")
(description
--
2.33.0
M
M
Maxime Devos wrote on 10 Sep 2021 12:25
4e2998599854e1f51347800600c97bf840bf97ba.camel@telenet.be
Hi,

Presuming minetest-advtrains and minetest-basic-trains build and actually
work, this largely LGTM. I looked at the source code of advtrains and basic_trains
and didn't find anything ‘suspicious’ (e.g. malware, though mods are sandboxed
so they would have a hard time doing anything bad).

minetest-advtrains and minetest-basic-trains are separate packages that
can be added separately, so theoretically, they should be added in separate
patches:

From (guix)Submitting Patches:
11. Verify that your patch contains only one set of related changes.
Bundling unrelated changes together makes reviewing harder and
slower.

(minetest-advtrains seems rather useless without minetest-basic-trains though,
so maybe it's ok?)

Vivien Kraus schreef op di 07-09-2021 om 12:45 [+0200]:
Toggle quote (20 lines)
> +(define-public minetest-advtrains
> + (package
> + (name "minetest-advtrains")
> + (version "2.3.1")
> + (source
> + (origin
> + (method git-fetch)
> + (uri (git-reference
> + (url "https://git.bananach.space/advtrains.git")
> + (commit "release-2.3.1")))
> + (sha256
> + (base32 "1ijqlchh269jpvmgmdmdvy3nsnk0bszkvvcqk6vaysvxam695ggw"))
> + (file-name (git-file-name name version))))
> + (build-system minetest-mod-build-system)
> + (home-page "http://advtrains.de/")
> + (synopsis
> + "Adds good-looking, realistic trains with realistic rails.")
> + (description
> + "This mod features realistic trains and various equipment for railways, with a focus on automated train operation.

Aren't the actual trains added with minetest-basic-trains?
Maybe replace ‘realistic trains’ -> ‘realistic train system’,
and note that minetest-basic-trains contains the actual train models.

I think a linter will have something to say about long lines here
(try "./pre-inst-env guix lint minetest-advtrains minetest-basic-trains").

Toggle quote (16 lines)
> +
> +Main features:
> +
> +@itemize
> +@item
> +almost-realistic tracks with actual curves and switches;
> +@item
> +railway signals, controllable by various means;
> +@item
> +ATC: simple, command-like automatic train control;
> +@item
> +LuaATC: Powerful Lua-scripted automatic train operation (requires some programming knowledge);
> +@item
> +an interlocking system, featuring track sections, routes and automatic stopping before signals.
> +@end itemize")

Greetings,
Maxime.
-----BEGIN PGP SIGNATURE-----

iI0EABYKADUWIQTB8z7iDFKP233XAR9J4+4iGRcl7gUCYTsymhccbWF4aW1lZGV2
b3NAdGVsZW5ldC5iZQAKCRBJ4+4iGRcl7mXaAP9qQDgfB7SAjrbIaTK/91EUT0wc
S5ORvp0NR66n0CAmNwD8CkCWqnNHV5oe33T0VOqBreKc/hNLxpgXuieFainPUQI=
=U2os
-----END PGP SIGNATURE-----


V
V
Vivien Kraus wrote on 10 Sep 2021 14:46
f2ffd872759461a313a2449591830e19f90e565b.camel@planete-kraus.eu
Hello Maxime,

Le vendredi 10 septembre 2021 à 12:25 +0200, Maxime Devos a écrit :
Toggle quote (5 lines)
> minetest-advtrains and minetest-basic-trains are separate packages
> that
> can be added separately, so theoretically, they should be added in
> separate
> patches:
It seems like the least surprising thing to do, so let’s go with the
split patch.

Toggle quote (1 lines)
> Maybe replace ‘realistic trains’ -> ‘realistic train system’,
I’m not sure. Upstream really likes to use "train" as a general term,
like in share/minetest/mods/advtrains/modpack.conf, the official
description, the wiki, ... so I think it’s better to use that.

Toggle quote (1 lines)
> and note that minetest-basic-trains contains the actual train models.
This sentence was missing, and I added it.

Toggle quote (3 lines)
> I think a linter will have something to say about long lines here
> (try "./pre-inst-env guix lint minetest-advtrains minetest-basic-
> trains").
You’re right, I fixed the long lines and other warnings. However, I
don’t know how to fix "no updater for minetest-xxx" for both minetest-
advtrains and minetest-basic-trains. I can’t see a meaningful
difference between my packages and the others.

Vivien
From fbe39d93acb01fbef9f7f497e9bb5814fdf3f6b5 Mon Sep 17 00:00:00 2001
From: Vivien Kraus <vivien@planete-kraus.eu>
Date: Fri, 10 Sep 2021 14:21:47 +0200
Subject: [PATCH 4/4] gnu: add minetest-basic-trains

* gnu/packages/minetest.scm (minetest-basic-trains): New variable.
---
gnu/packages/minetest.scm | 32 ++++++++++++++++++++++++++++++++
1 file changed, 32 insertions(+)

Toggle diff (42 lines)
diff --git a/gnu/packages/minetest.scm b/gnu/packages/minetest.scm
index 0b0e7ace96..c218fb827f 100644
--- a/gnu/packages/minetest.scm
+++ b/gnu/packages/minetest.scm
@@ -668,3 +668,35 @@ stopping before signals.
(list license:cc-by-sa3.0 license:agpl3+))
(properties
`((upstream-name . "orwell/advtrains")))))
+
+(define-public minetest-basic-trains
+ (package
+ (name "minetest-basic-trains")
+ (version "1.0.1")
+ (source
+ (origin
+ (method git-fetch)
+ (uri (git-reference
+ (url "http://git.bananach.space/basic_trains.git/")
+ (commit
+ "d44c410f7c2a7202ee68b66fc50febae89e0c5dc")))
+ (sha256
+ (base32
+ "0vvzndj48kgdz2bfgivfm217sbmc2lmxpp2mispcy7byn4i26prx"))
+ (file-name (git-file-name name version))))
+ (build-system minetest-mod-build-system)
+ (propagated-inputs
+ `(("minetest-advtrains" ,minetest-advtrains)))
+ (home-page
+ "http://advtrains.de/wiki/doku.php?id=usage:trains:basic_trains")
+ (synopsis
+ "\
+Collection of basic trains for the Advanced Trains mod")
+ (description
+ "\
+This modpack contains the trains which were the \"default\" trains in
+advtrains up to version 2.2.1.")
+ (license
+ (list license:cc-by-sa3.0 license:agpl3+))
+ (properties
+ `((upstream-name . "orwell/basic_trains")))))
--
2.33.0
From 9675601acba125905877f30b5bc47053db3b652d Mon Sep 17 00:00:00 2001
From: Vivien Kraus <vivien@planete-kraus.eu>
Date: Sun, 5 Sep 2021 15:21:35 +0200
Subject: [PATCH 3/4] gnu: add minetest-advtrains

* gnu/packages/minetest.scm (minetest-advtrains): New variable.
---
gnu/packages/minetest.scm | 44 +++++++++++++++++++++++++++++++++++++++
1 file changed, 44 insertions(+)

Toggle diff (54 lines)
diff --git a/gnu/packages/minetest.scm b/gnu/packages/minetest.scm
index 9e48d5c69c..0b0e7ace96 100644
--- a/gnu/packages/minetest.scm
+++ b/gnu/packages/minetest.scm
@@ -624,3 +624,47 @@ track of important locations.")
license:cc-by4.0 license:cc-by-sa3.0 license:public-domain
license:cc0 license:fdl1.2+))
(properties `((upstream-name . "RealBadAngel/unified_inventory")))))
+
+(define-public minetest-advtrains
+ (package
+ (name "minetest-advtrains")
+ (version "2.3.1")
+ (source
+ (origin
+ (method git-fetch)
+ (uri (git-reference
+ (url "https://git.bananach.space/advtrains.git")
+ (commit "release-2.3.1")))
+ (sha256
+ (base32 "1ijqlchh269jpvmgmdmdvy3nsnk0bszkvvcqk6vaysvxam695ggw"))
+ (file-name (git-file-name name version))))
+ (build-system minetest-mod-build-system)
+ (home-page "http://advtrains.de/")
+ (synopsis
+ "Adds good-looking, realistic trains with realistic rails")
+ (description
+ "\
+This mod features realistic trains and various equipment for railways, with a
+focus on automated train operation. This package contains no actual trains,
+please use minetest-basic-trains.
+
+Main features:
+
+@itemize
+@item
+almost-realistic tracks with actual curves and switches;
+@item
+railway signals, controllable by various means;
+@item
+ATC: simple, command-like automatic train control;
+@item
+LuaATC: Powerful Lua-scripted automatic train operation (requires some
+programming knowledge);
+@item
+an interlocking system, featuring track sections, routes and automatic
+stopping before signals.
+@end itemize")
+ (license
+ (list license:cc-by-sa3.0 license:agpl3+))
+ (properties
+ `((upstream-name . "orwell/advtrains")))))
--
2.33.0
From ba9014e45f9b237e23da58de075231bda87a0cab Mon Sep 17 00:00:00 2001
From: Vivien Kraus <vivien@planete-kraus.eu>
Date: Tue, 7 Sep 2021 12:28:35 +0200
Subject: [PATCH 2/4] gnu: minetest-mesecons: Fix indentation.

* gnu/packages/minetest.scm (minetest-mesecons): Fix indentation.
---
gnu/packages/minetest.scm | 50 +++++++++++++++++++--------------------
1 file changed, 25 insertions(+), 25 deletions(-)

Toggle diff (69 lines)
diff --git a/gnu/packages/minetest.scm b/gnu/packages/minetest.scm
index 6f4682ba47..9e48d5c69c 100644
--- a/gnu/packages/minetest.scm
+++ b/gnu/packages/minetest.scm
@@ -304,37 +304,37 @@ and a variety of other stuff.")
;; The release on ContentDB does not have its own version number.
(let ((commit "db5879706d04d3480bc4863ce0c03fa73e5f10c7")
(revision "0"))
- (package
- (name "minetest-mesecons")
- (version (git-version "1.2.1" revision commit))
- (source
- (origin
- (method git-fetch)
- (uri (git-reference
- (url "https://github.com/minetest-mods/mesecons")
- (commit commit)))
- (sha256
- (base32 "04m9s9l3frw1lgki41hgvjsw2zkrvfv0sy750b6j12arzb3lv645"))
- (file-name (git-file-name name version))))
- (build-system minetest-mod-build-system)
- (home-page "https://mesecons.net")
- (synopsis
- "Digital circuitry for Minetest, including wires, buttons and lights")
- (description
- "Mesecons is a mod for Minetest implementing various items related
+ (package
+ (name "minetest-mesecons")
+ (version (git-version "1.2.1" revision commit))
+ (source
+ (origin
+ (method git-fetch)
+ (uri (git-reference
+ (url "https://github.com/minetest-mods/mesecons")
+ (commit commit)))
+ (sha256
+ (base32 "04m9s9l3frw1lgki41hgvjsw2zkrvfv0sy750b6j12arzb3lv645"))
+ (file-name (git-file-name name version))))
+ (build-system minetest-mod-build-system)
+ (home-page "https://mesecons.net")
+ (synopsis
+ "Digital circuitry for Minetest, including wires, buttons and lights")
+ (description
+ "Mesecons is a mod for Minetest implementing various items related
to digital circuitry, such as wires, buttons, lights and programmable
controllers. Among other things, there are also pistons, solar panels,
pressure plates and note blocks.
Mesecons has a similar goal to Redstone in Minecraft, but works in its own way,
with different rules and mechanics.")
- ;; LGPL for code, CC-BY-SA for textures.
- ;; The README.md and COPYING.txt disagree about the "+" in license:lgpl3+.
- ;; For now, assume README.md is correct. Upstream has been asked to
- ;; correct the inconsistency:
- ;; <https://github.com/minetest-mods/mesecons/issues/575>.
- (license (list license:lgpl3+ license:cc-by-sa3.0))
- (properties `((upstream-name . "Jeija/mesecons"))))))
+ ;; LGPL for code, CC-BY-SA for textures.
+ ;; The README.md and COPYING.txt disagree about the "+" in license:lgpl3+.
+ ;; For now, assume README.md is correct. Upstream has been asked to
+ ;; correct the inconsistency:
+ ;; <https://github.com/minetest-mods/mesecons/issues/575>.
+ (license (list license:lgpl3+ license:cc-by-sa3.0))
+ (properties `((upstream-name . "Jeija/mesecons"))))))
(define-public minetest-mineclone
(package
--
2.33.0
From 77d3a0c63ae28a03e4408fa220a24605ad051e60 Mon Sep 17 00:00:00 2001
From: Vivien Kraus <vivien@planete-kraus.eu>
Date: Tue, 7 Sep 2021 12:27:59 +0200
Subject: [PATCH 1/4] gnu: minetest-data: Fix indentation.

* gnu/packages/minetest.scm (minetest-data): Fix indentation.
---
gnu/packages/minetest.scm | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)

Toggle diff (28 lines)
diff --git a/gnu/packages/minetest.scm b/gnu/packages/minetest.scm
index 5453f4d16a..6f4682ba47 100644
--- a/gnu/packages/minetest.scm
+++ b/gnu/packages/minetest.scm
@@ -154,8 +154,8 @@ in different ways.")
(source (origin
(method git-fetch)
(uri (git-reference
- (url "https://github.com/minetest/minetest_game")
- (commit version)))
+ (url "https://github.com/minetest/minetest_game")
+ (commit version)))
(file-name (git-file-name name version))
(sha256
(base32
@@ -172,8 +172,8 @@ in different ways.")
"/share/minetest/games/minetest_game")))
(mkdir-p install-dir)
(copy-recursively
- (assoc-ref %build-inputs "source")
- install-dir)
+ (assoc-ref %build-inputs "source")
+ install-dir)
#t))))
(synopsis "Main game data for the Minetest game engine")
(description
--
2.33.0
M
M
Maxime Devos wrote on 10 Sep 2021 18:26
021827be5ea5e4b1fe74555830d34c5b568a46c8.camel@telenet.be
Hi,

Vivien Kraus schreef op vr 10-09-2021 om 14:46 [+0200]:
Toggle quote (5 lines)
> > Maybe replace ‘realistic trains’ -> ‘realistic train system’,
> I’m not sure. Upstream really likes to use "train" as a general term,
> like in share/minetest/mods/advtrains/modpack.conf, the official
> description, the wiki, ... so I think it’s better to use that.
>
Ok.

Toggle quote (8 lines)
> > I think a linter will have something to say about long lines here
> > (try "./pre-inst-env guix lint minetest-advtrains minetest-basic-
> > trains").
> You’re right, I fixed the long lines and other warnings. However, I
> don’t know how to fix "no updater for minetest-xxx" for both minetest-
> advtrains and minetest-basic-trains. I can’t see a meaningful
> difference between my packages and the others.

There is no updater for mods from ContentDB at the moment, so this is
expected. There's a patch series for allowing updating git-fetch
origins: https://issues.guix.gnu.org/50072#0. Once (the final version of)
that patch series is merged, it should be feasible to add an updater
for mods from ContentDB.

Greetings,
Maxime.
-----BEGIN PGP SIGNATURE-----

iI0EABYKADUWIQTB8z7iDFKP233XAR9J4+4iGRcl7gUCYTuHOxccbWF4aW1lZGV2
b3NAdGVsZW5ldC5iZQAKCRBJ4+4iGRcl7jCPAQD9J42wJTixobjEoTYZshmEJSR8
7dN+50NLW2C37Wse7AEAxjEJfjgMxSEA8A1M+cqsTEvtKeR763BltFrpgFUBeAI=
=mZoa
-----END PGP SIGNATURE-----


X
X
Xinglu Chen wrote on 16 Sep 2021 10:03
87y27xq75c.fsf@yoctocell.xyz
On Fri, Sep 10 2021, Vivien Kraus via Guix-patches via wrote:

Toggle quote (75 lines)
> Hello Maxime,
>
> Le vendredi 10 septembre 2021 à 12:25 +0200, Maxime Devos a écrit :
>> minetest-advtrains and minetest-basic-trains are separate packages
>> that
>> can be added separately, so theoretically, they should be added in
>> separate
>> patches:
> It seems like the least surprising thing to do, so let’s go with the
> split patch.
>
>> Maybe replace ‘realistic trains’ -> ‘realistic train system’,
> I’m not sure. Upstream really likes to use "train" as a general term,
> like in share/minetest/mods/advtrains/modpack.conf, the official
> description, the wiki, ... so I think it’s better to use that.
>
>> and note that minetest-basic-trains contains the actual train models.
> This sentence was missing, and I added it.
>
>> I think a linter will have something to say about long lines here
>> (try "./pre-inst-env guix lint minetest-advtrains minetest-basic-
>> trains").
> You’re right, I fixed the long lines and other warnings. However, I
> don’t know how to fix "no updater for minetest-xxx" for both minetest-
> advtrains and minetest-basic-trains. I can’t see a meaningful
> difference between my packages and the others.
>
> Vivien
> From fbe39d93acb01fbef9f7f497e9bb5814fdf3f6b5 Mon Sep 17 00:00:00 2001
> From: Vivien Kraus <vivien@planete-kraus.eu>
> Date: Fri, 10 Sep 2021 14:21:47 +0200
> Subject: [PATCH 4/4] gnu: add minetest-basic-trains
>
> * gnu/packages/minetest.scm (minetest-basic-trains): New variable.
> ---
> gnu/packages/minetest.scm | 32 ++++++++++++++++++++++++++++++++
> 1 file changed, 32 insertions(+)
>
> diff --git a/gnu/packages/minetest.scm b/gnu/packages/minetest.scm
> index 0b0e7ace96..c218fb827f 100644
> --- a/gnu/packages/minetest.scm
> +++ b/gnu/packages/minetest.scm
> @@ -668,3 +668,35 @@ stopping before signals.
> (list license:cc-by-sa3.0 license:agpl3+))
> (properties
> `((upstream-name . "orwell/advtrains")))))
> +
> +(define-public minetest-basic-trains
> + (package
> + (name "minetest-basic-trains")
> + (version "1.0.1")
> + (source
> + (origin
> + (method git-fetch)
> + (uri (git-reference
> + (url "http://git.bananach.space/basic_trains.git/")
> + (commit
> + "d44c410f7c2a7202ee68b66fc50febae89e0c5dc")))
> + (sha256
> + (base32
> + "0vvzndj48kgdz2bfgivfm217sbmc2lmxpp2mispcy7byn4i26prx"))
> + (file-name (git-file-name name version))))
> + (build-system minetest-mod-build-system)
> + (propagated-inputs
> + `(("minetest-advtrains" ,minetest-advtrains)))
> + (home-page
> + "http://advtrains.de/wiki/doku.php?id=usage:trains:basic_trains")
> + (synopsis
> + "\
> +Collection of basic trains for the Advanced Trains mod")
> + (description
> + "\
> +This modpack contains the trains which were the \"default\" trains in
> +advtrains up to version 2.2.1.")

I would use ``default'', which is the correct Texinfo syntax,
instead of "default". See the Texinfo manual for more details:


Toggle quote (37 lines)
> + (license
> + (list license:cc-by-sa3.0 license:agpl3+))
> + (properties
> + `((upstream-name . "orwell/basic_trains")))))
> --
> 2.33.0
>
> From 9675601acba125905877f30b5bc47053db3b652d Mon Sep 17 00:00:00 2001
> From: Vivien Kraus <vivien@planete-kraus.eu>
> Date: Sun, 5 Sep 2021 15:21:35 +0200
> Subject: [PATCH 3/4] gnu: add minetest-advtrains
>
> * gnu/packages/minetest.scm (minetest-advtrains): New variable.
> ---
> gnu/packages/minetest.scm | 44 +++++++++++++++++++++++++++++++++++++++
> 1 file changed, 44 insertions(+)
>
> diff --git a/gnu/packages/minetest.scm b/gnu/packages/minetest.scm
> index 9e48d5c69c..0b0e7ace96 100644
> --- a/gnu/packages/minetest.scm
> +++ b/gnu/packages/minetest.scm
> @@ -624,3 +624,47 @@ track of important locations.")
> license:cc-by4.0 license:cc-by-sa3.0 license:public-domain
> license:cc0 license:fdl1.2+))
> (properties `((upstream-name . "RealBadAngel/unified_inventory")))))
> +
> +(define-public minetest-advtrains
> + (package
> + (name "minetest-advtrains")
> + (version "2.3.1")
> + (source
> + (origin
> + (method git-fetch)
> + (uri (git-reference
> + (url "https://git.bananach.space/advtrains.git")
> + (commit "release-2.3.1")))

This would preferably be (commit (string-append "release-" version)),
that means that you only have to update the ‘version’ field and the hash
when updating the package (assuming it doesn’t need additional
dependencies or change build system, etc.).

Otherwise, LGTM!
-----BEGIN PGP SIGNATURE-----

iQJJBAEBCAAzFiEEAVhh4yyK5+SEykIzrPUJmaL7XHkFAmFC+k8VHHB1YmxpY0B5
b2N0b2NlbGwueHl6AAoJEKz1CZmi+1x5lHEP/j+anO+qMszBNGigpi86eKwLB5x4
B4bC4mqSv0jx8BRgq4OZjGh4WRaTiGGj5KiDrl5I6j9os3i0RBlyMfCFQKKFRG9j
YVDBenA4B8pInV3Ab2qIdtzO4AvuisGnT1fRq9hls7+vUrDiDMajnXMNer+izvmJ
5cklsISu+z9ACzNOnlajDJ3CTL+VqdopqQPYLrTfTGn9KQBgv/JuuSCNsjjqL1y1
Ah0fbSMZEo1PC+X3Y9GSShjsuHw9j+O6eAjQoyuVbnXZP2WlPDoGyOpsmDgGqTZr
PIWMssSIsIYivYNtPezyxK8u9uOs3QY5z6LAiA1cJ5ZKY7qV6vsfJkypQNle6ZBB
ags+c6C0L9bI5V4Y3upn78mtsPBDX2mkAh11A23q1PVae9vMbCHq3b+RhkZe78rK
FWdvnC4TfTTSEFYKghaJlH2VeMB84DOP6eNw5AQ4XIOrGb5c3CLfFeOBkhlu32HP
rQb1qgXoH/F7greZ8PZLXJSlYSau8LOeENtnN3zc+cAVejT1frW3b51yluVaPKmI
a+KMBv9TQGC+4K6KKupPr7WGkfv34ut7GPTK90MbXLtCCHCkllo7Ob6DNgxEY5/H
qbMfrzxXaLd1m0O4IdYMHFertJcCKwPPbWPC9xVsGqNK4wKkGPGwKOqHTkJeuakK
2t7+LUQkuOVNfWmf
=n9w7
-----END PGP SIGNATURE-----

V
V
Vivien Kraus wrote on 16 Sep 2021 10:36
6f08e05d775fc7b1e770d1944a2e87130db193d0.camel@planete-kraus.eu
Hello,

Le jeudi 16 septembre 2021 à 10:03 +0200, Xinglu Chen a écrit :
Toggle quote (8 lines)
> > +This modpack contains the trains which were the \"default\" trains
> in
> > +advtrains up to version 2.2.1.")
>
> I would use ``default'', which is the correct Texinfo syntax,
> instead of "default". See the Texinfo manual for more details:
> https://www.gnu.org/software/texinfo/manual/texinfo/html_node/Inserting-Quotation-Marks.html
>
You are correct, I changed it. Guix lint does not like it that much,
because it thinks it is a lazy shortcut for @code{}, but I guess guix
lint can be wrong sometimes.

Toggle quote (17 lines)
> > +(define-public minetest-advtrains
> > + (package
> > + (name "minetest-advtrains")
> > + (version "2.3.1")
> > + (source
> > + (origin
> > + (method git-fetch)
> > + (uri (git-reference
> > + (url "https://git.bananach.space/advtrains.git")
> > + (commit "release-2.3.1")))
>
> This would preferably be (commit (string-append "release-" version)),
> that means that you only have to update the ‘version’ field and the
> hash
> when updating the package (assuming it doesn’t need additional
> dependencies or change build system, etc.).

This is a temporary fix because the contentdb importer (and future
updater) should get the commit ID from the minetest contentdb. For
minetest-advtrains, it is broken right now, so I had to guess which
commit it is, but it may be fixed soon. I added your change because it
makes more sense, but I don’t know if it will be relevant for long.

Vivien
From 4a50fc1af3698ae13e855fb859eb98455a6d29a9 Mon Sep 17 00:00:00 2001
From: Vivien Kraus <vivien@planete-kraus.eu>
Date: Fri, 10 Sep 2021 14:21:47 +0200
Subject: [PATCH 4/4] gnu: add minetest-basic-trains

* gnu/packages/minetest.scm (minetest-basic-trains): New variable.
---
gnu/packages/minetest.scm | 32 ++++++++++++++++++++++++++++++++
1 file changed, 32 insertions(+)

Toggle diff (42 lines)
diff --git a/gnu/packages/minetest.scm b/gnu/packages/minetest.scm
index 1e2ee54800..7ddff52925 100644
--- a/gnu/packages/minetest.scm
+++ b/gnu/packages/minetest.scm
@@ -668,3 +668,35 @@ stopping before signals.
(list license:cc-by-sa3.0 license:agpl3+))
(properties
`((upstream-name . "orwell/advtrains")))))
+
+(define-public minetest-basic-trains
+ (package
+ (name "minetest-basic-trains")
+ (version "1.0.1")
+ (source
+ (origin
+ (method git-fetch)
+ (uri (git-reference
+ (url "http://git.bananach.space/basic_trains.git/")
+ (commit
+ "d44c410f7c2a7202ee68b66fc50febae89e0c5dc")))
+ (sha256
+ (base32
+ "0vvzndj48kgdz2bfgivfm217sbmc2lmxpp2mispcy7byn4i26prx"))
+ (file-name (git-file-name name version))))
+ (build-system minetest-mod-build-system)
+ (propagated-inputs
+ `(("minetest-advtrains" ,minetest-advtrains)))
+ (home-page
+ "http://advtrains.de/wiki/doku.php?id=usage:trains:basic_trains")
+ (synopsis
+ "\
+Collection of basic trains for the Advanced Trains mod")
+ (description
+ "\
+This modpack contains the trains which were the ``default'' trains in
+advtrains up to version 2.2.1.")
+ (license
+ (list license:cc-by-sa3.0 license:agpl3+))
+ (properties
+ `((upstream-name . "orwell/basic_trains")))))
--
2.33.0
From 556c240d5cdfc1f009c6f2e45c8a435ad1123ca8 Mon Sep 17 00:00:00 2001
From: Vivien Kraus <vivien@planete-kraus.eu>
Date: Sun, 5 Sep 2021 15:21:35 +0200
Subject: [PATCH 3/4] gnu: add minetest-advtrains

* gnu/packages/minetest.scm (minetest-advtrains): New variable.
---
gnu/packages/minetest.scm | 44 +++++++++++++++++++++++++++++++++++++++
1 file changed, 44 insertions(+)

Toggle diff (54 lines)
diff --git a/gnu/packages/minetest.scm b/gnu/packages/minetest.scm
index 9e48d5c69c..1e2ee54800 100644
--- a/gnu/packages/minetest.scm
+++ b/gnu/packages/minetest.scm
@@ -624,3 +624,47 @@ track of important locations.")
license:cc-by4.0 license:cc-by-sa3.0 license:public-domain
license:cc0 license:fdl1.2+))
(properties `((upstream-name . "RealBadAngel/unified_inventory")))))
+
+(define-public minetest-advtrains
+ (package
+ (name "minetest-advtrains")
+ (version "2.3.1")
+ (source
+ (origin
+ (method git-fetch)
+ (uri (git-reference
+ (url "https://git.bananach.space/advtrains.git")
+ (commit (string-append "release-" "2.3.1"))))
+ (sha256
+ (base32 "1ijqlchh269jpvmgmdmdvy3nsnk0bszkvvcqk6vaysvxam695ggw"))
+ (file-name (git-file-name name version))))
+ (build-system minetest-mod-build-system)
+ (home-page "http://advtrains.de/")
+ (synopsis
+ "Adds good-looking, realistic trains with realistic rails")
+ (description
+ "\
+This mod features realistic trains and various equipment for railways, with a
+focus on automated train operation. This package contains no actual trains,
+please use minetest-basic-trains.
+
+Main features:
+
+@itemize
+@item
+almost-realistic tracks with actual curves and switches;
+@item
+railway signals, controllable by various means;
+@item
+ATC: simple, command-like automatic train control;
+@item
+LuaATC: Powerful Lua-scripted automatic train operation (requires some
+programming knowledge);
+@item
+an interlocking system, featuring track sections, routes and automatic
+stopping before signals.
+@end itemize")
+ (license
+ (list license:cc-by-sa3.0 license:agpl3+))
+ (properties
+ `((upstream-name . "orwell/advtrains")))))
--
2.33.0
From c7ef72b8648d5800935f9b353d3c03807de08d11 Mon Sep 17 00:00:00 2001
From: Vivien Kraus <vivien@planete-kraus.eu>
Date: Tue, 7 Sep 2021 12:28:35 +0200
Subject: [PATCH 2/4] gnu: minetest-mesecons: Fix indentation.

* gnu/packages/minetest.scm (minetest-mesecons): Fix indentation.
---
gnu/packages/minetest.scm | 50 +++++++++++++++++++--------------------
1 file changed, 25 insertions(+), 25 deletions(-)

Toggle diff (69 lines)
diff --git a/gnu/packages/minetest.scm b/gnu/packages/minetest.scm
index 6f4682ba47..9e48d5c69c 100644
--- a/gnu/packages/minetest.scm
+++ b/gnu/packages/minetest.scm
@@ -304,37 +304,37 @@ and a variety of other stuff.")
;; The release on ContentDB does not have its own version number.
(let ((commit "db5879706d04d3480bc4863ce0c03fa73e5f10c7")
(revision "0"))
- (package
- (name "minetest-mesecons")
- (version (git-version "1.2.1" revision commit))
- (source
- (origin
- (method git-fetch)
- (uri (git-reference
- (url "https://github.com/minetest-mods/mesecons")
- (commit commit)))
- (sha256
- (base32 "04m9s9l3frw1lgki41hgvjsw2zkrvfv0sy750b6j12arzb3lv645"))
- (file-name (git-file-name name version))))
- (build-system minetest-mod-build-system)
- (home-page "https://mesecons.net")
- (synopsis
- "Digital circuitry for Minetest, including wires, buttons and lights")
- (description
- "Mesecons is a mod for Minetest implementing various items related
+ (package
+ (name "minetest-mesecons")
+ (version (git-version "1.2.1" revision commit))
+ (source
+ (origin
+ (method git-fetch)
+ (uri (git-reference
+ (url "https://github.com/minetest-mods/mesecons")
+ (commit commit)))
+ (sha256
+ (base32 "04m9s9l3frw1lgki41hgvjsw2zkrvfv0sy750b6j12arzb3lv645"))
+ (file-name (git-file-name name version))))
+ (build-system minetest-mod-build-system)
+ (home-page "https://mesecons.net")
+ (synopsis
+ "Digital circuitry for Minetest, including wires, buttons and lights")
+ (description
+ "Mesecons is a mod for Minetest implementing various items related
to digital circuitry, such as wires, buttons, lights and programmable
controllers. Among other things, there are also pistons, solar panels,
pressure plates and note blocks.
Mesecons has a similar goal to Redstone in Minecraft, but works in its own way,
with different rules and mechanics.")
- ;; LGPL for code, CC-BY-SA for textures.
- ;; The README.md and COPYING.txt disagree about the "+" in license:lgpl3+.
- ;; For now, assume README.md is correct. Upstream has been asked to
- ;; correct the inconsistency:
- ;; <https://github.com/minetest-mods/mesecons/issues/575>.
- (license (list license:lgpl3+ license:cc-by-sa3.0))
- (properties `((upstream-name . "Jeija/mesecons"))))))
+ ;; LGPL for code, CC-BY-SA for textures.
+ ;; The README.md and COPYING.txt disagree about the "+" in license:lgpl3+.
+ ;; For now, assume README.md is correct. Upstream has been asked to
+ ;; correct the inconsistency:
+ ;; <https://github.com/minetest-mods/mesecons/issues/575>.
+ (license (list license:lgpl3+ license:cc-by-sa3.0))
+ (properties `((upstream-name . "Jeija/mesecons"))))))
(define-public minetest-mineclone
(package
--
2.33.0
From 04d842a8355b9b9dedcfc51b78c3baccdcdb1e64 Mon Sep 17 00:00:00 2001
From: Vivien Kraus <vivien@planete-kraus.eu>
Date: Tue, 7 Sep 2021 12:27:59 +0200
Subject: [PATCH 1/4] gnu: minetest-data: Fix indentation.

* gnu/packages/minetest.scm (minetest-data): Fix indentation.
---
gnu/packages/minetest.scm | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)

Toggle diff (28 lines)
diff --git a/gnu/packages/minetest.scm b/gnu/packages/minetest.scm
index 5453f4d16a..6f4682ba47 100644
--- a/gnu/packages/minetest.scm
+++ b/gnu/packages/minetest.scm
@@ -154,8 +154,8 @@ in different ways.")
(source (origin
(method git-fetch)
(uri (git-reference
- (url "https://github.com/minetest/minetest_game")
- (commit version)))
+ (url "https://github.com/minetest/minetest_game")
+ (commit version)))
(file-name (git-file-name name version))
(sha256
(base32
@@ -172,8 +172,8 @@ in different ways.")
"/share/minetest/games/minetest_game")))
(mkdir-p install-dir)
(copy-recursively
- (assoc-ref %build-inputs "source")
- install-dir)
+ (assoc-ref %build-inputs "source")
+ install-dir)
#t))))
(synopsis "Main game data for the Minetest game engine")
(description
--
2.33.0
V
V
Vivien Kraus wrote on 16 Sep 2021 10:39
b149d7970b9ef9a00d9359d7aad785209e5edc3c.camel@planete-kraus.eu
Sorry, the last patches contained an error: (string-append "release-"
"2.3.1") insntead of (string-append "release-" version), please
consider these ones instead:
From 5c00d1eca59c36d613dcccd13d90acf09f86071f Mon Sep 17 00:00:00 2001
From: Vivien Kraus <vivien@planete-kraus.eu>
Date: Fri, 10 Sep 2021 14:21:47 +0200
Subject: [PATCH 4/4] gnu: add minetest-basic-trains

* gnu/packages/minetest.scm (minetest-basic-trains): New variable.
---
gnu/packages/minetest.scm | 32 ++++++++++++++++++++++++++++++++
1 file changed, 32 insertions(+)

Toggle diff (42 lines)
diff --git a/gnu/packages/minetest.scm b/gnu/packages/minetest.scm
index 43dbe161da..8a4d81de03 100644
--- a/gnu/packages/minetest.scm
+++ b/gnu/packages/minetest.scm
@@ -668,3 +668,35 @@ stopping before signals.
(list license:cc-by-sa3.0 license:agpl3+))
(properties
`((upstream-name . "orwell/advtrains")))))
+
+(define-public minetest-basic-trains
+ (package
+ (name "minetest-basic-trains")
+ (version "1.0.1")
+ (source
+ (origin
+ (method git-fetch)
+ (uri (git-reference
+ (url "http://git.bananach.space/basic_trains.git/")
+ (commit
+ "d44c410f7c2a7202ee68b66fc50febae89e0c5dc")))
+ (sha256
+ (base32
+ "0vvzndj48kgdz2bfgivfm217sbmc2lmxpp2mispcy7byn4i26prx"))
+ (file-name (git-file-name name version))))
+ (build-system minetest-mod-build-system)
+ (propagated-inputs
+ `(("minetest-advtrains" ,minetest-advtrains)))
+ (home-page
+ "http://advtrains.de/wiki/doku.php?id=usage:trains:basic_trains")
+ (synopsis
+ "\
+Collection of basic trains for the Advanced Trains mod")
+ (description
+ "\
+This modpack contains the trains which were the ``default'' trains in
+advtrains up to version 2.2.1.")
+ (license
+ (list license:cc-by-sa3.0 license:agpl3+))
+ (properties
+ `((upstream-name . "orwell/basic_trains")))))
--
2.33.0
From 7df9af22de4fdacf1e81653754a19f5484f68f2c Mon Sep 17 00:00:00 2001
From: Vivien Kraus <vivien@planete-kraus.eu>
Date: Sun, 5 Sep 2021 15:21:35 +0200
Subject: [PATCH 3/4] gnu: add minetest-advtrains

* gnu/packages/minetest.scm (minetest-advtrains): New variable.
---
gnu/packages/minetest.scm | 44 +++++++++++++++++++++++++++++++++++++++
1 file changed, 44 insertions(+)

Toggle diff (54 lines)
diff --git a/gnu/packages/minetest.scm b/gnu/packages/minetest.scm
index 9e48d5c69c..43dbe161da 100644
--- a/gnu/packages/minetest.scm
+++ b/gnu/packages/minetest.scm
@@ -624,3 +624,47 @@ track of important locations.")
license:cc-by4.0 license:cc-by-sa3.0 license:public-domain
license:cc0 license:fdl1.2+))
(properties `((upstream-name . "RealBadAngel/unified_inventory")))))
+
+(define-public minetest-advtrains
+ (package
+ (name "minetest-advtrains")
+ (version "2.3.1")
+ (source
+ (origin
+ (method git-fetch)
+ (uri (git-reference
+ (url "https://git.bananach.space/advtrains.git")
+ (commit (string-append "release-" version))))
+ (sha256
+ (base32 "1ijqlchh269jpvmgmdmdvy3nsnk0bszkvvcqk6vaysvxam695ggw"))
+ (file-name (git-file-name name version))))
+ (build-system minetest-mod-build-system)
+ (home-page "http://advtrains.de/")
+ (synopsis
+ "Adds good-looking, realistic trains with realistic rails")
+ (description
+ "\
+This mod features realistic trains and various equipment for railways, with a
+focus on automated train operation. This package contains no actual trains,
+please use minetest-basic-trains.
+
+Main features:
+
+@itemize
+@item
+almost-realistic tracks with actual curves and switches;
+@item
+railway signals, controllable by various means;
+@item
+ATC: simple, command-like automatic train control;
+@item
+LuaATC: Powerful Lua-scripted automatic train operation (requires some
+programming knowledge);
+@item
+an interlocking system, featuring track sections, routes and automatic
+stopping before signals.
+@end itemize")
+ (license
+ (list license:cc-by-sa3.0 license:agpl3+))
+ (properties
+ `((upstream-name . "orwell/advtrains")))))
--
2.33.0
From c7ef72b8648d5800935f9b353d3c03807de08d11 Mon Sep 17 00:00:00 2001
From: Vivien Kraus <vivien@planete-kraus.eu>
Date: Tue, 7 Sep 2021 12:28:35 +0200
Subject: [PATCH 2/4] gnu: minetest-mesecons: Fix indentation.

* gnu/packages/minetest.scm (minetest-mesecons): Fix indentation.
---
gnu/packages/minetest.scm | 50 +++++++++++++++++++--------------------
1 file changed, 25 insertions(+), 25 deletions(-)

Toggle diff (69 lines)
diff --git a/gnu/packages/minetest.scm b/gnu/packages/minetest.scm
index 6f4682ba47..9e48d5c69c 100644
--- a/gnu/packages/minetest.scm
+++ b/gnu/packages/minetest.scm
@@ -304,37 +304,37 @@ and a variety of other stuff.")
;; The release on ContentDB does not have its own version number.
(let ((commit "db5879706d04d3480bc4863ce0c03fa73e5f10c7")
(revision "0"))
- (package
- (name "minetest-mesecons")
- (version (git-version "1.2.1" revision commit))
- (source
- (origin
- (method git-fetch)
- (uri (git-reference
- (url "https://github.com/minetest-mods/mesecons")
- (commit commit)))
- (sha256
- (base32 "04m9s9l3frw1lgki41hgvjsw2zkrvfv0sy750b6j12arzb3lv645"))
- (file-name (git-file-name name version))))
- (build-system minetest-mod-build-system)
- (home-page "https://mesecons.net")
- (synopsis
- "Digital circuitry for Minetest, including wires, buttons and lights")
- (description
- "Mesecons is a mod for Minetest implementing various items related
+ (package
+ (name "minetest-mesecons")
+ (version (git-version "1.2.1" revision commit))
+ (source
+ (origin
+ (method git-fetch)
+ (uri (git-reference
+ (url "https://github.com/minetest-mods/mesecons")
+ (commit commit)))
+ (sha256
+ (base32 "04m9s9l3frw1lgki41hgvjsw2zkrvfv0sy750b6j12arzb3lv645"))
+ (file-name (git-file-name name version))))
+ (build-system minetest-mod-build-system)
+ (home-page "https://mesecons.net")
+ (synopsis
+ "Digital circuitry for Minetest, including wires, buttons and lights")
+ (description
+ "Mesecons is a mod for Minetest implementing various items related
to digital circuitry, such as wires, buttons, lights and programmable
controllers. Among other things, there are also pistons, solar panels,
pressure plates and note blocks.
Mesecons has a similar goal to Redstone in Minecraft, but works in its own way,
with different rules and mechanics.")
- ;; LGPL for code, CC-BY-SA for textures.
- ;; The README.md and COPYING.txt disagree about the "+" in license:lgpl3+.
- ;; For now, assume README.md is correct. Upstream has been asked to
- ;; correct the inconsistency:
- ;; <https://github.com/minetest-mods/mesecons/issues/575>.
- (license (list license:lgpl3+ license:cc-by-sa3.0))
- (properties `((upstream-name . "Jeija/mesecons"))))))
+ ;; LGPL for code, CC-BY-SA for textures.
+ ;; The README.md and COPYING.txt disagree about the "+" in license:lgpl3+.
+ ;; For now, assume README.md is correct. Upstream has been asked to
+ ;; correct the inconsistency:
+ ;; <https://github.com/minetest-mods/mesecons/issues/575>.
+ (license (list license:lgpl3+ license:cc-by-sa3.0))
+ (properties `((upstream-name . "Jeija/mesecons"))))))
(define-public minetest-mineclone
(package
--
2.33.0
From 04d842a8355b9b9dedcfc51b78c3baccdcdb1e64 Mon Sep 17 00:00:00 2001
From: Vivien Kraus <vivien@planete-kraus.eu>
Date: Tue, 7 Sep 2021 12:27:59 +0200
Subject: [PATCH 1/4] gnu: minetest-data: Fix indentation.

* gnu/packages/minetest.scm (minetest-data): Fix indentation.
---
gnu/packages/minetest.scm | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)

Toggle diff (28 lines)
diff --git a/gnu/packages/minetest.scm b/gnu/packages/minetest.scm
index 5453f4d16a..6f4682ba47 100644
--- a/gnu/packages/minetest.scm
+++ b/gnu/packages/minetest.scm
@@ -154,8 +154,8 @@ in different ways.")
(source (origin
(method git-fetch)
(uri (git-reference
- (url "https://github.com/minetest/minetest_game")
- (commit version)))
+ (url "https://github.com/minetest/minetest_game")
+ (commit version)))
(file-name (git-file-name name version))
(sha256
(base32
@@ -172,8 +172,8 @@ in different ways.")
"/share/minetest/games/minetest_game")))
(mkdir-p install-dir)
(copy-recursively
- (assoc-ref %build-inputs "source")
- install-dir)
+ (assoc-ref %build-inputs "source")
+ install-dir)
#t))))
(synopsis "Main game data for the Minetest game engine")
(description
--
2.33.0
M
M
Maxime Devos wrote on 16 Sep 2021 12:55
f02033dbaf83b0d79d124d6d45d080cab3c8a256.camel@telenet.be
Vivien Kraus schreef op do 16-09-2021 om 10:39 [+0200]:
Toggle quote (4 lines)
> Sorry, the last patches contained an error: (string-append "release-"
> "2.3.1") insntead of (string-append "release-" version), please
> consider these ones instead:

These patches look good to me.

Greetings,
Maxime.
-----BEGIN PGP SIGNATURE-----

iI0EABYKADUWIQTB8z7iDFKP233XAR9J4+4iGRcl7gUCYUMiqRccbWF4aW1lZGV2
b3NAdGVsZW5ldC5iZQAKCRBJ4+4iGRcl7l79AQDJwdH0wGxbM/GQbceq9TbXdG8a
qcrYry0f3u3h55+/ggEAhUwrrMHueA51XCw8ywNZ1Dr66z/ZYj6W2aG+spkBDAQ=
=q5JA
-----END PGP SIGNATURE-----


L
L
Ludovic Courtès wrote on 16 Sep 2021 22:44
Re: bug#50425: [PATCH] gnu: Add minetest-advtrains.
(name . Maxime Devos)(address . maximedevos@telenet.be)
874kakz1wc.fsf_-_@gnu.org
Hi,

Maxime Devos <maximedevos@telenet.be> skribis:

Toggle quote (7 lines)
> Vivien Kraus schreef op do 16-09-2021 om 10:39 [+0200]:
>> Sorry, the last patches contained an error: (string-append "release-"
>> "2.3.1") insntead of (string-append "release-" version), please
>> consider these ones instead:
>
> These patches look good to me.

Awesome. Applied with minor formatting tweaks.

Thanks Vivien & Maxime!

Ludo’.
Closed
?