[Freeciv-Dev] [patch #6939] Prefer pkg-config method for finding ImageMagick

2016-02-09 Thread mir3x
Follow-up Comment #1, patch #6939 (project freeciv): Is ImageMagick used only for saving minimap only? If yes, I would remove it, cairo or qt can do the same, also saving minimap is not useful at all. Better would be to save whole map - Eg client could stop updating screen, resize map to maximum,

[Freeciv-Dev] [patch #6939] Prefer pkg-config method for finding ImageMagick

2016-02-09 Thread Marko Lindqvist
Follow-up Comment #2, patch #6939 (project freeciv): Mapimg is typically saved on server side (no need to have client attached) > /help mapimg Cairo would be a bit heavy requirement (with all the dependencies) to have on servers just for this, but I wouldn't mind someone implementing saving as

[Freeciv-Dev] [patch #6941] Qt client - add default font

2016-02-09 Thread mir3x
URL: Summary: Qt client - add default font Project: Freeciv Submitted by: mir3x Submitted on: Tue 09 Feb 2016 08:07:24 PM CET Category: client-qt Priority: 5 - Normal

[Freeciv-Dev] S2_6 Datafileformat freeze (d3f) 09-Mar-16

2016-02-09 Thread Marko Lindqvist
We are one month from the date S2_6 d3f is finally supposed to happen. Please keep that in mind when considering any new changes to file formats. I still would take in changes fixing problems in current formats, something that would otherwise be likely to cause us headache during S2_6 lifetime,

[Freeciv-Dev] [patch #6922] Shadow warning fixes (from OpenBSD build)

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

[Freeciv-Dev] [patch #6329] pf_normal_map_iterate() optimization

2016-02-09 Thread pepeto
Follow-up Comment #2, patch #6329 (project freeciv): This is due to patch #5201. Using right values for AI calculations is slow... I never could find a better solution for reversed (a real one). If someone does, I would be interested to know how it works!

[Freeciv-Dev] [bug #24391] C-formated string haven't enough following parameters

2016-02-09 Thread Marko Lindqvist
Update of bug #24391 (project freeciv): Status: Ready For Test => In Progress ___ Follow-up Comment #1: It shouldn't be research_name (you should already know what team you are part of) but name of

[Freeciv-Dev] [patch #6913] Caravan destination search to not iterate cities of no-go players at all

2016-02-09 Thread Marko Lindqvist
Update of patch #6913 (project freeciv): Planned Release: 2.5.4, 2.6.0, 3.0.0 => 2.6.0, 3.0.0 ___ Follow-up Comment #2: > I'm going to run some performance tests on this to see if there's enough benefit to justify

[Freeciv-Dev] [patch #6937] Reserved AI callback slots

2016-02-09 Thread Marko Lindqvist
URL: Summary: Reserved AI callback slots Project: Freeciv Submitted by: cazfi Submitted on: Tue 09 Feb 2016 04:14:30 PM EET Category: general Priority: 5 - Normal

[Freeciv-Dev] [patch #6939] Prefer pkg-config method for finding ImageMagick

2016-02-09 Thread Marko Lindqvist
URL: Summary: Prefer pkg-config method for finding ImageMagick Project: Freeciv Submitted by: cazfi Submitted on: Tue 09 Feb 2016 05:22:22 PM EET Category: bootstrap Priority:

[Freeciv-Dev] [task #7963] Tracking release 2.6.0-beta1

2016-02-09 Thread Marko Lindqvist
Update of task #7963 (project freeciv): Depends on: => patch #6422 ___ Reply to this item at: ___ Message sent

[Freeciv-Dev] [bug #24380] Qt client - menu shortcuts active when in chat window.

2016-02-09 Thread mir3x
Update of bug #24380 (project freeciv): Status: Ready For Test => Fixed Open/Closed:Open => Closed ___ Reply to this item at:

[Freeciv-Dev] [patch #6938] [Metaticket] Get rid of gtkpixcomm

2016-02-09 Thread Marko Lindqvist
URL: Summary: [Metaticket] Get rid of gtkpixcomm Project: Freeciv Submitted by: cazfi Submitted on: Tue 09 Feb 2016 05:08:45 PM EET Category: client-gtk-3.0 Priority: 5 -

[Freeciv-Dev] [task #7963] Tracking release 2.6.0-beta1

2016-02-09 Thread Marko Lindqvist
Update of task #7963 (project freeciv): Depends on: => patch #6939 ___ Reply to this item at: ___ Message sent

[Freeciv-Dev] [patch #6936] Search jansson with pkg-config

2016-02-09 Thread Marko Lindqvist
URL: Summary: Search jansson with pkg-config Project: Freeciv Submitted by: cazfi Submitted on: Tue 09 Feb 2016 04:07:13 PM EET Category: bootstrap Priority: 5 - Normal

[Freeciv-Dev] [patch #6422] AI Module capstr for S2_6

2016-02-09 Thread Marko Lindqvist
Update of patch #6422 (project freeciv): Status: Ready For Test => In Progress Depends on: => patch #6937 ___ Reply to this item at:

[Freeciv-Dev] [patch #5650] Clean up rulesets protecting against multiple Barracks

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

[Freeciv-Dev] [patch #6931] Configure check for pathless SDL includes

2016-02-09 Thread Marko Lindqvist
Follow-up Comment #1, patch #6931 (project freeciv): - Name the macro as SDL2_PLAIN_INCLUDES to better prepare for the future possibility that one version of SDL requires pathless includes and another version accepts path. (file #2) ___

[Freeciv-Dev] [patch #6914] Radiobutton to automate the usual rule_name -> display name update

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

[Freeciv-Dev] [patch #6940] Merge ai.love to rest of ai table in europe_1901 scenario

2016-02-09 Thread Marko Lindqvist
URL: Summary: Merge ai.love to rest of ai table in europe_1901 scenario Project: Freeciv Submitted by: cazfi Submitted on: Tue 09 Feb 2016 05:34:40 PM EET Category: rulesets

[Freeciv-Dev] [bug #24396] Qt client - some menu issues

2016-02-09 Thread mir3x
URL: Summary: Qt client - some menu issues Project: Freeciv Submitted by: mir3x Submitted on: Tue 09 Feb 2016 04:48:13 PM CET Category: client-qt Severity: 3 - Normal

[Freeciv-Dev] [patch #6924] rulesets: remove blank line inside Paratroopers

2016-02-09 Thread Sveinung Kvilhaugsvik
Update of patch #6924 (project freeciv): Planned Release: 3.0.0 => 3.0.0, 2.6.0, 2.5.4 ___ Follow-up Comment #2: I forgot to target 2.6 and 2.5. Patch already attached.

[Freeciv-Dev] [patch #6926] Long range actions don't work in orders

2016-02-09 Thread Sveinung Kvilhaugsvik
Update of patch #6926 (project freeciv): Status: Ready For Test => Done Open/Closed:Open => Closed ___ Reply to this item at:

[Freeciv-Dev] [patch #6923] Hard action reqs: improve sees tile readability

2016-02-09 Thread Sveinung Kvilhaugsvik
Update of patch #6923 (project freeciv): Status: Ready For Test => Done Open/Closed:Open => Closed ___ Reply to this item at:

[Freeciv-Dev] [bug #24391] C-formated string haven't enough following parameters

2016-02-09 Thread Konstantin Vedeneev
Follow-up Comment #2, bug #24391 (project freeciv): Ok, I see the logic. Trying to be more informative: (file #26673) ___ Additional Item Attachment: File name: SrvEdit2.patch Size:0 KB

[Freeciv-Dev] [patch #6730] [Metaticket] Optional deprecation warnings

2016-02-09 Thread Marko Lindqvist
Update of patch #6730 (project freeciv): Summary: [Metaticket] Optional (lua) deprecation warnings => [Metaticket] Optional deprecation warnings ___ Reply to this item at:

[Freeciv-Dev] [patch #6943] Explicitle enable ruledit in Ruledit Installer configure

2016-02-09 Thread Marko Lindqvist
URL: Summary: Explicitle enable ruledit in Ruledit Installer configure Project: Freeciv Submitted by: cazfi Submitted on: Tue 09 Feb 2016 11:52:26 PM EET Category: general

[Freeciv-Dev] [patch #6730] [Metaticket] Optional deprecation warnings

2016-02-09 Thread Marko Lindqvist
Update of patch #6730 (project freeciv): Depends on: => patch #6942 ___ Reply to this item at: ___ Message

[Freeciv-Dev] [patch #6944] Moderating Temperature fix re: issue 24389

2016-02-09 Thread Robert Finch
URL: Summary: Moderating Temperature fix re: issue 24389 Project: Freeciv Submitted by: robfinch Submitted on: Tue 09 Feb 2016 10:49:43 PM UTC Category: None Priority: 5 -

[Freeciv-Dev] [patch #6916] Method for internal lua code to signal use of deprecated signals

2016-02-09 Thread Marko Lindqvist
Update of patch #6916 (project freeciv): Status: Ready For Test => In Progress ___ Follow-up Comment #2: Should be ported to use deprecations module from patch #6942.

[Freeciv-Dev] [patch #6942] deprecations.[ch]

2016-02-09 Thread Marko Lindqvist
URL: Summary: deprecations.[ch] Project: Freeciv Submitted by: cazfi Submitted on: Tue 09 Feb 2016 11:10:21 PM EET Category: general Priority: 5 - Normal

[Freeciv-Dev] [bug #24393] civ1 and civ2 doesn't document action required utype fields

2016-02-09 Thread Sveinung Kvilhaugsvik
Update of bug #24393 (project freeciv): Status: Ready For Test => Fixed Open/Closed:Open => Closed ___ Reply to this item at:

[Freeciv-Dev] [bug #24392] No paradrop while transporting isn't enforeced

2016-02-09 Thread Sveinung Kvilhaugsvik
Update of bug #24392 (project freeciv): Status: Ready For Test => Fixed Open/Closed:Open => Closed ___ Reply to this item at:

Re: [Freeciv-Dev] [patch #6938] [Metaticket] Get rid of gtkpixcomm

2016-02-09 Thread Vasco Alexandre da Silva Costa
GtkPixcomm is a widget I ported over from the Xaw client to GTK+. At the time GTK+'s GtkImage widget had memory leaks and did not include rectangle drawing routines. Without GtkPixcomm the Freeciv client would increase its memory resource usage quickly over the course of 30 minutes because of the

[Freeciv-Dev] [patch #6945] Fracture map generator (2.91) + Resource Clustering

2016-02-09 Thread Robert Finch
URL: Summary: Fracture map generator (2.91) + Resource Clustering Project: Freeciv Submitted by: robfinch Submitted on: Tue 09 Feb 2016 11:08:15 PM UTC Category: None Priority:

[Freeciv-Dev] [patch #6918] Map generator: use multiple lake terrains more sensibly

2016-02-09 Thread Jacob Nevins
Update of patch #6918 (project freeciv): Status: Ready For Test => Done Open/Closed:Open => Closed ___ Reply to this item at:

[Freeciv-Dev] [patch #6917] Make near_singularity() have less fierce effects on map generation

2016-02-09 Thread Jacob Nevins
Update of patch #6917 (project freeciv): Status: Ready For Test => Done Open/Closed:Open => Closed ___ Reply to this item at:

[Freeciv-Dev] [patch #6920] Add terrain flag "Frozen", and use it to colour minimap

2016-02-09 Thread Jacob Nevins
Update of patch #6920 (project freeciv): Status: Ready For Test => Done Open/Closed:Open => Closed ___ Reply to this item at:

[Freeciv-Dev] [bug #24389] Climate moderation at coasts not working as intended

2016-02-09 Thread Jacob Nevins
Follow-up Comment #3, bug #24389 (project freeciv): (patch now available at patch #6944) 1. Sure, but the reason I hesitated is that this code has never been in use, so should be treated like new code; i.e. we should check if it gives a sensible result before fixing the bug. (I haven't tried it

[Freeciv-Dev] [patch #6946] Keep action decision data until taken

2016-02-09 Thread Sveinung Kvilhaugsvik
URL: Summary: Keep action decision data until taken Project: Freeciv Submitted by: sveinung Submitted on: Wed 10 Feb 2016 03:15:48 AM UTC Category: general Priority: 5 - Normal

[Freeciv-Dev] [patch #6945] Fracture map generator (2.91) + Resource Clustering

2016-02-09 Thread mir3x
Follow-up Comment #1, patch #6945 (project freeciv): Hi. You need format code properly. Everyhting is descibed here: http://freeciv.wikia.com/wiki/Coding_Style Shortly what u missed: - tabs to spaces - max line length - 77 - always add brackets to if {} else {}, for{} .. etc .. Overall you can

[Freeciv-Dev] [patch #6947] Log level by name

2016-02-09 Thread Marko Lindqvist
URL: Summary: Log level by name Project: Freeciv Submitted by: cazfi Submitted on: Wed 10 Feb 2016 08:05:31 AM EET Category: general Priority: 5 - Normal

[Freeciv-Dev] [bug #24391] C-formated string haven't enough following parameters

2016-02-09 Thread Marko Lindqvist
Update of bug #24391 (project freeciv): Status: In Progress => Ready For Test ___ Reply to this item at: ___ Message