Bug#864572: openarena-server: consider to use StandardOutput=null in systemd service files

[Simon McVittie]

On Tue, 25 Jul 2017 at 23:50:24 +0200, Markus Koschany wrote:
> Am 24.07.2017 um 10:11 schrieb Simon McVittie:
> > We could maybe give the openarena-server wrapper script a
> > --logfile=LOGFILE option which would "exec >${logfile} 2>&1" before
> > running the actual server, and make the systemd units run with
> > "--logfile=/var/games/openarena-server/%i.oa/server.log" or
> > "--logfile=/var/log/openarena-server/%i/server.log" or something?
> > But then we'd need to implement at least simple log rotation (otherwise
> > whenever the server crashed, the log with crash details would
> > immediately be overwritten by the log for the restarted server).
> 
> Alright, if you don't like the /dev/null plan then this would be the
> next best option, although I'm pretty sure most output will be a
> duplicate of games.log.

Yeah - the Quake III Arena engine's approach to logging doesn't seem to
have been particularly well thought out, with a lot of messages only
going to stdout, but no built-in way to silence the ones that also go to
games.log.

> I'm just thinking that implementing the wrapper
> script option --logfile isn't really necessary because we can implement
> logging entirely in the service file
[...]
> ExecStart=/bin/sh -c '/usr/games/openarena-server +set com_homepath
> server.oa $DAEMON_OPTS +exec etc/openarena-server/server.cfg >
> /var/log/openarena-server/server.log 2>&1'

I think I'd prefer to put it in the wrapper script, since we already
have one.

    S