Bug#828018: minetest-server is not shutdown properly

[dr4Ke]

Package: minetest-server
Version: 0.4.14+repack-3
Severity: normal
Tags: patch

Dear Maintainer,

*** Reporter, please consider answering these questions, where appropriate ***

   * What led up to the situation?
Restarting the minetest-server service

   * What exactly did you do (or not do) that was effective (or
     ineffective)?
I just restarted the minetest-server service

   * What was the outcome of this action?
Some mod data was not present at the next boot, because of two combined issues:
- the mod is saving its data with register_on_shutdown which is not safe from
  a server failure
- the systemd service kills the minetest-server process instead of shuting it
  down properly.

   * What outcome did you expect instead?
We can shutdown the minetest-server properly with a kill signal: SIGINT.


PATCH:
--- /lib/systemd/system/minetest-server.service	2016-05-18 09:48:48.000000000 +0200
+++ minetest-server.service	2016-06-23 21:40:17.804380123 +0200
@@ -9,6 +9,7 @@
 User=Debian-minetest
 Group=games
 ExecStart=/usr/lib/minetest/minetestserver --config /etc/minetest/minetest.conf --logfile /var/log/minetest/minetest.log
+KillSignal=SIGINT
 
 [Install]
 WantedBy=multi-user.target
--- /lib/systemd/system/minetest-server at .service	2016-05-18 09:48:48.000000000 +0200
+++ minetest-server at .service	2016-06-23 21:40:24.632157955 +0200
@@ -9,6 +9,7 @@
 User=Debian-minetest
 Group=games
 ExecStart=/usr/lib/minetest/minetestserver --config /etc/minetest/%i.conf --logfile /var/log/minetest/%i.log
+KillSignal=SIGINT
 
 [Install]
 WantedBy=multi-user.target
END_OF_PATCH



-- System Information:
Debian Release: stretch/sid
  APT prefers testing
  APT policy: (500, 'testing')
Architecture: amd64 (x86_64)

Kernel: Linux 4.5.0-1-amd64 (SMP w/4 CPU cores)
Locale: LANG=fr_FR.UTF-8, LC_CTYPE=fr_FR.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/bash
Init: systemd (via /run/systemd/system)

Versions of packages minetest-server depends on:
ii  adduser              3.114
ii  init-system-helpers  1.29
ii  libc6                2.22-7
ii  libcurl3-gnutls      7.47.0-1
ii  libgcc1              1:5.3.1-14
ii  libjsoncpp1          1.7.2-1
ii  libleveldb1v5        1.18-5
ii  libluajit-5.1-2      2.0.4+dfsg-1
ii  libsqlite3-0         3.12.1-1
ii  libstdc++6           5.3.1-14
ii  minetest-data        0.4.14+repack-3
ii  zlib1g               1:1.2.8.dfsg-2+b1

minetest-server recommends no packages.

minetest-server suggests no packages.

-- Configuration Files:
/etc/minetest/minetest.conf changed [not included]

-- no debconf information