Re: [Freeciv-Dev] Terrain options & parameters

2013-03-30 Thread Emmet Hikory
On Sat, Mar 30, 2013 at 10:02:02PM +0200, Marko Lindqvist wrote: > On 30 March 2013 18:42, Emmet Hikory wrote: > > For terrain.ruleset, there are [options] and [parameters] sections. > > Do most of these still serve useful purposes? From a quick look at the > > code and a bit of testing, I wou

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

2013-03-30 Thread Emmet Hikory
URL: Summary: Generalise helptext for AttFromNonNative and Marines Project: Freeciv Submitted by: persia Submitted on: Sun 31 Mar 2013 04:44:58 AM GMT Category: client Priority:

[Freeciv-Dev] [patch #3800] activity_type_iterate() optimization

2013-03-30 Thread Marko Lindqvist
Update of patch #3800 (project freeciv): Status: Ready For Test => Done Assigned to:None => cazfi Open/Closed:Open => Closed

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

2013-03-30 Thread Marko Lindqvist
Update of bug #19474 (project freeciv): Depends on: => bugs #20690 ___ Reply to this item at: ___ Message sent

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

2013-03-30 Thread Marko Lindqvist
Update of bug #19474 (project freeciv): Depends on: => bugs #20689 ___ Reply to this item at: ___ Message sent

[Freeciv-Dev] [bug #20690] maxplayers setting does not prevent midgame player /create

2013-03-30 Thread Marko Lindqvist
URL: Summary: maxplayers setting does not prevent midgame player /create Project: Freeciv Submitted by: cazfi Submitted on: Sun 31 Mar 2013 02:39:37 AM EET Category: general Sev

[Freeciv-Dev] [bug #20689] Maxplayers setting gets set to number of game start players when loading savegame

2013-03-30 Thread Marko Lindqvist
URL: Summary: Maxplayers setting gets set to number of game start players when loading savegame Project: Freeciv Submitted by: cazfi Submitted on: Sun 31 Mar 2013 02:30:10 AM EET Category: gene

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

2013-03-30 Thread Marko Lindqvist
Update of bug #19474 (project freeciv): Summary: max_players dual meaning => max_players cleanup ___ Follow-up Comment #3: > I don't know yet how many separate bugs that makes. Turning this to meta ticket. __

[Freeciv-Dev] [bug #19474] max_players dual meaning

2013-03-30 Thread Marko Lindqvist
Follow-up Comment #2, bug #19474 (project freeciv): Further testing in TRUNK with midgame player /create (new feature in S2_4, which is likely to be affected too) 1. Start singleplayer game with default maxplayers: 126 2. Save 3. Inspect savegame to see that maxplayers setting is 126 there 4. Loa

[Freeciv-Dev] [bug #20661] unit.c::is_square_threatened() uses H_MAP instead of H_FOG

2013-03-30 Thread Marko Lindqvist
Update of bug #20661 (project freeciv): Status: Ready For Test => Fixed Open/Closed:Open => Closed ___ Reply to this item at:

[Freeciv-Dev] [patch #3818] Ruleset option to disable civil war

2013-03-30 Thread Marko Lindqvist
Update of patch #3818 (project freeciv): Category:None => general Status:None => Ready For Test Planned Release: 2.6.0 => 2.5.0 _

[Freeciv-Dev] [patch #3818] Ruleset option to disable civil war

2013-03-30 Thread Marko Lindqvist
URL: Summary: Ruleset option to disable civil war Project: Freeciv Submitted by: cazfi Submitted on: Sat 30 Mar 2013 11:48:35 PM EET Category: None Priority: 5 - Normal

[Freeciv-Dev] [bug #20669] Missing migration from old killcitizen value

2013-03-30 Thread Marko Lindqvist
Update of bug #20669 (project freeciv): Status: Ready For Test => Fixed Assigned to:None => cazfi Open/Closed:Open => Closed _

[Freeciv-Dev] [patch #3807] Update config.rpath

2013-03-30 Thread Marko Lindqvist
Update of patch #3807 (project freeciv): Status: Ready For Test => Done Assigned to:None => cazfi Open/Closed:Open => Closed

[Freeciv-Dev] [bug #20686] assert 'ai->phase_initialized || game.info.phase_mode != PMT_CONCURRENT' failed

2013-03-30 Thread Marko Lindqvist
URL: Summary: assert 'ai->phase_initialized || game.info.phase_mode != PMT_CONCURRENT' failed Project: Freeciv Submitted by: cazfi Submitted on: Sat 30 Mar 2013 11:27:59 PM EET Category: ai

[Freeciv-Dev] [patch #3027] libicu

2013-03-30 Thread Marko Lindqvist
Update of patch #3027 (project freeciv): Planned Release: 2.5.0 => 2.6.0 ___ Follow-up Comment #3: http://site.icu-project.org/ Not only UTF-8, but stuff like collations. As for portability co

[Freeciv-Dev] [bug #20668] Loaded submarines are not stealth

2013-03-30 Thread anonymous
Follow-up Comment #2, bug #20668 (project freeciv): I'm using default ruleset. Cannot say if missiles had been loaded or unloaded recently since there are submarines of AI player which I see with missiles on board. I saw several times missile(s) in the middle of the ocean with my battleship or AWA

Re: [Freeciv-Dev] Terrain options & parameters

2013-03-30 Thread Marko Lindqvist
On 30 March 2013 18:42, Emmet Hikory wrote: > For terrain.ruleset, there are [options] and [parameters] sections. > Do most of these still serve useful purposes? From a quick look at the > code and a bit of testing, I would think they are mostly also handled in > a different way: > > [options

[Freeciv-Dev] Terrain options & parameters

2013-03-30 Thread Emmet Hikory
For terrain.ruleset, there are [options] and [parameters] sections. Do most of these still serve useful purposes? From a quick look at the code and a bit of testing, I would think they are mostly also handled in a different way: [options] may_road: This is also managed per-terrain by the

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

2013-03-30 Thread Emmet Hikory
URL: Summary: Generalise MR_BAD_TYPE_FOR_CITY_TAKE_OVER_BY_SEA Project: Freeciv Submitted by: persia Submitted on: Sat 30 Mar 2013 06:47:00 PM GMT Category: general Priority: 5

[Freeciv-Dev] [patch #3804] Enabling restricted air transport

2013-03-30 Thread Marko Lindqvist
Follow-up Comment #18, patch #3804 (project freeciv): About threading: see recent discussion on freeciv-dev mailing list. Freeciv is essentially single-threaded at the moment, but we sho0uld be preparing to the time it's not. ___ Reply to t

[Freeciv-Dev] [patch #3804] Enabling restricted air transport

2013-03-30 Thread Emmet Hikory
Follow-up Comment #17, patch #3804 (project freeciv): Thanks for the clarification. If the implementation model is acceptable, that's enough for me, and I'll keep rebasing it against trunk locally until it seems safe to apply. If anyone has trouble testing the patch currently attached due to tru

[Freeciv-Dev] [patch #3804] Enabling restricted air transport

2013-03-30 Thread Marko Lindqvist
Follow-up Comment #16, patch #3804 (project freeciv): I'd say that if S2_4 wipe_unit() is not proven to be broken, we don't want to touch it any more. I just am not committing this patch immediately after 36h inspection period, but wait closer to S2_5 branching date (02-May). btw. Should wipe_un

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

2013-03-30 Thread Marko Lindqvist
URL: Summary: Cleanup nativity handling to ignore specials Project: Freeciv Submitted by: cazfi Submitted on: Sat 30 Mar 2013 06:10:10 PM EET Category: general Priority: 5 - Nor

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

2013-03-30 Thread Emmet Hikory
URL: Summary: Provide information about unit class nativity in terrain help Project: Freeciv Submitted by: persia Submitted on: Sat 30 Mar 2013 03:46:16 PM GMT Category: client

[Freeciv-Dev] [patch #3730] Remove MAX_LEN_USERNAME

2013-03-30 Thread pepeto
Update of patch #3730 (project freeciv): Status:None => Ready For Test Planned Release: => 2.5.0 ___ Additional Item Attachment: File name: r

[Freeciv-Dev] [bug #20685] City dialog sometimes comes up empty on Ubuntu Unity

2013-03-30 Thread Emmet Hikory
Follow-up Comment #1, bug #20685 (project freeciv): Aside from Ubuntu 11.10 (Oneiric), all current Ubuntu packages match the Debian packages, and so if there is a desire for a change, it's probably better to ship that change through Debian (although I'll upload something directly if someone happen

[Freeciv-Dev] [bug #18901] Trunk client becomes unresponsive on clicking "Connect"

2013-03-30 Thread Jacob Nevins
Update of bug #18901 (project freeciv): Status:None => Invalid Open/Closed:Open => Closed ___ Reply to this item at:

[Freeciv-Dev] [bug #13563] [RT #34034] wishlist: contol AI aggressiveness

2013-03-30 Thread Jacob Nevins
Follow-up Comment #1, bug #13563 (project freeciv): Does recent work on traits help with this? (Not sure the user can adjust trait values.) ___ Reply to this item at:

[Freeciv-Dev] [bug #20663] Unloading assert failure when autoattack kills transport

2013-03-30 Thread Jacob Nevins
Follow-up Comment #5, bug #20663 (project freeciv): > For the client side I also wonder what happens when allied > transport (of which you see units inside) moves out of sight. > Does transport get removed from the client when cargo is still > in another tile? I assume this case is why we have

[Freeciv-Dev] [bug #20663] Unloading assert failure when autoattack kills transport

2013-03-30 Thread Marko Lindqvist
Follow-up Comment #4, bug #20663 (project freeciv): I think it's the same. I got assert from server side on autogame, but client would probably get same assert upon handling packets server sends in same order. For the client side I also wonder what happens when allied transport (of which you see

[Freeciv-Dev] [bug #20685] City dialog sometimes comes up empty on Ubuntu Unity

2013-03-30 Thread Jacob Nevins
URL: Summary: City dialog sometimes comes up empty on Ubuntu Unity Project: Freeciv Submitted by: jtn Submitted on: Sat Mar 30 14:04:57 2013 Category: client-gtk-2.0 Severity: 3

[Freeciv-Dev] [bug #20679] [Metaticket] move_unit bugs

2013-03-30 Thread Jacob Nevins
Follow-up Comment #1, bug #20679 (project freeciv): Do issues like bug #20682 fall into this category? ___ Reply to this item at: ___ Message sent via/by Gna!

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

2013-03-30 Thread Jacob Nevins
Update of bug #20577 (project freeciv): Summary: new parameter gameloss_style in game.ruleset => new parameter gameloss_style: GameLoss player's property is redistributed instead of disappearing ___ Reply to this item at:

[Freeciv-Dev] [bug #20663] Unloading assert failure when autoattack kills transport

2013-03-30 Thread Jacob Nevins
Follow-up Comment #3, bug #20663 (project freeciv): > I got failed assert about cargo and transport being in same tile > when wipe_unit() tries to unload cargo. Is this the same assertion that was failing at the start of bug #20045? (unit_transport_unload() [unit.c::2072]: assertion 'same_pos(un

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

2013-03-30 Thread Jacob Nevins
Follow-up Comment #7, bug #20680 (project freeciv): > Problem is in documentation. I had hard time deciding which > order cvercmp -binary should take parameters as neither seemed > to work as "natural" in all contexts. Sorry, I was unclear -- I think the algorithm is giving the wrong answer. I.e

[Freeciv-Dev] [patch #3804] Enabling restricted air transport

2013-03-30 Thread Emmet Hikory
Follow-up Comment #15, patch #3804 (project freeciv): Moving the units to a separate patch seems perfectly reasonable, and I'm fairly unconcerned if that ever lands: the units and vector definitions are only set for testing, and may not actually work well in play. Shall I also separate out the re

[Freeciv-Dev] [patch #3804] Enabling restricted air transport

2013-03-30 Thread Marko Lindqvist
Update of patch #3804 (project freeciv): Status:None => Ready For Test Assigned to:None => cazfi ___ Follow-up Comment #14: Reading the patch

[Freeciv-Dev] [patch #3801] Helptext for defense 0 fortify

2013-03-30 Thread Marko Lindqvist
Update of patch #3801 (project freeciv): Status: Ready For Test => Done Assigned to:None => cazfi Open/Closed:Open => Closed

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

2013-03-30 Thread Marko Lindqvist
Update of bug #20652 (project freeciv): Status: In Progress => Ready For Test ___ Follow-up Comment #3: Attached pathc makes homeless caravans to seek new homecity before even considering traderoutes.

[Freeciv-Dev] [patch #3814] Remove gui-win32 from S2_4

2013-03-30 Thread Marko Lindqvist
URL: Summary: Remove gui-win32 from S2_4 Project: Freeciv Submitted by: cazfi Submitted on: Sat 30 Mar 2013 10:21:01 AM EET Category: bootstrap Priority: 5 - Normal