Discussion:
[Freeciv-Dev] [patch #6680] EFT_MAKE_CONTENT_MIL should not be evaluated client side
Marko Lindqvist
2015-12-14 09:15:05 UTC
Permalink
URL:
<http://gna.org/patch/?6680>

Summary: EFT_MAKE_CONTENT_MIL should not be evaluated client
side
Project: Freeciv
Submitted by: cazfi
Submitted on: Mon 14 Dec 2015 11:15:05 AM EET
Category: general
Priority: 5 - Normal
Status: None
Privacy: Public
Assigned to: None
Originator Email:
Open/Closed: Open
Discussion Lock: Any
Planned Release: 2.6.0, 3.0.0

_______________________________________________________

Details:

EFT_MAKE_CONTENT_MIL should be evaluated on omniscient server only, and
results just sent to the client as needed.




_______________________________________________________

Reply to this item at:

<http://gna.org/patch/?6680>

_______________________________________________
Message sent via/by Gna!
http://gna.org/
Marko Lindqvist
2016-06-14 20:11:45 UTC
Permalink
Update of patch #6680 (project freeciv):

Planned Release: 2.6.0, 3.0.0 => 3.0.0

_______________________________________________________

Follow-up Comment #1:

This might have consequences I don't want to play with at this point of S2_6
stabilization. I rather make this in TRUNK only, so that if it's a bad idea
(information in client side not in sync without big changes) we can revert it.
In S2_6 such network protocol changes we make now might end up unrevertable by
npf before problems are detected.

_______________________________________________________

Reply to this item at:

<http://gna.org/patch/?6680>

_______________________________________________
Message sent via/by Gna!
http://gna.org/
Marko Lindqvist
2017-05-03 14:41:08 UTC
Permalink
Update of patch #6680 (project freeciv):

Status: None => Duplicate
Assigned to: None => cazfi
Open/Closed: Open => Closed

_______________________________________________________

Follow-up Comment #2:

Handled at hostedredmine: https://www.hostedredmine.com/issues/657284

_______________________________________________________

Reply to this item at:

<http://gna.org/patch/?6680>

_______________________________________________
Message sent via/by Gna!
http://gna.org/

Loading...