[Freeciv-Dev] [bug #23824] Too low CURL version required

2015-08-26 Thread Sveinung Kvilhaugsvik
Follow-up Comment #2, bug #23824 (project freeciv): 3.0: * Increase straight to 7.15.4 because of patch #6295 * Add required version to INSTALL * Update README.packaging 2.6: * Increase to 7.12.1 * Add required version to INSTALL 2.5: * Increase to 7.12.1 * Add required version to INSTALL *

[Freeciv-Dev] [bug #23824] Too low CURL version required

2015-08-26 Thread Sveinung Kvilhaugsvik
Update of bug #23824 (project freeciv): Status:None = Ready For Test Release: = TRUNK, 2.6, 2.5.1 Operating System:None = Any

[Freeciv-Dev] [bug #23824] Too low CURL version required

2015-08-26 Thread Marko Lindqvist
Update of bug #23824 (project freeciv): Status: Ready For Test = None Release: TRUNK, 2.6, 2.5.1 = Operating System: Any = None

[Freeciv-Dev] [patch #6173] WShadow

2015-08-26 Thread Marko Lindqvist
Update of patch #6173 (project freeciv): Depends on: = patch #6297 ___ Reply to this item at: http://gna.org/patch/?6173 ___ Message sent

[Freeciv-Dev] [patch #6288] Fix Shadow warnings from gtk clients specific code

2015-08-26 Thread Marko Lindqvist
Update of patch #6288 (project freeciv): Status: Ready For Test = Done Assigned to:None = cazfi Open/Closed:Open = Closed

[Freeciv-Dev] [bug #23824] Too low CURL version required

2015-08-26 Thread Sveinung Kvilhaugsvik
Follow-up Comment #4, bug #23824 (project freeciv): _As a hard requirement, curl should be properly documented in INSTALL._ Is adding the minimum version (like I did) enough for this patches? ___ Reply to this item at:

[Freeciv-Dev] [patch #6295] JSON protocol: support URI encoded strings

2015-08-26 Thread Sveinung Kvilhaugsvik
Additional Item Attachment, patch #6295 (project freeciv): File name: 0002-JSON-protocol-support-URI-encoded-strings.patch Size:11 KB ___ Reply to this item at: http://gna.org/patch/?6295

[Freeciv-Dev] [patch #6297] Fix Shadow warnings from xaw client specific code

2015-08-26 Thread Marko Lindqvist
URL: http://gna.org/patch/?6297 Summary: Fix Shadow warnings from xaw client specific code Project: Freeciv Submitted by: cazfi Submitted on: Wed 26 Aug 2015 10:35:40 PM EEST Category: client-xaw Priority:

[Freeciv-Dev] [bug #23824] To low CURL version required

2015-08-26 Thread Sveinung Kvilhaugsvik
URL: http://gna.org/bugs/?23824 Summary: To low CURL version required Project: Freeciv Submitted by: sveinung Submitted on: Wed 26 Aug 2015 05:29:37 PM UTC Category: None Severity: 3 - Normal

[Freeciv-Dev] [patch #6295] JSON protocol: support URI encoded strings

2015-08-26 Thread Sveinung Kvilhaugsvik
Follow-up Comment #3, patch #6295 (project freeciv): What's the reason to that minimum curl version? curl_easy_unescape() and curl_easy_escape() needs 7.15.4. But see bug #23824 _why your implementation of the function to get the handle differs from one in netfile.c in that the handle is in

[Freeciv-Dev] [bug #23824] Too low CURL version required

2015-08-26 Thread Sveinung Kvilhaugsvik
Update of bug #23824 (project freeciv): Assigned to:None = sveinung ___ Follow-up Comment #1: Is it OK to add the version requirement to INSTALL? (It wasn't there already) Is it OK to just

[Freeciv-Dev] [patch #6296] protocol: change city name type to estring

2015-08-26 Thread Sveinung Kvilhaugsvik
URL: http://gna.org/patch/?6296 Summary: protocol: change city name type to estring Project: Freeciv Submitted by: sveinung Submitted on: Wed 26 Aug 2015 03:43:16 PM UTC Category: freeciv-web Priority: 5 -

[Freeciv-Dev] [patch #6296] protocol: change city name type to estring

2015-08-26 Thread Sveinung Kvilhaugsvik
Follow-up Comment #1, patch #6296 (project freeciv): (An estring is a regular string in the raw protocol) ___ Reply to this item at: http://gna.org/patch/?6296 ___ Message sent via/by Gna!

[Freeciv-Dev] [bug #23824] Too low CURL version required

2015-08-26 Thread Marko Lindqvist
Update of bug #23824 (project freeciv): Category:None = bootstrap Planned Release: = 2.5.2, 2.6.0, 3.0.0 Summary: To low CURL version required = Too low CURL version required

[Freeciv-Dev] [patch #6296] protocol: change city name type to estring

2015-08-26 Thread Sveinung Kvilhaugsvik
Update of patch #6296 (project freeciv): Depends on: = patch #6295 ___ Reply to this item at: http://gna.org/patch/?6296 ___ Message sent

[Freeciv-Dev] [patch #6295] JSON protocol: support URI encoded strings

2015-08-26 Thread Marko Lindqvist
Follow-up Comment #4, patch #6295 (project freeciv): Since this change is not going to stable branches (where we shoulsn't bump the minimum requirement more than what is already the truth: bug #23824), I wouldn't object just bumping the minimum curl version to 7.15.4 in TRUNK and forget about

[Freeciv-Dev] [bug #23817] ?bonus:None in some cases just None

2015-08-26 Thread Marko Lindqvist
Update of bug #23817 (project freeciv): Status: Ready For Test = Fixed Assigned to:None = cazfi Open/Closed:Open = Closed

[Freeciv-Dev] [patch #6295] JSON protocol: support URI encoded strings

2015-08-26 Thread Andreas Røsdal
Follow-up Comment #1, patch #6295 (project freeciv): Looks good, I support this change very much! ___ Reply to this item at: http://gna.org/patch/?6295 ___ Message sent via/by Gna!

[Freeciv-Dev] [patch #6295] JSON protocol: support URI encoded strings

2015-08-26 Thread Marko Lindqvist
Follow-up Comment #2, patch #6295 (project freeciv): What's the reason to that minimum curl version? There's no point in checking curl twice. Rather set the minimum version to a variable and use that the existence of suitable curl. It might indeed be a good idea to have a separate curl handle for

[Freeciv-Dev] [bug #23809] Crashing update_unit_info_label(NULL)

2015-08-26 Thread Marko Lindqvist
Update of bug #23809 (project freeciv): Status: Ready For Test = Fixed Assigned to:None = cazfi Open/Closed:Open = Closed

[Freeciv-Dev] [patch #6298] orders system: support the Steal Maps action

2015-08-26 Thread Sveinung Kvilhaugsvik
URL: http://gna.org/patch/?6298 Summary: orders system: support the Steal Maps action Project: Freeciv Submitted by: sveinung Submitted on: Wed 26 Aug 2015 08:42:39 PM UTC Category: general Priority: 5 -

[Freeciv-Dev] [patch #6104] [Killstack] Patch allows unit to escape instead of been killed

2015-08-26 Thread Sveinung Kvilhaugsvik
Follow-up Comment #33, patch #6104 (project freeciv): Updated. (file #24997) ___ Additional Item Attachment: File name: 0001-Rebase.patch Size:16 KB ___ Reply to this

[Freeciv-Dev] [bug #23824] Too low CURL version required

2015-08-26 Thread Sveinung Kvilhaugsvik
Update of bug #23824 (project freeciv): Status:None = Ready For Test Release: = TRUNK, 2.6, 2.5.1 Operating System:None = Any

[Freeciv-Dev] [bug #23813] Middle click claims occupied city is unoccupied

2015-08-26 Thread Sveinung Kvilhaugsvik
Update of bug #23813 (project freeciv): Status: Ready For Test = Fixed Open/Closed:Open = Closed ___ Reply to this item at:

[Freeciv-Dev] [bug #23818] Requirements aren't auto documented

2015-08-26 Thread Sveinung Kvilhaugsvik
Update of bug #23818 (project freeciv): Status: Ready For Test = Fixed Open/Closed:Open = Closed ___ Reply to this item at:

[Freeciv-Dev] [patch #5306] Add the new action Expel Unit

2015-08-26 Thread J. M. Gorbach
Follow-up Comment #27, patch #5306 (project freeciv): Finish it completely ___ Reply to this item at: http://gna.org/patch/?5306 ___ Message sent via/by Gna! http://gna.org/

[Freeciv-Dev] [patch #6295] JSON protocol: support URI encoded strings

2015-08-26 Thread Sveinung Kvilhaugsvik
URL: http://gna.org/patch/?6295 Summary: JSON protocol: support URI encoded strings Project: Freeciv Submitted by: sveinung Submitted on: Wed 26 Aug 2015 03:41:58 PM UTC Category: freeciv-web Priority: 5 -

[Freeciv-Dev] [bug #23825] Last AI phase of the game not finalized

2015-08-26 Thread Marko Lindqvist
URL: http://gna.org/bugs/?23825 Summary: Last AI phase of the game not finalized Project: Freeciv Submitted by: cazfi Submitted on: Thu 27 Aug 2015 06:43:36 AM EEST Category: ai Severity: 3 - Normal

[Freeciv-Dev] [bug #23824] Too low CURL version required

2015-08-26 Thread Marko Lindqvist
Follow-up Comment #6, bug #23824 (project freeciv): (I assume that Marko changed these fields by accident) Yeah, my post was based on earlier ticket state, not to the version you had just posted. ___ Reply to this item at: