[Freeciv-Dev] [bug #15499] Wrong handle of the illness_trade stuff

2010-03-05 Thread Matthias Pfafferodt
Update of bug #15499 (project freeciv): Open/Closed:Open = Closed ___ Reply to this item at: http://gna.org/bugs/?15499 ___ Nachricht

[Freeciv-Dev] [bug #15499] Wrong handle of the illness_trade stuff

2010-02-27 Thread pepeto
URL: http://gna.org/bugs/?15499 Summary: Wrong handle of the illness_trade stuff Project: Freeciv Submitted by: pepeto Submitted on: samedi 27.02.2010 à 13:30 Category: client Severity: 3 - Normal

[Freeciv-Dev] [bug #15499] Wrong handle of the illness_trade stuff

2010-02-27 Thread Matthias Pfafferodt
Follow-up Comment #1, bug #15499 (project freeciv): This was cased by me applying the patch to fast. I will create a fix for it. The tests affect the client capability, which will always return TRUE because we have this capability. The server would affect the server capability

[Freeciv-Dev] [bug #15499] Wrong handle of the illness_trade stuff

2010-02-27 Thread pepeto
Follow-up Comment #2, bug #15499 (project freeciv): The tests affect the client capability, which will always return TRUE because we have this capability. The server would affect the server capability (client.conn.capability). Is it enough to use 'client.conn.capability' in the test to do

[Freeciv-Dev] [bug #15499] Wrong handle of the illness_trade stuff

2010-02-27 Thread Matthias Pfafferodt
Follow-up Comment #3, bug #15499 (project freeciv): still working on it. There are some strange things happening. I have compiled 2.2.0 and S2_2 with the patch and I'm trying to test the combination old server - new client. I added only an additional output line printing client.conn.capability

[Freeciv-Dev] [bug #15499] Wrong handle of the illness_trade stuff

2010-02-27 Thread pepeto
Follow-up Comment #4, bug #15499 (project freeciv): Somethere the server changed its capabilities ... ? I don't think so. I cannot reproduce, but I think you connected an old server just before and you are using non-initialized datas. The capability string of the server is set in

[Freeciv-Dev] [bug #15499] Wrong handle of the illness_trade stuff

2010-02-27 Thread pepeto
Follow-up Comment #5, bug #15499 (project freeciv): Ouch, I found what is wrong... handle_conn_info() overwrite with the server capability with our capability... ___ Reply to this item at: http://gna.org/bugs/?15499

[Freeciv-Dev] [bug #15499] Wrong handle of the illness_trade stuff

2010-02-27 Thread Matthias Pfafferodt
Follow-up Comment #6, bug #15499 (project freeciv): the server capabilities are changed within handle_conn_info() (see bug #15500): If the patch for bug #15500 is applied, the combination old server - new client works as expected: the illness value minus trade is displayed within the city

[Freeciv-Dev] [bug #15499] Wrong handle of the illness_trade stuff

2010-02-27 Thread Matthias Pfafferodt
Update of bug #15499 (project freeciv): Depends on: = bugs #15500 ___ Reply to this item at: http://gna.org/bugs/?15499 ___ Nachricht

[Freeciv-Dev] [bug #15499] Wrong handle of the illness_trade stuff

2010-02-27 Thread Matthias Pfafferodt
Follow-up Comment #7, bug #15499 (project freeciv): Ouch, I found what is wrong... handle_conn_info() overwrite with the server capability with our capability... I needed a little bit longer to validate this fact ;-) A hack to prevent this can be found in bug #15500