[Freeciv-Dev] [patch #6985] Enable NLS in msys2 based Installer builds

2016-05-09 Thread Marko Lindqvist
Update of patch #6985 (project freeciv):

  Status: In Progress => Ready For Test 

___

Follow-up Comment #2:

Patches

(file #27361, file #27362)
___

Additional Item Attachment:

File name: NlsRestore.patch   Size:2 KB
File name: NlsRestore-S2_6.patch  Size:2 KB


___

Reply to this item at:

  

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #7198] Add "-msys2" to version label of msys2 builds

2016-05-09 Thread Marko Lindqvist
Update of patch #7198 (project freeciv):

  Status:  Ready For Test => Done   
 Assigned to:None => cazfi  
 Open/Closed:Open => Closed 


___

Reply to this item at:

  

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [bug #24080] Include packet/field name in "Trying to put X into Y bits" messages?

2016-05-09 Thread Marko Lindqvist
Follow-up Comment #2, bug #24080 (project freeciv):

> The maximal number of fragments is 15

-> bug #24654

___

Reply to this item at:

  

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [bug #24654] Limit of 15 fragments

2016-05-09 Thread Marko Lindqvist
URL:
  

 Summary: Limit of 15 fragments
 Project: Freeciv
Submitted by: cazfi
Submitted on: Tue 10 May 2016 02:18:44 AM EEST
Category: general
Severity: 3 - Normal
Priority: 5 - Normal
  Status: None
 Assigned to: None
Originator Email: 
 Open/Closed: Open
 Release: 
 Discussion Lock: Any
Operating System: None
 Planned Release: 

___

Details:

From bug #24080 comments:
Remotely related observation: The maximal number of fragments is 15, otherwise
I get a loud bang and error messages telling me that X * 16 doesn't fit into 8
bits for 15 < X. (I wanted 100 fragments to get percent ;-)





___

Reply to this item at:

  

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [bug #24463] 'enter maketplace' without marketplace

2016-05-09 Thread Marko Lindqvist
Update of bug #24463 (project freeciv):

 Planned Release: => 2.6.0, 3.0.0   

___

Follow-up Comment #3:

You should be able to do that with action enablers. For caravan (and diplomat)
actions those are available in freeciv-2.6 already (System is being expanded
in 3.0).

___

Reply to this item at:

  

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #6826] IPv6 support without inet_pton()

2016-05-09 Thread Marko Lindqvist
Follow-up Comment #2, patch #6826 (project freeciv):

> - No longer have inet_pton() as a hard requirement for enabling IPv6 support
at configure time

...and that's how this patch could enable IPv6 support on msys1 based builds.

___

Reply to this item at:

  

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #7212] Action not enabled explain unknown target tile

2016-05-09 Thread Sveinung Kvilhaugsvik
URL:
  

 Summary: Action not enabled explain unknown target tile
 Project: Freeciv
Submitted by: sveinung
Submitted on: Mon 09 May 2016 10:44:50 PM UTC
Category: general
Priority: 5 - Normal
  Status: Ready For Test
 Privacy: Public
 Assigned to: sveinung
Originator Email: 
 Open/Closed: Open
 Discussion Lock: Any
 Planned Release: 3.0.0

___

Details:





___

File Attachments:


---
Date: Mon 09 May 2016 10:44:50 PM UTC  Name:
0007-Action-not-enabled-explain-unknown-target-tile.patch  Size: 3kB   By:
sveinung



___

Reply to this item at:

  

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [bug #23790] Capital_City is evaluated on the client

2016-05-09 Thread Sveinung Kvilhaugsvik
Update of bug #23790 (project freeciv):

 Planned Release:3.0.0, 2.6.0 => 3.0.0  

___

Follow-up Comment #5:

> rather big change to S2_6 at this point
Agreed.

> Any progress on this?
I did some work on storing non map player facts like capital, amount of gold
etc and when it was last seen.

___

Reply to this item at:

  

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #7211] sandbox: allow helping allies build wonders

2016-05-09 Thread Sveinung Kvilhaugsvik
URL:
  

 Summary: sandbox: allow helping allies build wonders
 Project: Freeciv
Submitted by: sveinung
Submitted on: Mon 09 May 2016 09:42:29 PM UTC
Category: rulesets
Priority: 5 - Normal
  Status: Ready For Test
 Privacy: Public
 Assigned to: sveinung
Originator Email: 
 Open/Closed: Open
 Discussion Lock: Any
 Planned Release: 3.0.0, 2.6.0

___

Details:

Make it possible to do the "Help Wonder" action to help cities belonging to
allies and team mates in addition to domestic cities.



___

File Attachments:


---
Date: Mon 09 May 2016 09:42:29 PM UTC  Name:
3.0-sandbox-allow-helping-allies-build-wonders.patch  Size: 2kB   By: sveinung


---
Date: Mon 09 May 2016 09:42:29 PM UTC  Name:
2.6-sandbox-allow-helping-allies-build-wonders.patch  Size: 1kB   By: sveinung



___

Reply to this item at:

  

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #7210] Document portable storage directory dir separator

2016-05-09 Thread Marko Lindqvist
URL:
  

 Summary: Document portable storage directory dir separator
 Project: Freeciv
Submitted by: cazfi
Submitted on: Tue 10 May 2016 12:33:37 AM EEST
Category: docs
Priority: 5 - Normal
  Status: Ready For Test
 Privacy: Public
 Assigned to: None
Originator Email: 
 Open/Closed: Open
 Discussion Lock: Any
 Planned Release: 3.0.0

___

Details:

Document the build time dir separator conversion - and that '/' is thus
considered portable dir separator in the .project file.



___

File Attachments:


---
Date: Tue 10 May 2016 12:33:37 AM EEST  Name:
PortableStorageDirSeparator.patch  Size: 625B   By: cazfi



___

Reply to this item at:

  

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #7209] Make NoStackDeath an Extra flag.

2016-05-09 Thread Sveinung Kvilhaugsvik
URL:
  

 Summary: Make NoStackDeath an Extra flag.
 Project: Freeciv
Submitted by: sveinung
Submitted on: Mon 09 May 2016 09:11:21 PM UTC
Category: general
Priority: 5 - Normal
  Status: Ready For Test
 Privacy: Public
 Assigned to: sveinung
Originator Email: 
 Open/Closed: Open
 Discussion Lock: Any
 Planned Release: 3.0.0

___

Details:

Change the flag NoStackDeath from a Base flag to an Extra flag. This allows
non base Extras to use it too.

Applies on top of patch #7208



___

File Attachments:


---
Date: Mon 09 May 2016 09:11:21 PM UTC  Name:
0005-Make-NoStackDeath-an-Extra-flag.patch  Size: 20kB   By: sveinung



___

Reply to this item at:

  

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [bug #23790] Capital_City is evaluated on the client

2016-05-09 Thread Marko Lindqvist
Follow-up Comment #4, bug #23790 (project freeciv):

I've been thinking this, from the perspective of it being, with all required
changes to make it work well, rather big change to S2_6 at this point (one of
the last open issues against npf). In fact, I think this can't be done in S2_6
any more, but it would also require savefile format change (-> must be in a
branch before d3f) for keeping track where the player thinks capital of each
of the opponents to be.

___

Reply to this item at:

  

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [bug #24642] Supported units and present units have different size canvas on citydlg

2016-05-09 Thread Marko Lindqvist
Update of bug #24642 (project freeciv):

 Planned Release: => 3.0.0  


___

Reply to this item at:

  

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [bug #24254] Client man page lacking freeciv-gtk3/x specific commandline parameters

2016-05-09 Thread Marko Lindqvist
Follow-up Comment #2, bug #24254 (project freeciv):

--zoom should not be document now, since it's not built in by default and is
not considered ready for use.

___

Reply to this item at:

  

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #5445] Windows Installer build without freeciv-manual

2016-05-09 Thread Marko Lindqvist
Update of patch #5445 (project freeciv):

  Status:   Need Info => Wont Do
 Assigned to:None => cazfi  
 Open/Closed:Open => Closed 


___

Reply to this item at:

  

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #3065] Option to use system luasql?

2016-05-09 Thread Marko Lindqvist
Update of patch #3065 (project freeciv):

 Planned Release:2.6.0, 3.0.0 => 3.0.0  

___

Follow-up Comment #2:

At least in Debian luasql is packaged so that it's not a one package but each
backend has their own pkg-config file. Handling the combinations of what parts
are available and what are not is bigger change than what I thought to make in
S2_6.

___

Reply to this item at:

  

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #7159] civ2civ3 - Second palace small wonder

2016-05-09 Thread Marko Lindqvist
Update of patch #7159 (project freeciv):

  Status:  Ready For Test => Need Info  


___

Reply to this item at:

  

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #7195] Msys2 based builds to clearly distinguish themselves from msys1 builds

2016-05-09 Thread Marko Lindqvist
Update of patch #7195 (project freeciv):

  Status: In Progress => Ready For Test 
 Planned Release:   2.6.0 => 2.6.0, 3.0.0   

___

Follow-up Comment #2:

Attached patch adds "-msys2" part to the installer exe name. That's enough to
resolve this ticket (and to get first test builds forward) though I hope to
make the msys2/msys1 distinction clear in more places in the future.

(file #27355)
___

Additional Item Attachment:

File name: Msys2InstallerName.patch   Size:1 KB


___

Reply to this item at:

  

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #7199] Fix some shadow warnings seen on OpenBSD / set 5

2016-05-09 Thread Marko Lindqvist
Update of patch #7199 (project freeciv):

  Status:  Ready For Test => Done   
 Assigned to:None => cazfi  
 Open/Closed:Open => Closed 


___

Reply to this item at:

  

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #7208] Save documentation comment about 'extra types' format to ruleset

2016-05-09 Thread Marko Lindqvist
URL:
  

 Summary: Save documentation comment about 'extra types'
format to ruleset
 Project: Freeciv
Submitted by: cazfi
Submitted on: Mon 09 May 2016 10:50:01 PM EEST
Category: freeciv-ruledit
Priority: 5 - Normal
  Status: Ready For Test
 Privacy: Public
 Assigned to: None
Originator Email: 
 Open/Closed: Open
 Discussion Lock: Any
 Planned Release: 3.0.0

___

Details:





___

File Attachments:


---
Date: Mon 09 May 2016 10:50:01 PM EEST  Name: RuleditExtrasComment.patch 
Size: 9kB   By: cazfi



___

Reply to this item at:

  

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [bug #24630] Errors about retired Base flags when loading 2.6 rulesets (in compatibility mode)

2016-05-09 Thread Sveinung Kvilhaugsvik
Update of bug #24630 (project freeciv):

  Status:None => Ready For Test 
 Assigned to:None => sveinung   
 Release: => TRUNK  
Operating System:None => Any

___

Follow-up Comment #2:

Simple fix: add a parameter to the ruleset sanity checking that tells it to
ignore retired stuff. Set it when called before ruleset upgrade.

(file #27353)
___

Additional Item Attachment:

File name: 0003-Stop-pre-upgrade-retired-Base-flags-complaints.patch Size:2 KB


___

Reply to this item at:

  

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #7207] sandbox: require movement to expel a unit

2016-05-09 Thread Sveinung Kvilhaugsvik
URL:
  

 Summary: sandbox: require movement to expel a unit
 Project: Freeciv
Submitted by: sveinung
Submitted on: Mon 09 May 2016 05:46:47 PM UTC
Category: rulesets
Priority: 5 - Normal
  Status: Ready For Test
 Privacy: Public
 Assigned to: sveinung
Originator Email: 
 Open/Closed: Open
 Discussion Lock: Any
 Planned Release: 3.0.0

___

Details:

No current client is able to order a unit without any moves left to expel a
foreign unit. If the server receives an order to do so it will accept it
unless a rule forbids it.

A unit should have some movement left to expel a foreign unit. Add it as a
requirement to the "Expel Unit" action enabler.





___

File Attachments:


---
Date: Mon 09 May 2016 05:46:47 PM UTC  Name:
0002-sandbox-require-movement-to-expel-a-unit.patch  Size: 1kB   By: sveinung



___

Reply to this item at:

  

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [bug #24647] Msys2 build always crashes when opening city dialog

2016-05-09 Thread Marko Lindqvist
Update of bug #24647 (project freeciv):

 Planned Release: 2.5.5, 2.6.0, 3.0.0 => 2.5.4, 2.6.0, 3.0.0

___

Follow-up Comment #2:

Targeting to 2.5.4 as general memory management problem (though seen to cause
real trouble in msys2 only). Existing S2_6 patch does applies against S2_5
HEAD (no need to rebase as I assumed).

___

Reply to this item at:

  

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [bug #24647] Msys2 build always crashes when opening city dialog

2016-05-09 Thread Marko Lindqvist
Update of bug #24647 (project freeciv):

  Status:None => Ready For Test 
 Planned Release: => 2.5.5, 2.6.0, 3.0.0

___

Follow-up Comment #1:

The crash takes place on free(s1) in cma dialog cell_data_func()
In the initial testing using g_free() for full glib memory management resolved
the issue.

Patch attached

(file #27350, file #27351)
___

Additional Item Attachment:

File name: CmaGFree.patch Size:3 KB
File name: CmaGFree-S2_6.patchSize:2 KB


___

Reply to this item at:

  

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #7206] Correct placement for variable declaration in create_line_at_mouse_pos()

2016-05-09 Thread Marko Lindqvist
URL:
  

 Summary: Correct placement for variable declaration in
create_line_at_mouse_pos()
 Project: Freeciv
Submitted by: cazfi
Submitted on: Mon 09 May 2016 05:04:46 PM EEST
Category: client-gtk-3.0
Priority: 5 - Normal
  Status: Ready For Test
 Privacy: Public
 Assigned to: None
Originator Email: 
 Open/Closed: Open
 Discussion Lock: Any
 Planned Release: 2.5.5, 2.6.0, 3.0.0

___

Details:

CodingStyle still prohibits variable declarations in the middle of the
function. Correct such case in gtk-client create_line_at_mouse_pos()



___

File Attachments:


---
Date: Mon 09 May 2016 05:04:46 PM EEST  Name:
CreateLineAtMousePosVariableC89.patch  Size: 3kB   By: cazfi


---
Date: Mon 09 May 2016 05:04:46 PM EEST  Name:
CreateLineAtMousePosVariableC89-S2_6.patch  Size: 3kB   By: cazfi



___

Reply to this item at:

  

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #7205] Correct references to 'ai_choice' to 'adv_choice'

2016-05-09 Thread Marko Lindqvist
URL:
  

 Summary: Correct references to 'ai_choice' to 'adv_choice'
 Project: Freeciv
Submitted by: cazfi
Submitted on: Mon 09 May 2016 04:59:42 PM EEST
Category: docs
Priority: 5 - Normal
  Status: Ready For Test
 Privacy: Public
 Assigned to: None
Originator Email: 
 Open/Closed: Open
 Discussion Lock: Any
 Planned Release: 2.6.0, 3.0.0

___

Details:





___

File Attachments:


---
Date: Mon 09 May 2016 04:59:42 PM EEST  Name: AdvNotAiChoice.patch  Size: 1kB 
 By: cazfi



___

Reply to this item at:

  

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #7204] sandbox: let the Leader scorch the earth

2016-05-09 Thread Sveinung Kvilhaugsvik
URL:
  

 Summary: sandbox: let the Leader scorch the earth
 Project: Freeciv
Submitted by: sveinung
Submitted on: Mon 09 May 2016 01:56:56 PM UTC
Category: rulesets
Priority: 5 - Normal
  Status: Ready For Test
 Privacy: Public
 Assigned to: sveinung
Originator Email: 
 Open/Closed: Open
 Discussion Lock: Any
 Planned Release: 3.0.0

___

Details:

Allow the Leader unit to do the action "Destroy City" to domestic cities.

Permit scorched earth by making it possible to completely destroy cites. Use
costs and limitations to balance it's power.

Only a Leader unit can destroy a city. It has to be in or adjacent to the
target city. This will often increase the risk that he is killed.

It is impossible to destroy foreign cities. A city must be therefore conquered
before it can be destroyed.



___

File Attachments:


---
Date: Mon 09 May 2016 01:56:56 PM UTC  Name:
0001-sandbox-let-the-Leader-scorch-the-earth.patch  Size: 3kB   By: sveinung



___

Reply to this item at:

  

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #7179] Initial patch for run-time language switching; low-level switch_lang()

2016-05-09 Thread Marko Lindqvist
Update of patch #7179 (project freeciv):

  Status:  Ready For Test => Done   
 Assigned to:None => cazfi  
 Open/Closed:Open => Closed 


___

Reply to this item at:

  

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [bug #24271] Attempt to add a tech just mess tech "None"

2016-05-09 Thread Marko Lindqvist
Update of bug #24271 (project freeciv):

  Status:  Ready For Test => Fixed  
 Assigned to:None => cazfi  
 Open/Closed:Open => Closed 


___

Reply to this item at:

  

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #5903] Zoom key

2016-05-09 Thread Marko Lindqvist
Update of patch #5903 (project freeciv):

  Status:  Ready For Test => Done   
 Assigned to:None => cazfi  
 Open/Closed:Open => Closed 


___

Reply to this item at:

  

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #7191] Show scenario format version

2016-05-09 Thread Marko Lindqvist
Update of patch #7191 (project freeciv):

  Status:  Ready For Test => Done   
 Assigned to:None => cazfi  
 Open/Closed:Open => Closed 


___

Reply to this item at:

  

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #7171] Ruleup

2016-05-09 Thread Marko Lindqvist
Follow-up Comment #4, patch #7171 (project freeciv):

- Added newline after the error message.

(file #27345)
___

Additional Item Attachment:

File name: RuleUp-4.patch Size:5 KB


___

Reply to this item at:

  

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #7171] Ruleup

2016-05-09 Thread Sveinung Kvilhaugsvik
Follow-up Comment #3, patch #7171 (project freeciv):

This would also be useful for third party ruleset editors.

___

Reply to this item at:

  

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev