[Freeciv-Dev] [patch #4903] Coding style: maxTileUnits => max_tile_units

2014-07-06 Thread Sveinung Kvilhaugsvik
Update of patch #4903 (project freeciv): Status: Ready For Test => Done Open/Closed:Open => Closed ___ Reply to this item at:

[Freeciv-Dev] [patch #4877] Obsolete "NoBonus" road move mode

2014-07-06 Thread Marko Lindqvist
Update of patch #4877 (project freeciv): Status: Ready For Test => Done Assigned to:None => cazfi Open/Closed:Open => Closed

Re: [Freeciv-Dev] [bug #22274] Database backend '' not supported by database.lua

2014-07-06 Thread Marko Lindqvist
On 7 July 2014 02:48, Jacob Nevins wrote: > Follow-up Comment #10, bug #22274 (project freeciv): > >> fc_assert(entry_str_get(pentry, &value)); > Good grief, what was I thinking? I think you were *not* thinking, for a decisecond or two. Coding is extreme sport - losing your alertness for a fract

[Freeciv-Dev] [bug #22274] Database backend '' not supported by database.lua

2014-07-06 Thread Jacob Nevins
Follow-up Comment #10, bug #22274 (project freeciv): > fc_assert(entry_str_get(pentry, &value)); Good grief, what was I thinking? ___ Reply to this item at: ___

[Freeciv-Dev] [patch #4878] Cities cannot be built on Alien ruleset Boiling Oceans

2014-07-06 Thread Marko Lindqvist
Follow-up Comment #1, patch #4878 (project freeciv): - S2_5 version (file #21313) ___ Additional Item Attachment: File name: BoilingCities-S2_5.patch Size:1 KB ___ Reply to t

[Freeciv-Dev] [bug #22274] Database backend '' not supported by database.lua

2014-07-06 Thread Marko Lindqvist
Update of bug #22274 (project freeciv): Status:None => Ready For Test ___ Follow-up Comment #9: Fix (file #21312) ___ Additional Item

[Freeciv-Dev] [bug #22287] Calculated advance costs written into ruleset

2014-07-06 Thread Sveinung Kvilhaugsvik
Follow-up Comment #1, bug #22287 (project freeciv): The attached patch fixes the issue for me. ___ Reply to this item at: ___ Message sent via/by Gna! http://g

[Freeciv-Dev] [bug #22274] Database backend '' not supported by database.lua

2014-07-06 Thread Marko Lindqvist
Follow-up Comment #8, bug #22274 (project freeciv): > I had "-DNDEBUG" > and searching patch of bug #19909 for "assert" brings up this suspect: + fc_assert(entry_str_get(pentry, &value)); ___ Reply to this item at:

[Freeciv-Dev] [bug #22274] Database backend '' not supported by database.lua

2014-07-06 Thread Marko Lindqvist
Follow-up Comment #7, bug #22274 (project freeciv): I had to do more than --enable-debug - I had "-DNDEBUG" which didn't work well with "--enable-debug" (and I'm inclined to say we don't need/want to support such combination, so not opening bug about that). But, once I removed "-DNDEBUG" and adde

[Freeciv-Dev] [bug #22288] Ruledit stores (rounded) calculated tech cost

2014-07-06 Thread Sveinung Kvilhaugsvik
Update of bug #22288 (project freeciv): Status:None => Duplicate Open/Closed:Open => Closed ___ Follow-up Comment #1: Dupe of bug #22287

[Freeciv-Dev] [bug #22288] Ruledit stores (rounded) calculated tech cost

2014-07-06 Thread Sveinung Kvilhaugsvik
URL: Summary: Ruledit stores (rounded) calculated tech cost Project: Freeciv Submitted by: sveinung Submitted on: Sun 06 Jul 2014 09:46:41 PM UTC Category: freeciv-ruledit Sever

[Freeciv-Dev] [bug #22184] Windows installer has translations from freeciv domain only

2014-07-06 Thread Marko Lindqvist
Update of bug #22184 (project freeciv): Status: Ready For Test => Fixed Assigned to:None => cazfi Open/Closed:Open => Closed _

[Freeciv-Dev] [bug #22287] Calculated advance costs written into ruleset

2014-07-06 Thread pepeto
URL: Summary: Calculated advance costs written into ruleset Project: Freeciv Submitted by: pepeto Submitted on: dim. 06 juil. 2014 23:40:08 CEST Category: freeciv-ruledit Severi

[Freeciv-Dev] [patch #4879] Mark redundant buildings in sdl-client

2014-07-06 Thread Marko Lindqvist
Update of patch #4879 (project freeciv): Status: Ready For Test => Done Open/Closed:Open => Closed ___ Reply to this item at:

[Freeciv-Dev] [bug #22266] Crosser build of freeciv-mp-cli fails

2014-07-06 Thread Marko Lindqvist
Update of bug #22266 (project freeciv): Status: Ready For Test => Fixed Assigned to:None => cazfi Open/Closed:Open => Closed _

[Freeciv-Dev] [bug #22061] Team pooled research and AI science cost

2014-07-06 Thread pepeto
Update of bug #22061 (project freeciv): Status:None => In Progress Assigned to:None => pepeto Depends on: => bugs #22059 __

[Freeciv-Dev] [bug #22060] EFT_TECH_COST_FACTOR isn't handle with team pooled research

2014-07-06 Thread pepeto
Update of bug #22060 (project freeciv): Status:None => In Progress Assigned to:None => pepeto Depends on: => bugs #22059 __

[Freeciv-Dev] [bug #22059] tech_leakage doesn't handle team pooled research

2014-07-06 Thread pepeto
Update of bug #22059 (project freeciv): Status:None => Ready For Test Assigned to:None => pepeto Planned Release: => 2.6.0 __

[Freeciv-Dev] [patch #4911] Rename and move player_research_step()

2014-07-06 Thread pepeto
URL: Summary: Rename and move player_research_step() Project: Freeciv Submitted by: pepeto Submitted on: dim. 06 juil. 2014 23:01:54 CEST Category: general Priority: 5 - Normal

[Freeciv-Dev] [bug #22286] Ruledit won't compile because of renamed field

2014-07-06 Thread Marko Lindqvist
Follow-up Comment #3, bug #22286 (project freeciv): > Since they are stored as an int the calculated values are now > rounded. Calculated values? Has the ruleset format changed in a way that saving them should reflect? Simple rule: If you do not make any changes to the loaded ruleset in freeciv-r

[Freeciv-Dev] [bug #22286] Ruledit won't compile because of renamed field

2014-07-06 Thread Sveinung Kvilhaugsvik
Follow-up Comment #2, bug #22286 (project freeciv): Note that all done here was to make it compile again. Calculated values are now written. Since they are stored as an int the calculated values are now rounded. Should I have commented out the support for writing tech cost in stead of doing what I

[Freeciv-Dev] [bug #22274] Database backend '' not supported by database.lua

2014-07-06 Thread Marko Lindqvist
Follow-up Comment #6, bug #22274 (project freeciv): bug #22275 seems to be related (or more like the cause). I have two entries in my database config file (attached) and with "-d 3" I get the bug #22275 warning twice between the message that server starts to read the config file and the "invalid

[Freeciv-Dev] [bug #22275] Warning about 0-size calloc at luascript_signal.c:223

2014-07-06 Thread Marko Lindqvist
Follow-up Comment #1, bug #22275 (project freeciv): Original submission used line from version where bug #19909 was reverted (see bug #22274). Without the revert I get same warning from luascript_func.c:133. ___ Reply to this item at:

[Freeciv-Dev] [bug #22286] Ruledit won't compile because of renamed field

2014-07-06 Thread Sveinung Kvilhaugsvik
Update of bug #22286 (project freeciv): Status: Ready For Test => Fixed Open/Closed:Open => Closed ___ Reply to this item at:

[Freeciv-Dev] [bug #22286] Ruledit won't compile because of renamed field

2014-07-06 Thread Sveinung Kvilhaugsvik
URL: Summary: Ruledit won't compile because of renamed field Project: Freeciv Submitted by: sveinung Submitted on: Sun 06 Jul 2014 06:59:08 PM UTC Category: freeciv-ruledit Seve

[Freeciv-Dev] [bug #22187] Client allows attempted violation of embarks/disembarks restrictions

2014-07-06 Thread pepeto
Update of bug #22187 (project freeciv): Category:None => general Status: In Progress => Ready For Test ___ Follow-up Comment #6: Fix attached for th

[Freeciv-Dev] [bug #22187] Client allows attempted violation of embarks/disembarks restrictions

2014-07-06 Thread pepeto
Update of bug #22187 (project freeciv): Depends on: => patch #4910 ___ Reply to this item at: ___ Message post

[Freeciv-Dev] [patch #4910] pf_parameter with unit_type pointer

2014-07-06 Thread pepeto
Update of patch #4910 (project freeciv): Depends on: => patch #4889 ___ Reply to this item at: ___ Message pos

[Freeciv-Dev] [patch #4910] pf_parameter with unit_type pointer

2014-07-06 Thread pepeto
URL: Summary: pf_parameter with unit_type pointer Project: Freeciv Submitted by: pepeto Submitted on: dim. 06 juil. 2014 20:32:33 CEST Category: ai Priority: 5 - Normal

[Freeciv-Dev] [bug #22062] Cannot see the real cost of the technologies with some values of tech_leakage

2014-07-06 Thread pepeto
Update of bug #22062 (project freeciv): Status:None => Ready For Test Assigned to:None => pepeto Planned Release: => 2.6.0 __

[Freeciv-Dev] [patch #4909] Remove advance_by_flag()

2014-07-06 Thread pepeto
URL: Summary: Remove advance_by_flag() Project: Freeciv Submitted by: pepeto Submitted on: dim. 06 juil. 2014 20:25:42 CEST Category: general Priority: 5 - Normal

[Freeciv-Dev] [bug #22187] Client allows attempted violation of embarks/disembarks restrictions

2014-07-06 Thread pepeto
Update of bug #22187 (project freeciv): Status:None => In Progress ___ Follow-up Comment #5: Worse than pathfinding violation, the server allows Riflemen to load to Helicopter into Ocean. B

Re: [Freeciv-Dev] styles.ruleset missing

2014-07-06 Thread Marko Lindqvist
On 1 July 2014 18:15, Andrew McGrath wrote: > Hi Everyone, > > I'm trying to get a freeciv server up and running and I'm faced with an > issue when running freeciv-web: > > ubuntu@ip-10-236-131-195:~/freeciv$ freeciv-web The interface between freeciv and freeciv-web is not very clean at the mome

[Freeciv-Dev] [patch #4650] Split single input xx.po file into translations/*/xx.po

2014-07-06 Thread Jacob Nevins
Follow-up Comment #3, patch #4650 (project freeciv): >> I believe this ends up with every source translation in exactly >> one place, > But is that what we want? It's possible that a few strings > (governor titles) are needed for both core nations and extended > nations. You're right, what I des

[Freeciv-Dev] [bug #22281] Help should show Courthouse has no effect in presence of Palace

2014-07-06 Thread Jacob Nevins
Update of bug #22281 (project freeciv): Status: In Progress => Ready For Test Planned Release: => 2.4.3, 2.5.0, 2.6.0 ___ Follow-up Comment #4: * Explicitly noted

[Freeciv-Dev] [patch #4908] Document global and national starting buildings/techs/units in their help

2014-07-06 Thread Jacob Nevins
Update of patch #4908 (project freeciv): Status: In Progress => Ready For Test ___ Follow-up Comment #1: Currently any barbarian/pirate initial goodies are not acknowledged on S2_5 and later, due to t

[Freeciv-Dev] [bug #22281] Help should show Courthouse has no effect in presence of Palace

2014-07-06 Thread Jacob Nevins
Update of bug #22281 (project freeciv): Status:None => In Progress Assigned to:None => jtn ___ Follow-up Comment #3: > It's just that th

[Freeciv-Dev] [bug #20618] Goto shouldn't attack in fogged tiles

2014-07-06 Thread pepeto
Follow-up Comment #1, bug #20618 (project freeciv): I see two ways to implements this: * add a client goto option to have 'vigilant' orders ; * stop the unit if a unit or a city is found on the way, in the middle of the patch. Then, it will requires another order type (so it is not possible to imp

[Freeciv-Dev] [bug #22281] Help should show Courthouse has no effect in presence of Palace

2014-07-06 Thread anonymous
Follow-up Comment #2, bug #22281 (project freeciv): Yep, unless the Palace incorporates all the abilities of a Courthouse (including its ability to be built in multiple cities), the Courthouse shouldn't be marked as redundant in the UI even if the Palace is around. The Palace can move after all, a

[Freeciv-Dev] [patch #4908] Document global and national starting buildings/techs/units in their help

2014-07-06 Thread Jacob Nevins
URL: Summary: Document global and national starting buildings/techs/units in their help Project: Freeciv Submitted by: jtn Submitted on: Sun 06 Jul 2014 14:05:07 BST Category: docs

[Freeciv-Dev] styles.ruleset missing

2014-07-06 Thread Andrew McGrath
Hi Everyone, I'm trying to get a freeciv server up and running and I'm faced with an issue when running freeciv-web: ubuntu@ip-10-236-131-195:~/freeciv$ freeciv-web This is the server for Freeciv version 2.5.99-dev You can learn a lot about Freeciv at http://www.freeciv.org/ This freeciv-server p

[Freeciv-Dev] [bug #22281] Help should show Courthouse has no effect in presence of Palace

2014-07-06 Thread Jacob Nevins
Follow-up Comment #1, bug #22281 (project freeciv): Hm, yes, that could be clearer. Also the Palace help could state explicitly that Palace means your capital can't be incited to revolt, to contrast with the Courthouse's increase in incite cost. (This is also a useful example of the notion of "r

[Freeciv-Dev] [bug #22283] Poor error reporting on failure to open file

2014-07-06 Thread Jacob Nevins
URL: Summary: Poor error reporting on failure to open file Project: Freeciv Submitted by: jtn Submitted on: Sun 06 Jul 2014 12:47:19 BST Category: None Severity: 3 - Normal

[Freeciv-Dev] [bug #22282] convert_string() / iconv() error reporting could be more specific

2014-07-06 Thread Jacob Nevins
URL: Summary: convert_string() / iconv() error reporting could be more specific Project: Freeciv Submitted by: jtn Submitted on: Sun 06 Jul 2014 12:39:24 BST Category: None Seve

[Freeciv-Dev] [patch #4889] Pathfinding refactoring

2014-07-06 Thread pepeto
Update of patch #4889 (project freeciv): Status: In Progress => Ready For Test ___ Follow-up Comment #2: New version: * clarify the usage of paranthesizes in pf_get_action() ; * split 'enum pf_action'

[Freeciv-Dev] [bug #22274] Database backend '' not supported by database.lua

2014-07-06 Thread Jacob Nevins
Follow-up Comment #5, bug #22274 (project freeciv): Using an SQLite database is still working for me, so I'll probably need to know more about your environment: * Are there any non-ASCII characters anywhere in sight (in the configuration file, in its filename, in the database)? * What does your --

[Freeciv-Dev] [bug #22281] Help should show Courthouse has no effect in presence of Palace

2014-07-06 Thread anonymous
URL: Summary: Help should show Courthouse has no effect in presence of Palace Project: Freeciv Submitted by: None Submitted on: Sun 06 Jul 2014 11:15:30 AM UTC Category: docs Se

[Freeciv-Dev] [patch #4864] Distance-based calculation for meeting charges

2014-07-06 Thread Jacob Nevins
Follow-up Comment #3, patch #4864 (project freeciv): Would the simple S2_5 fix be beneficial for S2_4 too? It looks like it could be quite important -- without it, are sea bodyguards going to meet their charge and then immediately wander off? _

[Freeciv-Dev] [bug #21380] Property editor does not honour nation set restrictions

2014-07-06 Thread Jacob Nevins
Follow-up Comment #3, bug #21380 (project freeciv): ...now if you edit a scenario which has start positions for nations not in your current set (which you can't change while editing), it says "1 Nation", but you can't see who it is or change it. Enh. Maybe a "show all" button? ___

[Freeciv-Dev] [patch #4889] Pathfinding refactoring

2014-07-06 Thread pepeto
Update of patch #4889 (project freeciv): Status: Ready For Test => In Progress ___ Reply to this item at: ___ Message pos

[Freeciv-Dev] [bug #21350] Supplied scenario start positions vs core nations

2014-07-06 Thread Jacob Nevins
Follow-up Comment #3, bug #21350 (project freeciv): > When the core nation set is rigidly enforced [...] our supplied > scenarios which specify nation-specific start positions will > cause trouble Oh, past-me, why so vague? I'm guessing that by "trouble" I meant "looks a bit ugly", in that the

[Freeciv-Dev] [patch #4892] Move last harmless functions from tech.[ch] to research.[ch]

2014-07-06 Thread pepeto
Update of patch #4892 (project freeciv): Status: Ready For Test => Done Open/Closed:Open => Closed ___ Reply to this item at:

[Freeciv-Dev] [bug #22280] Obsoleting tech of not-yet-loaded buildings checked

2014-07-06 Thread Marko Lindqvist
URL: Summary: Obsoleting tech of not-yet-loaded buildings checked Project: Freeciv Submitted by: cazfi Submitted on: Sun 06 Jul 2014 12:04:12 PM EEST Category: general Severity:

[Freeciv-Dev] [patch #4891] techs_precalc_data() cleanup

2014-07-06 Thread pepeto
Update of patch #4891 (project freeciv): Status: Ready For Test => Done Open/Closed:Open => Closed ___ Reply to this item at:

[Freeciv-Dev] [patch #4890] Utility for iterating all requirements of an advance

2014-07-06 Thread pepeto
Update of patch #4890 (project freeciv): Status: Ready For Test => Done Open/Closed:Open => Closed ___ Reply to this item at:

[Freeciv-Dev] [bug #22252] Giving techonologies in editor does not work

2014-07-06 Thread pepeto
Update of bug #22252 (project freeciv): Status: Ready For Test => Fixed Open/Closed:Open => Closed ___ Reply to this item at:

[Freeciv-Dev] [patch #4888] Move research advance name accessors

2014-07-06 Thread pepeto
Update of patch #4888 (project freeciv): Status: Ready For Test => Done Open/Closed:Open => Closed ___ Reply to this item at:

[Freeciv-Dev] [bug #22258] "Map topology and tileset incompatible" is not an error

2014-07-06 Thread pepeto
Update of bug #22258 (project freeciv): Status: Ready For Test => Fixed Open/Closed:Open => Closed ___ Reply to this item at:

[Freeciv-Dev] [patch #4671] Move limits on what terrain a spy action can be done from to action enablers

2014-07-06 Thread pepeto
Follow-up Comment #8, patch #4671 (project freeciv): See also bug #22279. ___ Reply to this item at: ___ Message posté via/par Gna! http://gna.org/

[Freeciv-Dev] [bug #22279] Unit cannnot perform diplomat action from Ocean

2014-07-06 Thread pepeto
Follow-up Comment #1, bug #22279 (project freeciv): The unit flag "Marines" is currently reserved for attacks. This is a lack of control for diplomat action currently. But Sveinung is working on it. See patch #4671. ___ Reply to this item

[Freeciv-Dev] [bug #20618] Goto shouldn't attack in fogged tiles

2014-07-06 Thread pepeto
Update of bug #20618 (project freeciv): Assigned to:None => pepeto ___ Reply to this item at: ___ Message post

[Freeciv-Dev] [bug #22276] city_build_free_buildings() doesn't send great wonder info as intended

2014-07-06 Thread pepeto
Follow-up Comment #1, bug #22276 (project freeciv): I confirm this wasn't my intent for bug #15548, this was a typo. ___ Reply to this item at: ___ Message post