[Freeciv-Dev] [patch #3805] Allow paratroopers to land on transport

2013-04-01 Thread Marko Lindqvist
Follow-up Comment #3, patch #3805 (project freeciv): New boolean setting in game.ruleset [civstyle] for controlling if dropping to transport is allowed should be sufficient. ___ Reply to this item at: http://gna.org/patch/?3805

[Freeciv-Dev] [patch #3815] Provide information about unit class nativity in terrain help

2013-04-01 Thread Marko Lindqvist
Update of patch #3815 (project freeciv): Status:None = Ready For Test Assigned to:None = cazfi ___ Follow-up Comment #1: Usually jtn has had

[Freeciv-Dev] [patch #3815] Provide information about unit class nativity in terrain help

2013-04-01 Thread Emmet Hikory
Follow-up Comment #2, patch #3815 (project freeciv): I didn't mean to rush processing of this patch with the comment in patch #3816 , rather I just wanted to provide a hint for testing, as when I checked the submitted patch against revision 22638 it didn't apply cleanly. Anyway, I believe that

[Freeciv-Dev] [bug #20695] Disaster nreq processing causes disasters mostly not to happen

2013-04-01 Thread Marko Lindqvist
Update of bug #20695 (project freeciv): Status:None = Ready For Test Assigned to:None = cazfi ___ Follow-up Comment #1: Going with nreqs

[Freeciv-Dev] [bug #20695] Disaster nreq processing causes disasters mostly not to happen

2013-04-01 Thread Emmet Hikory
Follow-up Comment #2, bug #20695 (project freeciv): Only for nreqs for effects, which I would expect to be removed in the spirit of having one correct function to do each thing that needs doing. The place it might be would be sanity_check_req_set, but that doesn't seem to be aware of the idea of

[Freeciv-Dev] [patch #3805] Allow paratroopers to land on transport

2013-04-01 Thread Emmet Hikory
Follow-up Comment #4, patch #3805 (project freeciv): This updated patch is controlled by game.info.paradrop_to_transport. This is disabled in most rulesets, but enabled in experimental for testing. If the commiter doesn't feel it appropriate to enable it in experimental, please change TRUE to

[Freeciv-Dev] [patch #3815] Provide information about unit class nativity in terrain help

2013-04-01 Thread Jacob Nevins
Follow-up Comment #3, patch #3815 (project freeciv): Seems like a fine change to me. ___ Reply to this item at: http://gna.org/patch/?3815 ___ Message sent via/by Gna! http://gna.org/

Re: [Freeciv-Dev] [patch #3815] Provide information about unit class nativity in terrain help

2013-04-01 Thread Jacob Nevins
Emmet Hikory writes: I was unable to build S2_4 from revision 22639 due to a LUA error (which I presume related to some local configuration on my machine), so this is entirely theoretical, rather than a tested patch. What's the nature of the failure? If we're releasing 2.4.0-beta2 soon, I'm

[Freeciv-Dev] [bug #20698] Version check incorrectly considers 2.4.0 older than 2.4.0-beta1

2013-04-01 Thread Jacob Nevins
URL: http://gna.org/bugs/?20698 Summary: Version check incorrectly considers 2.4.0 older than 2.4.0-beta1 Project: Freeciv Submitted by: jtn Submitted on: Mon Apr 1 12:58:16 2013 Category: general

[Freeciv-Dev] [bug #20680] Client crash on contacting metaserver, dependent on latest version advertised by metaserver

2013-04-01 Thread Jacob Nevins
Follow-up Comment #10, bug #20680 (project freeciv): Giving this ticket to jtn to decide whjat to in freeciv copy. I've decided to prevaricate :) -- bug #20698 raised, so we can get this crasher out of the way. (Which I'll do immediately, with a patch taking into account the upstream changes for

[Freeciv-Dev] [bug #20680] Client crash on contacting metaserver, dependent on latest version advertised by metaserver

2013-04-01 Thread Jacob Nevins
Update of bug #20680 (project freeciv): Status: Ready For Test = Fixed Open/Closed:Open = Closed ___ Reply to this item at:

Re: [Freeciv-Dev] [patch #3815] Provide information about unit class nativity in terrain help

2013-04-01 Thread Emmet Hikory
Mon, Apr 01, 2013, Jacob Nevins wrote: What's the nature of the failure? If we're releasing 2.4.0-beta2 soon, I'm interested in portability hazards (Guess: are you switching a single working directory between trunk and S2_4? This is currently fraught, requiring make maintainer-clean and

[Freeciv-Dev] Switching working directory between branches (was: [patch #3815] Provide information about unit) class nativity in terrain help

2013-04-01 Thread Jacob Nevins
Emmet Hikory writes: Jacob Nevins wrote: (Guess: are you switching a single working directory between trunk and S2_4? This is currently fraught, requiring make maintainer-clean and starting again. This bites me in my mad git-svn setup.) Ah, yes, precisely. Thanks for the explanation: I

[Freeciv-Dev] [bug #20699] Bugs in undisbandable unit drowning

2013-04-01 Thread Marko Lindqvist
URL: http://gna.org/bugs/?20699 Summary: Bugs in undisbandable unit drowning Project: Freeciv Submitted by: cazfi Submitted on: Mon 01 Apr 2013 04:20:51 PM EEST Category: general Severity: 3 - Normal

[Freeciv-Dev] [bug #20630] Changing tileset twice crashes client

2013-04-01 Thread Jacob Nevins
Follow-up Comment #2, bug #20630 (project freeciv): I vaguely wonder if the assertion failure in bug #20694 (after changing tileset) is somehow related. ___ Reply to this item at: http://gna.org/bugs/?20630

[Freeciv-Dev] [bug #20626] Client crashes with GLib-GObject-CRITICAL **: g_object_unref: assertion `G_IS_OBJECT (object)' failed

2013-04-01 Thread Jacob Nevins
Update of bug #20626 (project freeciv): Status:None = Confirmed Planned Release: = 2.3.5,2.4.0,2.5.0 ___ Follow-up Comment #1: I can confirm

[Freeciv-Dev] [bug #20595] Errow while setting temperature below 14 with temperate map

2013-04-01 Thread Jacob Nevins
Follow-up Comment #3, bug #20595 (project freeciv): The requested map is too cold to allocate starting postions. Try setting the temperature to a value greater than 14. Unfortunately, trying to anticipate all the ways in which generator settings could lead to failure to allocate starting

[Freeciv-Dev] [bug #20498] Pirate archers alone in the water

2013-04-01 Thread Marko Lindqvist
Follow-up Comment #14, bug #20498 (project freeciv): Maybe so, I had no reproducible case to test against. But now I had, and it got fixed almost accidentally by bug #20699 patch. When pirate ship is wiped (usually retired) and it has both undisbandable Barbarian Leader and men in it, and

[Freeciv-Dev] [bug #20561] Allow server messages to be translated to client's locale

2013-04-01 Thread Jacob Nevins
Update of bug #20561 (project freeciv): Release: 2.5.0 = Summary: Move translations from freeciv-server to the clients. = Allow server messages to be translated to client's locale

[Freeciv-Dev] [bug #20543] lua error: Execution time limit exceeded in script

2013-04-01 Thread Jacob Nevins
Update of bug #20543 (project freeciv): Planned Release: = 2.5.0 ___ Reply to this item at: http://gna.org/bugs/?20543 ___ Message sent

[Freeciv-Dev] [bug #20534] Worklist glitch: unit icons bleed past their boundaries

2013-04-01 Thread Jacob Nevins
Follow-up Comment #3, bug #20534 (project freeciv): My guess would be you're hitting following documented case: When the destination rectangle contains parts not in the source image, the data at the edges of the source image is replicated to infinity. (documented for instance here

[Freeciv-Dev] [patch #3805] Allow paratroopers to land on transport

2013-04-01 Thread Marko Lindqvist
Follow-up Comment #5, patch #3805 (project freeciv): I think enabling this in experimental ruleset makes sense. But that requires update also to doc/README.ruleset_experimental. ___ Reply to this item at: http://gna.org/patch/?3805

[Freeciv-Dev] [patch #3806] Update mkinstalldirs

2013-04-01 Thread Marko Lindqvist
Update of patch #3806 (project freeciv): Status: Ready For Test = Done Assigned to:None = cazfi Open/Closed:Open = Closed

[Freeciv-Dev] [patch #3821] Drop terrain options

2013-04-01 Thread Marko Lindqvist
Update of patch #3821 (project freeciv): Planned Release: = 2.5.0 ___ Reply to this item at: http://gna.org/patch/?3821 ___ Message sent

[Freeciv-Dev] [patch #3820] Avoid selecting RMM_RELAXED roads when they are less good

2013-04-01 Thread Marko Lindqvist
Update of patch #3820 (project freeciv): Planned Release: = 2.5.0 ___ Reply to this item at: http://gna.org/patch/?3820 ___ Message sent

[Freeciv-Dev] [patch #3817] Generalise MR_BAD_TYPE_FOR_CITY_TAKE_OVER_BY_SEA

2013-04-01 Thread Marko Lindqvist
Update of patch #3817 (project freeciv): Planned Release: = 2.5.0 ___ Follow-up Comment #1: Does some earlier check prevents sea units (that can_attack_from_non_native for shore

[Freeciv-Dev] [patch #3811] Remove S_RIVER

2013-04-01 Thread Marko Lindqvist
Update of patch #3811 (project freeciv): Status: Ready For Test = Done Assigned to:None = cazfi Open/Closed:Open = Closed

[Freeciv-Dev] [patch #3812] Leader to civil war player

2013-04-01 Thread Marko Lindqvist
Update of patch #3812 (project freeciv): Status: Ready For Test = Done Assigned to:None = cazfi Open/Closed:Open = Closed

[Freeciv-Dev] [bug #20600] dai_player_load: add 'const' to struct section_file

2013-04-01 Thread Marko Lindqvist
Update of bug #20600 (project freeciv): Status:None = Ready For Test Planned Release: = 2.5.0 ___ Follow-up Comment #1: This change makes

[Freeciv-Dev] [bug #20577] new parameter gameloss_style: GameLoss player's property is redistributed instead of disappearing

2013-04-01 Thread Marko Lindqvist
Follow-up Comment #22, bug #20577 (project freeciv): - Use give_midgame_initial_units() (from patch #3812) to give units for newly created player. This also corrects behavior so that Leader units are given instead of GameLoss ones (if this ever makes any difference) I now also tested this. It

[Freeciv-Dev] [patch #3726] Booleanize all save functions for savegame v2

2013-04-01 Thread Marko Lindqvist
Follow-up Comment #5, patch #3726 (project freeciv): Could you produce new version of this patch to go in before S2_5 branching in a month? ___ Reply to this item at: http://gna.org/patch/?3726

[Freeciv-Dev] [patch #3539] Scorelog: new tags and details

2013-04-01 Thread Marko Lindqvist
Update of patch #3539 (project freeciv): Status: Ready For Test = None ___ Follow-up Comment #6: I'd like to have this one finished to conclude scorelog changes for 2.5. Do you happen to

[Freeciv-Dev] [patch #3616] Give ransom from multiple barbarian leaders

2013-04-01 Thread Marko Lindqvist
Update of patch #3616 (project freeciv): Planned Release: 2.5.0 = 2.6.0 ___ Reply to this item at: http://gna.org/patch/?3616 ___ Message sent

[Freeciv-Dev] [bug #19474] max_players cleanup

2013-04-01 Thread Marko Lindqvist
Update of bug #19474 (project freeciv): Depends on: = bugs #20693 ___ Reply to this item at: http://gna.org/bugs/?19474 ___ Message sent

[Freeciv-Dev] [bug #19830] Gtk2 client may segfault at network game page

2013-04-01 Thread Marko Lindqvist
Update of bug #19830 (project freeciv): Status:None = Duplicate Assigned to:None = cazfi Open/Closed:Open = Closed

[Freeciv-Dev] [bug #17582] [metaticket] Headers depend on config.h

2013-04-01 Thread Marko Lindqvist
Update of bug #17582 (project freeciv): Planned Release: 2.5.0 = 2.6.0 ___ Reply to this item at: http://gna.org/bugs/?17582 ___ Message sent

[Freeciv-Dev] [bug #14585] Wish: client-side progress bar for end-turn activities

2013-04-01 Thread Marko Lindqvist
Update of bug #14585 (project freeciv): Status: In Progress = None Planned Release: 2.5.0 = 2.6.0 ___ Follow-up Comment #3: Probably the best

[Freeciv-Dev] [bug #16164] Aircraft commit suicide if 'autoattack' is set

2013-04-01 Thread Marko Lindqvist
Follow-up Comment #4, bug #16164 (project freeciv): I think units that have fuel should not autoattack at all except when defending refuel point where they would then remain. ___ Reply to this item at: http://gna.org/bugs/?16164

[Freeciv-Dev] [patch #3651] Setting TRUNK for 2.6 development

2013-04-01 Thread Marko Lindqvist
Follow-up Comment #1, patch #3651 (project freeciv): - Updated against current svn (file #17649) ___ Additional Item Attachment: File name: 2.6Trunk-2.patch Size:8 KB

[Freeciv-Dev] [bug #20652] Homeless ai caravan crash

2013-04-01 Thread Marko Lindqvist
Update of bug #20652 (project freeciv): Status: Ready For Test = Fixed Assigned to:None = cazfi Open/Closed:Open = Closed

[Freeciv-Dev] [patch #3824] Allow sprites for 2 upkeep

2013-04-01 Thread Jacob Nevins
URL: http://gna.org/patch/?3824 Summary: Allow sprites for 2 upkeep Project: Freeciv Submitted by: jtn Submitted on: Mon Apr 1 23:25:11 2013 Category: client Priority: 5 - Normal Status:

[Freeciv-Dev] [patch #3824] Allow sprites for 2 unit upkeep

2013-04-01 Thread Jacob Nevins
Update of patch #3824 (project freeciv): Summary: Allow sprites for 2 upkeep = Allow sprites for 2 unit upkeep ___ Reply to this item at: http://gna.org/patch/?3824 ___

[Freeciv-Dev] [patch #3824] Allow sprites for 2 unit upkeep

2013-04-01 Thread Jacob Nevins
Update of patch #3824 (project freeciv): Status: In Progress = Ready For Test ___ Additional Item Attachment: File name: trunk-unit-upkeep-sprites.patch Size:3 KB

[Freeciv-Dev] [bug #20695] Disaster nreq processing causes disasters mostly not to happen

2013-04-01 Thread Marko Lindqvist
Update of bug #20695 (project freeciv): Planned Release: = 2.5.0 ___ Reply to this item at: http://gna.org/bugs/?20695 ___ Message sent

[Freeciv-Dev] [bug #20691] road_superhighway_trade_bonus in terrain.ruleset doesn't work

2013-04-01 Thread Marko Lindqvist
Update of bug #20691 (project freeciv): Status:None = Ready For Test Assigned to:None = cazfi Planned Release: = 2.3.5, 2.4.0, 2.5.0

[Freeciv-Dev] [patch #3815] Provide information about unit class nativity in terrain help

2013-04-01 Thread Marko Lindqvist
Update of patch #3815 (project freeciv): Planned Release: = 2.4.0, 2.5.0 ___ Reply to this item at: http://gna.org/patch/?3815 ___ Message sent

[Freeciv-Dev] [patch #3819] Generalise helptext for AttFromNonNative and Marines

2013-04-01 Thread Marko Lindqvist
Update of patch #3819 (project freeciv): Status:None = Ready For Test Assigned to:None = cazfi Planned Release: = 2.5.0

[Freeciv-Dev] [patch #3816] Cleanup nativity handling to ignore specials

2013-04-01 Thread Marko Lindqvist
Update of patch #3816 (project freeciv): Status:None = Ready For Test Assigned to:None = cazfi ___ Reply to this item at:

[Freeciv-Dev] [patch #3808] Fast units avoid autoattack

2013-04-01 Thread Marko Lindqvist
Update of patch #3808 (project freeciv): Status: In Progress = None ___ Reply to this item at: http://gna.org/patch/?3808 ___ Message sent

[Freeciv-Dev] [patch #3540] Effect description

2013-04-01 Thread Marko Lindqvist
Update of patch #3540 (project freeciv): Planned Release: 2.5.0 = 2.6.0 ___ Reply to this item at: http://gna.org/patch/?3540 ___ Message sent

[Freeciv-Dev] [patch #3793] [metaticket] Help updates for 2.5.0

2013-04-01 Thread Marko Lindqvist
Update of patch #3793 (project freeciv): Depends on: = patch #3137 ___ Reply to this item at: http://gna.org/patch/?3793 ___ Message sent

[Freeciv-Dev] [bug #20700] assert (index) = 0 (index) MAP_INDEX_SIZE failed

2013-04-01 Thread Marko Lindqvist
URL: http://gna.org/bugs/?20700 Summary: assert (index) = 0 (index) MAP_INDEX_SIZE failed Project: Freeciv Submitted by: cazfi Submitted on: Tue 02 Apr 2013 02:05:14 AM EEST Category: None Severity: 3 -

[Freeciv-Dev] [patch #3539] Scorelog: new tags and details

2013-04-01 Thread Davide Baldini
Follow-up Comment #7, patch #3539 (project freeciv): I have tools to process the new scorelog format. If you are interested to see how they work, I have created this thread http://freeciv-mundi.org/forum/viewtopic.php?f=17t=535 to answer the question.

[Freeciv-Dev] [bug #20521] Government Upkeep_Factor help wrong

2013-04-01 Thread Jacob Nevins
Follow-up Comment #1, bug #20521 (project freeciv): However, the help for [civ2] Settlers says: Settlers in a Republic or Democracy require twice as much food per turn, and in Communist or Fundamentalist societies, three times as much. I haven't checked the original game, but two references

[Freeciv-Dev] [bug #20700] assert (index) = 0 (index) MAP_INDEX_SIZE failed

2013-04-01 Thread Marko Lindqvist
Follow-up Comment #1, bug #20700 (project freeciv): Note that text mentioning AEGIS Cruiser in static buffer is from previous unit_tile_link() call (or so I believe). Currently handled unit seems to have corrupted type and tile, later values in structure seem sane. So this seems like memory

[Freeciv-Dev] [bug #20700] assert (index) = 0 (index) MAP_INDEX_SIZE failed

2013-04-01 Thread Marko Lindqvist
Follow-up Comment #2, bug #20700 (project freeciv): I don't know if it's the problem I'm seeing, but there's apparent problem with the code when multiple units are being sold. If there's both transport and its cargo in the list of potential units to sell, selling transport first will cause, due

[Freeciv-Dev] [bug #20700] assert (index) = 0 (index) MAP_INDEX_SIZE failed

2013-04-01 Thread Marko Lindqvist
Update of bug #20700 (project freeciv): Category:None = general Status:None = Ready For Test Planned Release: = 2.3.5, 2.4.0, 2.5.0

[Freeciv-Dev] [patch #3817] Generalise MR_BAD_TYPE_FOR_CITY_TAKE_OVER_BY_SEA

2013-04-01 Thread Emmet Hikory
Follow-up Comment #2, patch #3817 (project freeciv): Not so concretely or specifically, but ruleset authors may model shore bombardment by not allowing the unit to take over the city (set either UTYF_CIVILIAN or fail to set UCF_CAN_OCCUPY_CITY). In all shipped rulesets, unit classes identifed as

[Freeciv-Dev] [bug #16164] Aircraft commit suicide if 'autoattack' is set

2013-04-01 Thread Emmet Hikory
Follow-up Comment #5, bug #16164 (project freeciv): In the case where occupychance 0, should a fueled unit with autoattack still autoattack from a refueling point? If it does, should it return to base immediately thereafter in the event it does occupy, or should it avoid autoattacking in the

[Freeciv-Dev] [patch #3805] Allow paratroopers to land on transport

2013-04-01 Thread Emmet Hikory
Follow-up Comment #6, patch #3805 (project freeciv): Oops, right. Updated patch (transport-paradrop+option+docs.patch) includes that. (file #17654) ___ Additional Item Attachment: File name: transport-paradrop+option+docs.patch Size:8 KB