[Freeciv-Dev] [bug #21443] Empty global_init_techs not saved

2014-01-19 Thread Marko Lindqvist
Update of bug #21443 (project freeciv):

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


___

Reply to this item at:

  http://gna.org/bugs/?21443

___
  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 #4439] meta.ruleset

2014-01-19 Thread Marko Lindqvist
URL:
  http://gna.org/patch/?4439

 Summary: meta.ruleset
 Project: Freeciv
Submitted by: cazfi
Submitted on: Sun 19 Jan 2014 10:37:44 AM EET
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: 2.6.0

___

Details:

Freeciv-ruledit will need to save some metadata related to the ruleset so that
user is not required to set those settings every time (s)he continues to edit
the ruleset.
Attached patch adds meta.ruleset for storing such information in the future.
The ruleset is loaded in ruleset.c like any other ruleset. While it could make
sense for now to make it specific to freeciv-ruledit (not loaded to
freeciv-server), server too will want meta.ruleset once we implement ruleset
object deprecation framework.
Ruleset is not required to have meta.ruleset. There's empty one, satisfying
ruleset loading code needs, in default/ to be used by the rulesets that do not
have it. Freeciv-ruledit always creates meta.ruleset for the ruleset while
saving it.



___

File Attachments:


---
Date: Sun 19 Jan 2014 10:37:44 AM EET  Name: MetaRuleset.patch  Size: 5kB  
By: cazfi

http://gna.org/patch/download.php?file_id=19785

___

Reply to this item at:

  http://gna.org/patch/?4439

___
  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 #21514] Extra space in description of nameless rulesets

2014-01-19 Thread Marko Lindqvist
URL:
  http://gna.org/bugs/?21514

 Summary: Extra space in description of nameless rulesets
 Project: Freeciv
Submitted by: cazfi
Submitted on: Sun 19 Jan 2014 10:44:09 AM EET
Category: freeciv-ruledit
Severity: 3 - Normal
Priority: 5 - Normal
  Status: Ready For Test
 Assigned to: None
Originator Email: 
 Open/Closed: Open
 Release: 
 Discussion Lock: Any
Operating System: None
 Planned Release: 2.6.0

___

Details:

When saving nameless ruleset from freeciv-ruledit, descriptions have extra
space:

Template  building data for Freeciv


Fix attached.




___

File Attachments:


---
Date: Sun 19 Jan 2014 10:44:09 AM EET  Name: RuleditTemplateNameSpace.patch 
Size: 573B   By: cazfi

http://gna.org/bugs/download.php?file_id=19786

___

Reply to this item at:

  http://gna.org/bugs/?21514

___
  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 #4440] Default Ruleset Name edit value to real ruleset name

2014-01-19 Thread Marko Lindqvist
URL:
  http://gna.org/patch/?4440

 Summary: Default Ruleset Name edit value to real ruleset
name
 Project: Freeciv
Submitted by: cazfi
Submitted on: Sun 19 Jan 2014 11:11:40 AM EET
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: 2.6.0

___

Details:


- Use game.control.name as default value for Ruleset Name edit
- Save Ruleset Name edit value as game.about.name.




___

File Attachments:


---
Date: Sun 19 Jan 2014 11:11:40 AM EET  Name: RulesetNameDefault.patch  Size:
1kB   By: cazfi

http://gna.org/patch/download.php?file_id=19787

___

Reply to this item at:

  http://gna.org/patch/?4440

___
  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 #4441] Editing tech requirements

2014-01-19 Thread Marko Lindqvist
URL:
  http://gna.org/patch/?4441

 Summary: Editing tech requirements
 Project: Freeciv
Submitted by: cazfi
Submitted on: Sun 19 Jan 2014 01:18:53 PM EET
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: 2.6.0

___

Details:

This patch implements changing tech requirements in freeciv-ruledit.



___

File Attachments:


---
Date: Sun 19 Jan 2014 01:18:53 PM EET  Name: TechTreeEdit.patch  Size: 7kB  
By: cazfi

http://gna.org/patch/download.php?file_id=19788

___

Reply to this item at:

  http://gna.org/patch/?4441

___
  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 #21516] fileinfoname_required() is cruft?

2014-01-19 Thread Jacob Nevins
URL:
  http://gna.org/bugs/?21516

 Summary: fileinfoname_required() is cruft?
 Project: Freeciv
Submitted by: jtn
Submitted on: Sun Jan 19 14:23:16 2014
Category: None
Severity: 2 - Minor
Priority: 5 - Normal
  Status: None
 Assigned to: None
Originator Email: 
 Open/Closed: Open
 Release: trunk r24159
 Discussion Lock: Any
Operating System: None
 Planned Release: 

___

Details:

Nothing seems to invoke it (on trunk at least).

It dies if a required file can't be found, and gives bad advice about how to
fix the problem (it talks about ensuring files are on FREECIV_PATH, which is
often not what you actually want, I think).

Propose to delete it from any branches where it's not used, unless anyone
objects.




___

Reply to this item at:

  http://gna.org/bugs/?21516

___
  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 #21517] Man pages documentation for paths is wrong

2014-01-19 Thread Jacob Nevins
URL:
  http://gna.org/bugs/?21517

 Summary: Man pages documentation for paths is wrong
 Project: Freeciv
Submitted by: jtn
Submitted on: Sun Jan 19 14:24:47 2014
Category: docs
Severity: 3 - Normal
Priority: 5 - Normal
  Status: In Progress
 Assigned to: jtn
Originator Email: 
 Open/Closed: Open
 Release: trunk r24159
 Discussion Lock: Any
Operating System: Any
 Planned Release: 

___

Details:

Firstly I noticed that freeciv-server.6 typos FREECIV_SCENARIO_PATH for
FREECIV_DATA_PATH, but on digging further I find more important discrepancies
with actual behaviour.

Haven't checked branches other than trunk yet.




___

Reply to this item at:

  http://gna.org/bugs/?21517

___
  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 #4442] Add minimal governments help to freeciv-manual

2014-01-19 Thread Jacob Nevins
URL:
  http://gna.org/patch/?4442

 Summary: Add minimal governments help to freeciv-manual
 Project: Freeciv
Submitted by: jtn
Submitted on: Sun Jan 19 14:27:46 2014
Category: None
Priority: 3 - Low
  Status: In Progress
 Privacy: Public
 Assigned to: jtn
Originator Email: 
 Open/Closed: Open
 Discussion Lock: Any
 Planned Release: 

___

Details:

I want to bulk-review changes to government help in light of my fixes to bug
#20521. Adding a governments page to the freeciv-manual output seems the
quickest way, and having implemented it we may as well keep it.

Note that the output is completely different to the current wiki doc format
http://www.freeciv.org/wiki/Government, so not directly suitable for
updating that. (I've no intention of trying to fix this.)




___

Reply to this item at:

  http://gna.org/patch/?4442

___
  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 #21516] fileinfoname_required() is cruft?

2014-01-19 Thread Jacob Nevins
Update of bug #21516 (project freeciv):

  Status:None = In Progress
 Assigned to:None = jtn

___

Follow-up Comment #1:

Also, utility/shared.h has a declaration for


struct strvec *fileinfonames(const struct strvec *dirs,
 const char *filename);


which doesn't appear to actually exist.

Looks like these things have been true forever (since at least the split into
FREECIV_*_PATH in r16297
http://svn.gna.org/viewcvs/freeciv?revision=16297view=revision in 2009).

___

Reply to this item at:

  http://gna.org/bugs/?21516

___
  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 #4443] Remove references to config.h

2014-01-19 Thread Marko Lindqvist
URL:
  http://gna.org/patch/?4443

 Summary: Remove references to config.h
 Project: Freeciv
Submitted by: cazfi
Submitted on: Sun 19 Jan 2014 04:50:11 PM EET
Category: bootstrap
Priority: 5 - Normal
  Status: In Progress
 Privacy: Public
 Assigned to: cazfi
Originator Email: 
 Open/Closed: Open
 Discussion Lock: Any
 Planned Release: 2.4.2, 2.5.0, 2.6.0

___

Details:

Got an error from shared.c compilation mentioning config.h instead of
fc_config.h, and started checking other places where old config file name is
still used.





___

Reply to this item at:

  http://gna.org/patch/?4443

___
  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 #4443] Remove references to config.h

2014-01-19 Thread Marko Lindqvist
Update of patch #4443 (project freeciv):

  Status: In Progress = Ready For Test 

___

Follow-up Comment #1:

...and patches.

(file #19789, file #19790)
___

Additional Item Attachment:

File name: FcConfigH.patchSize:5 KB
File name: FcConfigH-S2_4.patch   Size:5 KB


___

Reply to this item at:

  http://gna.org/patch/?4443

___
  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 #20521] Government Upkeep_Factor help wrong

2014-01-19 Thread Jacob Nevins
Follow-up Comment #4, bug #20521 (project freeciv):

Second WIP version -- quite different.

Notable changes from first version (probably some of these should be split to
separate patches):
* discard present=FALSE reqs
* more asserts for things that should have been caught during ruleset loading
* cope with effects with both output type and unit type reqs
silence output in more cases in the presence of extra requirements (otherwise
could lie)
* add PL_() i18n where it was missing
* substantially rework veterans help:
** remain silent if more than one (class, flag, type) req (longstanding
FIXME)
** add i18n disambiguation for class/flag/type output (longstanding FIXME)
** in the case of a type req, output the specific veteran rank name
** reword to make it clearer that it applies to newly built units

Also attached is the current difference it makes to government help with
supplied rulesets + variant2. I haven't investigated the following unexpected
diffs, but suspect they are more accurate rather than less:
* civ2civ3 waste/corruption
* alien max tax rates

(file #19791, file #19792)
___

Additional Item Attachment:

File name: trunk-gov-help-worldbonus-wip-bis.patch Size:35 KB
File name: trunk-gov-help-worldbonus-wip-bis-output.diff Size:66 KB


___

Reply to this item at:

  http://gna.org/bugs/?20521

___
  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 #21516] fileinfoname_required() is cruft?

2014-01-19 Thread Jacob Nevins
Update of bug #21516 (project freeciv):

  Status: In Progress = Ready For Test 
 Planned Release: = 2.3.5,2.4.2,2.5.0,2.6.0

___

Additional Item Attachment:

File name: trunk-S2_5-S2_4-S2_3-unused-fileinfo-fns.patch Size:2 KB


___

Reply to this item at:

  http://gna.org/bugs/?21516

___
  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 #21517] Man pages documentation for paths is wrong

2014-01-19 Thread Jacob Nevins
Update of bug #21517 (project freeciv):

  Status: In Progress = Ready For Test 
 Planned Release: = 2.4.2,2.5.0,2.6.0  

___

Follow-up Comment #1:

Trunk patch doesn't apply cleanly to S2_3, and IIRC the situation may be
different there. I may investigate a patch for S2_3 later.

(file #19794)
___

Additional Item Attachment:

File name: trunk-S2_5-S2_4-man-pages-paths.patch Size:9 KB


___

Reply to this item at:

  http://gna.org/bugs/?21517

___
  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 #4442] Add minimal governments help to freeciv-manual

2014-01-19 Thread Jacob Nevins
Update of patch #4442 (project freeciv):

  Status: In Progress = Ready For Test 
 Planned Release: = 2.4,2.2.5.0,2.6.0  

___

Follow-up Comment #1:

Patch applies cleanly to S2_3 but I don't plan to commit it there, as it adds
a new string.

(file #19795)
___

Additional Item Attachment:

File name: trunk-S2_5-S2_4-S2_3-freeciv_manual-govs.patch Size:2 KB


___

Reply to this item at:

  http://gna.org/patch/?4442

___
  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 #4444] Castro culture nation

2014-01-19 Thread anonymous
URL:
  http://gna.org/patch/?

 Summary: Castro culture nation
 Project: Freeciv
Submitted by: None
Submitted on: dom 19 ene 2014 16:54:08 UTC
Category: rulesets
Priority: 5 - Normal
  Status: None
 Privacy: Public
 Assigned to: None
Originator Email: paf...@hotmail.com
 Open/Closed: Open
 Discussion Lock: Any
 Planned Release: 

___

Details:

Just a suggest for a new nation.Hope you like it.



___

File Attachments:


---
Date: dom 19 ene 2014 16:54:08 UTC  Name: castro.ruleset  Size: 1kB   By: None

http://gna.org/patch/download.php?file_id=19796
---
Date: dom 19 ene 2014 16:54:08 UTC  Name: castro.svg  Size: 4kB   By: None

http://gna.org/patch/download.php?file_id=19797
---
Date: dom 19 ene 2014 16:54:08 UTC  Name: castro.jpg  Size: 6kB   By: None

http://gna.org/patch/download.php?file_id=19798

___

Reply to this item at:

  http://gna.org/patch/?

___
  Mensaje enviado vía/por Gna!
  http://gna.org/


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


[Freeciv-Dev] [patch #4434] Natural defense extras

2014-01-19 Thread Marko Lindqvist
Update of patch #4434 (project freeciv):

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


___

Reply to this item at:

  http://gna.org/patch/?4434

___
  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 #21518] Compilation failure due to size_t section file include counter

2014-01-19 Thread Marko Lindqvist
URL:
  http://gna.org/bugs/?21518

 Summary: Compilation failure due to size_t section file
include counter
 Project: Freeciv
Submitted by: cazfi
Submitted on: Sun 19 Jan 2014 08:34:35 PM EET
Category: general
Severity: 3 - Normal
Priority: 5 - Normal
  Status: Ready For Test
 Assigned to: None
Originator Email: 
 Open/Closed: Open
 Release: 
 Discussion Lock: Any
Operating System: None
 Planned Release: 2.6.0

___

Details:

Building current TRUNK with crosser (gcc version 4.6.4) says:
unknown conversion type character 'z' in format [-Werror=format]

I remember similar problem earlier where I did research to find out that there
isn't really portable (as in: works with older compilers than latest
standards) way to printf size_t.

Attached patch works around the problem.



___

File Attachments:


---
Date: Sun 19 Jan 2014 08:34:35 PM EET  Name: IntNumIncludes.patch  Size: 1kB  
By: cazfi

http://gna.org/bugs/download.php?file_id=19800

___

Reply to this item at:

  http://gna.org/bugs/?21518

___
  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 #21519] civ2 ruleset food upkeep excessive?

2014-01-19 Thread Jacob Nevins
URL:
  http://gna.org/bugs/?21519

 Summary: civ2 ruleset food upkeep excessive?
 Project: Freeciv
Submitted by: jtn
Submitted on: Mon Jan 20 01:11:28 2014
Category: rulesets
Severity: 3 - Normal
Priority: 5 - Normal
  Status: Need Info
 Assigned to: None
Originator Email: 
 Open/Closed: Open
 Release: 
 Discussion Lock: Any
Operating System: Any
 Planned Release: 

___

Details:

Split from bug #20521:

In the civ2 ruleset, Settlers' food upkeep for various governments is:
* Anarchy, Despotism, Monarchy: 1
* Republic, Democracy: 2
* Fundamentalism, Communism: 3

Bbug #20521 comment 1 contains some archaeology indicating that the upkeep=3
govs might have been accidental (in earlier Freeciv they were 2).

However, I haven't checked the original Civ2 -- probably that needs doing to
close this ticket.

The 3 upkeep is not at all evident in the game, due to a combination of bug
#20521 and upkeep icons only going up to 2 (patch #3824). If it's a mistake,
this may explain why it's gone unnoticed so long.

(The only other ruleset we ship having 3 upkeep anywhere is multiplayer,
where Fundamentalism has 3 upkeep. This one may have been deliberate.)




___

Reply to this item at:

  http://gna.org/bugs/?21519

___
  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 #20521] Government Upkeep_Factor help wrong

2014-01-19 Thread Jacob Nevins
Follow-up Comment #5, bug #20521 (project freeciv):

 Changing the civ2 food upkeep factors [...] to be the subject of 
 new tickets.
Now bug #21519.

___

Reply to this item at:

  http://gna.org/bugs/?20521

___
  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 #21517] Man pages documentation for paths is wrong

2014-01-19 Thread Jacob Nevins
Update of bug #21517 (project freeciv):

 Planned Release:   2.4.2,2.5.0,2.6.0 = 2.3.5,2.4.2,2.5.0,2.6.0

___

Follow-up Comment #2:

S2_3 version (slightly different as resolution to bug #17787 was different).

(file #19801)
___

Additional Item Attachment:

File name: S2_3-man-pages-paths.patch Size:9 KB


___

Reply to this item at:

  http://gna.org/bugs/?21517

___
  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 #4445] Specify reason for You can't attack there

2014-01-19 Thread Marko Lindqvist
URL:
  http://gna.org/patch/?4445

 Summary: Specify reason for You can't attack there
 Project: Freeciv
Submitted by: cazfi
Submitted on: Mon 20 Jan 2014 08:10:41 AM EET
Category: general
Priority: 5 - Normal
  Status: In Progress
 Privacy: Public
 Assigned to: None
Originator Email: 
 Open/Closed: Open
 Discussion Lock: Any
 Planned Release: 2.4.3, 2.5.0, 2.6.0

___

Details:

It came up in forum thread started by Arbogast that You can't attack there.
message can result from multiple reasons, and it's not always clear what
caused it:
http://forum.freeciv.org/f/viewtopic.php?f=17t=227

I'll look in to making message specify the reason.




___

Reply to this item at:

  http://gna.org/patch/?4445

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


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