bugFreeciv - Bugs: bug #24546, ruleset table syntax

 
 
Show feedback again

bug #24546: ruleset table syntax

Submitted by:  Frank <dunnoob>
Submitted on:  Thu Mar 31 06:43:07 2016  
 
Category: generalSeverity: 3 - Normal
Priority: 5 - NormalStatus: None
Assigned to: NoneOpen/Closed: Open
Release: 2.6Operating System: None
Planned Release: 3.0.0Contains string changes: None

Add a New Comment (Rich MarkupRich Markup):
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Tue Apr 12 08:52:12 2016, comment #3:

Yes, eventually I arrived at "just delete the stupid lines". Related, values of type list as in foo = x, y, z apparently have no empty list value, only ; foo = (comment out or delete line) works.

For some graphics in 2.6 "none" is not the same as "-", and commenting out or deleting those boring "-" lines triggers a hard "cannot load ruleset" error. Maybe "-" should be the default for this.

Frank <dunnoob>
Sat Apr 2 08:57:43 2016, comment #2:

The freeciv problem to fix here is that at least one value is expected when table name has been seen.
Anyway, this is low-level datafile format change and S2_6 formats have been frozen -> setting target to 3.0.0

Marko Lindqvist <cazfi>
Project Administrator
Sat Apr 2 08:55:03 2016, comment #1:

If you want to make completely empty table visible in human-readable way, make its header comment:

; obsolete_by = { "type", "name", "range" }

Marko Lindqvist <cazfi>
Project Administrator
Thu Mar 31 06:43:07 2016, original submission:

In a buildings.ruleset for 2.5 I had lots of...

obsolete_by = "None"

...lines. For 2.6 I changed these lines to...

obsolete_by = { "type", "name", "range" } ; same as 2.5 "None"

...but it didn't work as expected (trimmed error message):

Expected end of line file ".../.freeciv/test/dunnoob/buildings.ruleset", line 84, pos 43
looking at: ' } ; same as 2.5 "None"'

Apparently the '}' cannot be on the same line as the '{'. A case of https://en.wikipedia.org/wiki/Principle_of_least_astonishment - is it a bug, an undocumented feature, or only me? I'll try to find another solution for "None" in one line, two lines certainly work. :-P

Frank <dunnoob>

 

(Note: upload size limit is set to 1024 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Digest:
   task dependencies.

 

Carbon-Copy List
  • -unavailable- added by cazfi (Posted a comment)
  • -unavailable- added by dunnoob (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Error: not logged in

     

     

    Follow 2 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Sat Apr 2 08:57:43 2016cazfiCategoryrulesets=>general
      Planned Release=>3.0.0
    Show feedback again

    Back to the top


    Powered by Savane 3.1-cleanup