[Freeciv-Dev] [patch #3766] Pathfinding to rely on map_move_cost() on handling of UCF_TERRAIN_SPEED

2013-02-27 Thread Marko Lindqvist
URL: Summary: Pathfinding to rely on map_move_cost() on handling of UCF_TERRAIN_SPEED Project: Freeciv Submitted by: cazfi Submitted on: Thu 28 Feb 2013 09:44:04 AM EET Category: general

[Freeciv-Dev] [patch #3747] Move cost estimates with class considered

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

[Freeciv-Dev] [bug #20560] Local lua & tolua version info files not in tarball

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

[Freeciv-Dev] [patch #3765] Error output about failed "mapimg create" command

2013-02-27 Thread Marko Lindqvist
URL: Summary: Error output about failed "mapimg create" command Project: Freeciv Submitted by: cazfi Submitted on: Thu 28 Feb 2013 08:48:47 AM EET Category: general Priority: 5

[Freeciv-Dev] [bug #20579] Manual mapimg saving in pregame segfaults

2013-02-27 Thread Marko Lindqvist
Update of bug #20579 (project freeciv): Status:None => Ready For Test Planned Release: => 2.4.0, 2.5.0 ___ Follow-up Comment #1: Fix (file #17376)

[Freeciv-Dev] [bug #19168] mapimg automatic save dumps images in current directory

2013-02-27 Thread Marko Lindqvist
Follow-up Comment #4, bug #19168 (project freeciv): - Mapimgs created with manual "mapimg create" go to autosave directory to be consistent with how manual saves go there too. "mapimg colortest" still saves to current directory as it's not about state of the current game. (file #17375) __

[Freeciv-Dev] [bug #20579] Manual mapimg saving in pregame segfaults

2013-02-27 Thread Marko Lindqvist
URL: Summary: Manual mapimg saving in pregame segfaults Project: Freeciv Submitted by: cazfi Submitted on: Thu 28 Feb 2013 08:06:17 AM EET Category: general Severity: 3 - Normal

[Freeciv-Dev] [patch #3764] Make ISO topology the default

2013-02-27 Thread Marko Lindqvist
URL: Summary: Make ISO topology the default Project: Freeciv Submitted by: cazfi Submitted on: Thu 28 Feb 2013 07:46:45 AM EET Category: general Priority: 5 - Normal

[Freeciv-Dev] [patch #3762] Make partisan appearance depend on nationality

2013-02-27 Thread Marko Lindqvist
Update of patch #3762 (project freeciv): Depends on: => patch #3763 ___ Reply to this item at: ___ Message sen

[Freeciv-Dev] [patch #3763] city:inspire_partisans() for lua scripting

2013-02-27 Thread Marko Lindqvist
URL: Summary: city:inspire_partisans() for lua scripting Project: Freeciv Submitted by: cazfi Submitted on: Thu 28 Feb 2013 06:56:21 AM EET Category: general Priority: 5 - Norma

[Freeciv-Dev] [patch #3762] Make partisan appearance depend on nationality

2013-02-27 Thread Marko Lindqvist
URL: Summary: Make partisan appearance depend on nationality Project: Freeciv Submitted by: cazfi Submitted on: Thu 28 Feb 2013 06:21:11 AM EET Category: None Priority: 5 - Norm

[Freeciv-Dev] [bug #20575] configure failed under MinGW

2013-02-27 Thread Marko Lindqvist
Update of bug #20575 (project freeciv): Status: In Progress => Ready For Test ___ Follow-up Comment #2: Patch to conditionally set "-pthread" to CFLAGS, and "-lpthread" to LIBS. This didn't break any

[Freeciv-Dev] [bug #20575] configure failed under MinGW

2013-02-27 Thread Marko Lindqvist
Update of bug #20575 (project freeciv): Status:None => In Progress Assigned to:None => cazfi ___ Follow-up Comment #1: It seems that confi

[Freeciv-Dev] [bug #20577] new parameter gameloss_style in game.ruleset

2013-02-27 Thread Marko Lindqvist
Update of bug #20577 (project freeciv): Category:None => general ___ Follow-up Comment #1: While it still needs some work, it looks promising first version. - In the future, please submi

[Freeciv-Dev] [bug #20559] Defenseless city builds walls in grave danger

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

[Freeciv-Dev] [patch #3759] Make freesounds the default soundset

2013-02-27 Thread Daniel Markstedt
Follow-up Comment #2, patch #3759 (project freeciv): I went back to the source and found out that Gavin's statement wasn't entirely accurate, or at least not accurate anymore. The individual URLs to each audio file in freesounds.AUTHORS are are broken (presumably due to a website redesign) but I

[Freeciv-Dev] [patch #3759] Make freesounds the default soundset

2013-02-27 Thread Daniel Markstedt
Follow-up Comment #1, patch #3759 (project freeciv): Gavin said this when he released the package a few years back: "I've created a soundset for Freeciv based on samples from http://freesound.org, which licences its samples with a Creative Commons Sampling Plus 1.0 License." Here's a link to the

[Freeciv-Dev] [patch #3572] Use grids for voteinfo_bar

2013-02-27 Thread Jacob Nevins
Follow-up Comment #2, patch #3572 (project freeciv): You can test voting by: * Starting a separate server * Connecting several clients (I usually use 3) * After they've all connected (cf bug #20556), issue "/cmdlevel basic" on the server * Try to change some suitable setting, such as "size" (eithe

[Freeciv-Dev] [bug #19900] Font preferences not respected

2013-02-27 Thread Jacob Nevins
Update of bug #19900 (project freeciv): Status:None => Duplicate Open/Closed:Open => Closed ___ Reply to this item at:

[Freeciv-Dev] [patch #3743] /delegate tidy-up

2013-02-27 Thread Jacob Nevins
Update of patch #3743 (project freeciv): Status: In Progress => Ready For Test ___ Reply to this item at: ___ Message sen

[Freeciv-Dev] [patch #3743] /delegate tidy-up

2013-02-27 Thread Jacob Nevins
Additional Item Attachment, patch #3743 (project freeciv): File name: trunk-S2_4-delegate-rework-rm-debug.patch Size:0 KB ___ Reply to this item at: ___ Message

[Freeciv-Dev] [patch #3743] /delegate tidy-up

2013-02-27 Thread Jacob Nevins
Update of patch #3743 (project freeciv): Status:Done => In Progress Open/Closed: Closed => Open ___ Follow-up Comment #6: ...now of course i

[Freeciv-Dev] [patch #3743] /delegate tidy-up

2013-02-27 Thread Jacob Nevins
Update of patch #3743 (project freeciv): Status: Ready For Test => Done Open/Closed:Open => Closed ___ Follow-up Comment #5: No-one screamed, s

[Freeciv-Dev] [patch #3675] Rivers as road types in rulesets

2013-02-27 Thread Marko Lindqvist
Follow-up Comment #5, patch #3675 (project freeciv): - Added flag "Natural" for all rivers - Made trade increases to be terrain-independent - Removed flag "PreventsOtherRoads" flag from alien ruleset river as there road building is always possible on river tiles too - Removed a lot of fields with

[Freeciv-Dev] [patch #3761] Document river road type fields

2013-02-27 Thread Marko Lindqvist
URL: Summary: Document river road type fields Project: Freeciv Submitted by: cazfi Submitted on: Wed 27 Feb 2013 11:30:25 PM EET Category: docs Priority: 5 - Normal

[Freeciv-Dev] [patch #3738] Count defense bonuses from natural roads to sdl-client terrain defense bonus string

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

[Freeciv-Dev] [patch #3755] Road type output increase regardless of terrain

2013-02-27 Thread Marko Lindqvist
Follow-up Comment #2, patch #3755 (project freeciv): Also, as it's over a year since last ruleset capstr capability bump, I plan to do one in the same commit as this change goes in. ___ Reply to this item at:

[Freeciv-Dev] [bug #20008] Tileset priority not respected

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

[Freeciv-Dev] [patch #3756] RFC: UI for building generic roads, bases, etc

2013-02-27 Thread Marko Lindqvist
Follow-up Comment #2, patch #3756 (project freeciv): What about removal of the certain kind of extras? Will the p)illage to remain as it is now, to popup list of pillageable targets when pillageselect is enabled, and to automatically select target when not. What about cleaning of pollution or fall

[Freeciv-Dev] [bug #20576] Trait "Aggressive" uses non-standard scale in default AI.

2013-02-27 Thread Marko Lindqvist
Follow-up Comment #1, bug #20576 (project freeciv): Problem here preventing something like simple use of factor 3 (to make 50 -> 150) is that this value is used in more like exponential than linear fashion, so to have max value of 2500 to translate as 2500 * 3 = 7500 instead of just 2500 would be

[Freeciv-Dev] [patch #3755] Road type output increase regardless of terrain

2013-02-27 Thread Marko Lindqvist
Update of patch #3755 (project freeciv): Status:None => Ready For Test ___ Follow-up Comment #1: Attached patch solves this by adding second set of output increase entries for road types, thes

[Freeciv-Dev] [patch #3469] Get rid of most deprecated warning

2013-02-27 Thread RafałMużyło
Follow-up Comment #89, patch #3469 (project freeciv): Actually, most of my testing was done on 2.4 branch, so I still have my changes rebased on top of S2_4 branch. Though I did cherry-pick a few other patches from trunk, including one or two that I probably didn't need that much, so you'd need to

[Freeciv-Dev] [bug #20565] automake warning: variable `EXTRA_freeciv_manual_DEPENDENCIES' is defined but...

2013-02-27 Thread Marko Lindqvist
Follow-up Comment #5, bug #20565 (project freeciv): It's an error in the sense that "_DEPENDENCIES" is reserved suffix for a automake variable name, and for that reserved meaning this variable name makes no sense. ___ Reply to this item at:

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

2013-02-27 Thread RafałMużyło
Follow-up Comment #2, bug #20534 (project freeciv): Going deeper into the code, it could hint to a problem with gdk_pixbuf_composite in canvas_put_sprite, but hard to tell without more testing. My guess would be you're hitting following documented case: When the destination rectangle contains pa

[Freeciv-Dev] [bug #20577] new parameter gameloss_style in game.ruleset

2013-02-27 Thread Not Given
URL: Summary: new parameter gameloss_style in game.ruleset Project: Freeciv Submitted by: imhotep Submitted on: Wed 27 Feb 2013 06:18:26 PM GMT Category: None Severity: 3 - Norm

[Freeciv-Dev] [bug #20565] automake warning: variable `EXTRA_freeciv_manual_DEPENDENCIES' is defined but...

2013-02-27 Thread RafałMużyło
Follow-up Comment #4, bug #20565 (project freeciv): Which version of automake are you using ? AFAICT, while the construct does look redundant in this particular case, it's nevertheless valid (that is, it shouldn't trigger an error). Regarding the fixes: while they look OK, it seems that '../clien

[Freeciv-Dev] [bug #20576] Trait "Aggressive" uses non-standard scale in default AI.

2013-02-27 Thread Marko Lindqvist
URL: Summary: Trait "Aggressive" uses non-standard scale in default AI. Project: Freeciv Submitted by: cazfi Submitted on: Wed 27 Feb 2013 07:14:32 PM EET Category: ai Severity:

[Freeciv-Dev] [patch #3760] IPv6 support check from callers to net_lookup_service()

2013-02-27 Thread Marko Lindqvist
URL: Summary: IPv6 support check from callers to net_lookup_service() Project: Freeciv Submitted by: cazfi Submitted on: Wed 27 Feb 2013 05:53:30 PM EET Category: general Priori

[Freeciv-Dev] [patch #3759] Make freesounds the default soundset

2013-02-27 Thread Marko Lindqvist
URL: Summary: Make freesounds the default soundset Project: Freeciv Submitted by: cazfi Submitted on: Wed 27 Feb 2013 04:37:14 PM EET Category: None Priority: 5 - Normal

[Freeciv-Dev] [bug #20569] "Trying to put -1 into 16 bits" in transfer_city

2013-02-27 Thread Not Given
Follow-up Comment #1, bug #20569 (project freeciv): with r22446 I get the message: 1: Trying to put 98979 into 16 bits 98979 is the build_cost for the Leader ___ Reply to this item at:

[Freeciv-Dev] [bug #20575] configure failed under MinGW

2013-02-27 Thread anonymous
URL: Summary: configure failed under MinGW Project: Freeciv Submitted by: None Submitted on: Wed 27 Feb 2013 02:15:02 PM UTC Category: general Severity: 3 - Normal

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

2013-02-27 Thread Marko Lindqvist
Update of bug #20498 (project freeciv): Category:None => general Status:None => Ready For Test ___ Follow-up Comment #7: Fix (file #17363)

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

2013-02-27 Thread Marko Lindqvist
Follow-up Comment #6, bug #20498 (project freeciv): > 2) It uses unit_list_iterate() when it should use > unit_list_iterate_safe() as units are removed from the list > during the iteration ...which gave me the idea that I should check other users of unit_transport_unload(), and indeed wipe_unit(

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

2013-02-27 Thread Marko Lindqvist
Follow-up Comment #5, bug #20498 (project freeciv): > [13:14:57] in unit_virtual_destroy() > [../../src/common/unit.c::1890]: > assertion 'unit_list_size(punit->transporting) == 0' failed. This at least indicates easy to spot bug. It might be separate issue, as I didn't see this failed assert as

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

2013-02-27 Thread Marko Lindqvist
Follow-up Comment #4, bug #20498 (project freeciv): In a longish game I first got for a long time (many turns) every turn several: in begin_phase() [../../src/server/srv_main.c::945]: assertion 'ptrans != ((void *)0)' failed ...and now, maybe related... in unit_transport_unload() [../../src/comm

[Freeciv-Dev] [bug #20570] No defenders for wonder city

2013-02-27 Thread Marko Lindqvist
Follow-up Comment #1, bug #20570 (project freeciv): ...and this condition not to build defenders city assigned as wonder city applies even when there's no wonders to build. It still has no defenders, but now it's desperately going through build target fallbacks. When it cannot build anything else,

[Freeciv-Dev] [patch #3756] RFC: UI for building generic roads, bases, etc

2013-02-27 Thread Marko Lindqvist
Follow-up Comment #1, patch #3756 (project freeciv): I'll give you one more challenge: It would be great if you can incorporate to your design support (probably controlled by ruleset author) for some automatic decisions even when the keypress would otherwise seem ambiguous. For instance in the ali

[Freeciv-Dev] [bug #20572] AI observer timeout display badly off

2013-02-27 Thread Marko Lindqvist
URL: Summary: AI observer timeout display badly off Project: Freeciv Submitted by: cazfi Submitted on: Wed 27 Feb 2013 11:36:21 AM EET Category: None Severity: 3 - Normal

[Freeciv-Dev] [patch #3758] Remove dummy thread implementation stuff from fcthread.h

2013-02-27 Thread Marko Lindqvist
URL: Summary: Remove dummy thread implementation stuff from fcthread.h Project: Freeciv Submitted by: cazfi Submitted on: Wed 27 Feb 2013 11:23:28 AM EET Category: general Prior

[Freeciv-Dev] [bug #20571] No thread condition variable support for winthreads

2013-02-27 Thread Marko Lindqvist
URL: Summary: No thread condition variable support for winthreads Project: Freeciv Submitted by: cazfi Submitted on: Wed 27 Feb 2013 11:18:59 AM EET Category: None Severity: 3 -

[Freeciv-Dev] [bug #20570] No defenders for wonder city

2013-02-27 Thread Marko Lindqvist
URL: Summary: No defenders for wonder city Project: Freeciv Submitted by: cazfi Submitted on: Wed 27 Feb 2013 11:06:59 AM EET Category: ai Severity: 3 - Normal P

[Freeciv-Dev] [bug #20569] "Trying to put -1 into 16 bits" in transfer_city

2013-02-27 Thread Marko Lindqvist
URL: Summary: "Trying to put -1 into 16 bits" in transfer_city Project: Freeciv Submitted by: cazfi Submitted on: Wed 27 Feb 2013 10:51:59 AM EET Category: None Severity: 3 - No

[Freeciv-Dev] [patch #3715] Threaded AI cities generating Worker task requests

2013-02-27 Thread Marko Lindqvist
Update of patch #3715 (project freeciv): Status: In Progress => Ready For Test ___ Follow-up Comment #2: - Set want for task requests for workers to prioritize between cities As there's no fear to br

[Freeciv-Dev] [patch #3757] Value for worker tasks

2013-02-27 Thread Marko Lindqvist
URL: Summary: Value for worker tasks Project: Freeciv Submitted by: cazfi Submitted on: Wed 27 Feb 2013 10:29:25 AM EET Category: general Priority: 5 - Normal

[Freeciv-Dev] [bug #20567] Lua random() not really random

2013-02-27 Thread Marko Lindqvist
Follow-up Comment #2, bug #20567 (project freeciv): Turned out that start position assignment does not use mapseed, but main gameseed, so the main random state has been changing in mapgenerator. This does not explain lua script randomness problems, but attached patch anyway changes this so that ma