patchFreeciv - Patches: patch #4754, Experimental/Multiplayer have...

 
 
Show feedback again

patch #4754: Experimental/Multiplayer have their own ai_effects.ruleset

Submitted by:  Marko Lindqvist <cazfi>
Submitted on:  Sat 31 May 2014 10:03:03 PM UTC  
 
Category: rulesetsPriority: 5 - Normal
Status: NonePrivacy: Public
Assigned to: NoneOpen/Closed: Open
Planned Release: 

Add a New Comment (Rich MarkupRich Markup):
   

You are not logged in

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

 

Sun 07 Dec 2014 11:34:35 PM UTC, comment #2:

> in fact I'd already half-prepared a patch doing so


I assumed this to mean that you'd take this ticket...

Marko Lindqvist <cazfi>
Project Administrator
Sat 31 May 2014 10:06:04 PM UTC, comment #1:

I'm for removing them (in fact I'd already half-prepared a patch doing so).
The hassle of reintroducing them is IMO fairly minimal (at least in a new major version).

Jacob Nevins <jtn>
Project Administrator
Sat 31 May 2014 10:03:03 PM UTC, original submission:

As noted by jtn in bug #22094, experimental and multiplayer rulesets have ai_effects.rulesets of their own, but identical to default one.

It would be cleaner to have them removed and to include default/ai_effects.ruleset instead.
But they could very well diverge in the future (one would assume that different rulesets need different ai effects). Keeping them around would avoid the extra hassle of reintroducing them when they need to diverge.

I would slightly prefer removing them. As we don't know if, and when, they will be needed, there's no point in maintaining them meanwhile (and we have already had bugs because of we've missed one of them when changing others).

Marko Lindqvist <cazfi>
Project Administrator

 

(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

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by jtn (Posted a comment)
  • -unavailable- added by cazfi (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.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    No Changes Have Been Made to This Item
    Show feedback again

    Back to the top


    Powered by Savane 3.1-cleanup