[wesnoth-commits] [wesnoth/wesnoth] 02a0c1: bi: Make game distribution channel part of the bui...

[Iris Morelle]

  Branch: refs/heads/1.14
  Home:   https://github.com/wesnoth/wesnoth
  Commit: 02a0c1a18fe295cec83d32068c2ac9156cd7d17f
      https://github.com/wesnoth/wesnoth/commit/02a0c1a18fe295cec83d32068c2ac9156cd7d17f
  Author: Iris Morelle <shadowm at wesnoth.org>
  Date:   2020-01-26 (Sun, 26 Jan 2020)

  Changed paths:
    M src/build_info.cpp
    M src/build_info.hpp
    M src/game_initialization/multiplayer.cpp

  Log Message:
  -----------
  bi: Make game distribution channel part of the build_info API

This moves the relevant code out of the MP game setup flow into a
dedicated function, so it can also be used in the --report output.

However, it also fixes two bugs we just found out with the previous
code:

 * The channel id was extracted using istream's formatted input
   operator, resulting in everything beyond the very first word being
   skipped.
 * The channel id was expected to be stored in ./data/dist rather than
   <game data dir>/data/dist. This causes issues whenever the process
   working dir is not the game data dir (most notably on X11 when
   launching with distribution-provided .desktop files) because the file
   won't be found.

So now the file is read by swallowing the first line and stripping
whitespace around, as well as using game_config::path (guaranteed to be
non-empty by the time the game makes actual use of this info).


  Commit: 60cca8e7868d6c746860dea8c70cc93879aa7822
      https://github.com/wesnoth/wesnoth/commit/60cca8e7868d6c746860dea8c70cc93879aa7822
  Author: Iris Morelle <shadowm at wesnoth.org>
  Date:   2020-01-26 (Sun, 26 Jan 2020)

  Changed paths:
    M changelog.md

  Log Message:
  -----------
  Update changelog

[ci skip]


Compare: https://github.com/wesnoth/wesnoth/compare/abadb451a221...60cca8e7868d