[Freeciv-Dev] [patch #1284] update reset command

2010-02-16 Thread pepeto

Update of patch #1284 (project freeciv):

  Status:  Ready For Test = Done   
 Open/Closed:Open = Closed 


___

Reply to this item at:

  http://gna.org/patch/?1284

___
  Message posté via/par Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #1284] update reset command

2010-02-13 Thread pepeto

Follow-up Comment #16, patch #1284 (project freeciv):

Updated patch to current trunk.

(file #8065)
___

Additional Item Attachment:

File name: trunk_update_reset_command.diff Size:17 KB


___

Reply to this item at:

  http://gna.org/patch/?1284

___
  Message posté via/par Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #1284] update reset command

2010-02-09 Thread Matthias Pfafferodt

Follow-up Comment #7, patch #1284 (project freeciv):

rebased and updated patch

(file #8014)
___

Additional Item Attachment:

File name: 20100209-trunk-update-reset-command.patch Size:19 KB


___

Reply to this item at:

  http://gna.org/patch/?1284

___
  Nachricht geschickt von/durch Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #1284] update reset command

2010-02-09 Thread pepeto

Update of patch #1284 (project freeciv):

 Assigned to:   cazfi = pepeto 

___

Follow-up Comment #8:

Maybe a _/reset map_ would be required to unload a scenario?  Or maybe it
should be possible to do it from the _/reset game_ command?

___

Reply to this item at:

  http://gna.org/patch/?1284

___
  Message posté via/par Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #1284] update reset command

2010-02-09 Thread Matthias Pfafferodt

Follow-up Comment #9, patch #1284 (project freeciv):

 Or maybe it should be possible to do it from the /reset game
 command?

reset game restores the settings to the values used at the start of the game

 Maybe a /reset map would be required to unload a scenario?

This would be an additional patch. Is map_free() enough for this?

___

Reply to this item at:

  http://gna.org/patch/?1284

___
  Nachricht geschickt von/durch Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #1284] update reset command

2010-02-09 Thread pepeto

Follow-up Comment #10, patch #1284 (project freeciv):

 Is map_free() enough for this?

I have no idea about it.  Maybe additional map_init() would be needed. 
That's so sad that the game and map structures are absolutely not
object-deigned...

___

Reply to this item at:

  http://gna.org/patch/?1284

___
  Message posté via/par Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #1284] update reset command

2010-02-09 Thread pepeto

Update of patch #1284 (project freeciv):

  Status:None = Ready For Test 

___

Follow-up Comment #11:

 This would be an additional patch.

Then, in another item...


___

Reply to this item at:

  http://gna.org/patch/?1284

___
  Message posté via/par Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #1284] update reset command

2010-02-09 Thread Matthias Pfafferodt

Follow-up Comment #12, patch #1284 (project freeciv):

an 'reset map' is really a thing for another patch. I tried it using

* map_free(); map_init(): = crash


Program terminated with signal 11, Segmentation fault.
#0  0x00555c32 in player_invention_set (pplayer=value optimized
out, tech=0, value=TECH_KNOWN)
at tech.c:148
148   enum tech_state old = research-inventions[tech].state;
(gdb) where
#0  0x00555c32 in player_invention_set (pplayer=value optimized
out, tech=0, value=TECH_KNOWN)
at tech.c:148
#1  0x0041ab4b in init_tech (plr=0x9563a8, update=true) at
techtools.c:586
#2  0x0040cfdc in srv_ready () at srv_main.c:2320
#3  0x0040d495 in srv_main () at srv_main.c:2466
#4  0x00404529 in main (argc=1, argv=0x7fff11493b28) at
civserver.c:372


* game_reset() = crash


Program terminated with signal 6, Aborted.
#0  0x7fca6f227645 in raise () from /lib64/libc.so.6
(gdb) where
#0  0x7fca6f227645 in raise () from /lib64/libc.so.6
#1  0x7fca6f228c33 in abort () from /lib64/libc.so.6
#2  0x7fca6f220329 in __assert_fail () from /lib64/libc.so.6
#3  0x00476ba6 in pick_a_nation (choices=0x0, ignore_conflicts=false,
only_available=true,
barb_type=NOT_A_BARBARIAN) at plrhand.c:1341
#4  0x0040ab95 in generate_players () at srv_main.c:1915
#5  0x0040d278 in srv_ready () at srv_main.c:2291
#6  0x0040d495 in srv_main () at srv_main.c:2466
#7  0x00404529 in main (argc=3, argv=0x7fff4e5c19b8) at
civserver.c:372


so there is more work needed; was something like this available in warserver?

___

Reply to this item at:

  http://gna.org/patch/?1284

___
  Nachricht geschickt von/durch Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #1284] update reset command

2010-02-09 Thread pepeto

Follow-up Comment #13, patch #1284 (project freeciv):

 so there is more work needed; was something like this available
 in warserver?

You could have a look to the /unloadmap command.  But I am not sure the code
around is exactly the same.


___

Reply to this item at:

  http://gna.org/patch/?1284

___
  Message posté via/par Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #1284] update reset command

2010-02-09 Thread Matthias Pfafferodt

Follow-up Comment #14, patch #1284 (project freeciv):

 You could have a look to the /unloadmap command. But I am not
 sure the code around is exactly the same. 

I don't think I will have this much time in the near future so I will stay
with the open patches I have at the moment ...

___

Reply to this item at:

  http://gna.org/patch/?1284

___
  Nachricht geschickt von/durch Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #1284] update reset command

2010-02-09 Thread pepeto

Follow-up Comment #15, patch #1284 (project freeciv):

And there is already an item for this: bug #14311

___

Reply to this item at:

  http://gna.org/patch/?1284

___
  Message posté via/par Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #1284] update reset command

2010-02-05 Thread Matthias Pfafferodt

Follow-up Comment #6, patch #1284 (project freeciv):

 Shouldn't reset at a5  b6 give identical results? At least it
 would make debugging (bug reproducibility) easier.

The attached patch saves the settings at game start within the struct setting
and also saves/restores these values to/from a savefile.

(file #7932)
___

Additional Item Attachment:

File name: 20100205-trunk-update-reset-command.patch Size:20 KB


___

Reply to this item at:

  http://gna.org/patch/?1284

___
  Nachricht geschickt von/durch Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #1284] update reset command

2009-12-20 Thread Daniel Markstedt

Update of patch #1284 (project freeciv):

 Planned Release: = 2.3.0  


___

Reply to this item at:

  http://gna.org/patch/?1284

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


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #1284] update reset command

2009-11-08 Thread Matthias Pfafferodt

Follow-up Comment #5, patch #1284 (project freeciv):

typo corrected patch attached

 Shouldn't reset at a5  b6 give identical results? At least it
 would make debugging (bug reproducibility) easier.

This would need an additional section in the savegame which holds all the
setting values from the game start. A warning could be added if the settings
from a savegame are restored?

(file #7257)
___

Additional Item Attachment:

File name: 20091108-settings-0008-update-reset-command.diff Size:14 KB


___

Reply to this item at:

  http://gna.org/patch/?1284

___
  Nachricht geschickt von/durch Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #1284] update reset command

2009-10-10 Thread Matthias Pfafferodt

Follow-up Comment #3, patch #1284 (project freeciv):

updated and rebased version

(file #6959)
___

Additional Item Attachment:

File name: settings20091010-0010-update-reset-command.diff Size:14 KB


___

Reply to this item at:

  http://gna.org/patch/?1284

___
  Nachricht geschickt von/durch Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #1284] update reset command

2009-09-13 Thread Matthias Pfafferodt

Follow-up Comment #2, patch #1284 (project freeciv):

rebased version of the patch

* does _not_ need S_S_SAVEGAME anymore

(file #6693)
___

Additional Item Attachment:

File name: 0013-update-reset-command.diff Size:13 KB


___

Reply to this item at:

  http://gna.org/patch/?1284

___
  Nachricht geschickt von/durch Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev