Tíquete #43759

Metamessage resetted on ruleset load

: 2022-02-01 21:44 Última Atualização: 2022-03-25 00:45

Relator:
Dono:
Tipo:
Estado:
Fechado
Componente:
Marcos:
Prioridade:
5 - Medium
Gravidade:
5 - Medium
Resolução:
Fixed
Arquivo:
4

Details

As metamessage was turned to a server setting in freeciv-3.0, it now gets resetted to default (empty value) when a new ruleset is loaded. That's even when it has been set with the (supposedly backward compatible) dedicated 'metamessage' command, and not via 'set' command interface.

Ticket History (3/13 Histories)

2022-02-01 21:44 Updated by: cazfi
  • New Ticket "Metamessage reseted on ruleset load" created
2022-02-01 21:48 Updated by: cazfi
Comentário

Arguably resetting *any* server operation related settings just when the rules should change is a bug.

2022-02-01 21:48 Updated by: cazfi
  • Summary Updated
2022-02-01 22:51 Updated by: cazfi
Comentário

Attached is a S3_0 patch that I've applied to cazfi.net server as a quick fix. It might be a sensible starting point.

2022-02-03 23:51 Updated by: cazfi
2022-03-13 06:03 Updated by: cazfi
  • Dono Update from (Nenhum) to cazfi
  • Resolução Update from Nenhum to Accepted
Comentário

Attached is the earlier patch with minimal adjustments. I have a feeling that the change is a bit big for resolving a simple problem, but no better solution has been suggested and I consider this one of the most important issues to fix in 3.0.1.

2022-03-13 23:09 Updated by: cazfi
  • Resolução Update from Accepted to Nenhum
Comentário

Patch going to master should follow new CodingStyle compound literals recommendation.

2022-03-14 08:23 Updated by: cazfi
  • Resolução Update from Nenhum to Accepted
Comentário

- New master version with compound literals used

2022-03-25 00:45 Updated by: cazfi
  • Estado Update from Aberto to Fechado
  • Resolução Update from Accepted to Fixed

Editar

Please login to add comment to this ticket » Login