[Freeciv-Dev] Re: Status of submitted patches

[Raahul Kumar <raahul_da_man@xxxxxxxxx>]

--- Per I Mathisen <per@xxxxxxxxxxx> wrote:
> On Thu, 23 May 2002, Davide Pagnin wrote:
> > 1) http://arch.freeciv.org/freeciv-dev-200205/msg00233.html
> >
> > (There is an important bugfix for the actual angrycitizen
> > implementation, and the patch for the correct civ II behaviour
> > for the specialists)
> >
> > Ross, stated that this patch may raise problem for the AI
> > that has to reallocate theirs citizen, but he also said:
> > "These are not showstopper issues, so they shouldn't hold
> > up this patch. But they will do undesireable things on
> > occasion until fixed."
> >
> > So, There is not Strong reject on the patch, or I'm getting
> > something wrong?
> 
> If Raahul could look at the things Ross mentioned, test his changes
> against this, and tell me that this isn't a problem, I will commit his
> patch.

Actually, there are very undesirable effects. The AI really relies on Elvises
making happiness go away. I haven't fixed this, so for now no commit.

> > 2) http://arch.freeciv.org/freeciv-dev-200205/msg00201.html
> >
> > (The luxury cap problem, at the moment there is no consensus
> > to put a luxury cap into freeciv, unless it is a configurable
> > option, thus I urge mainteners to say what has to be done,
> > and how, and more important, if this effort can be introduced
> > into 1.13.0 (aka bugfix) or not (aka new features).
> 
> This is a new feature, and it should be controlled by a ruleset option. It
> won't get in 1.13.0 release. Sorry.

Thanks for reminding me. If I produce a version with a ruleset option(turned
off), would that be ok?


Aloha,
RK.

Asynchronous inputs are at the root of our race problems.
                -- D. Winker and F. Prosser

__________________________________________________
Do You Yahoo!?
LAUNCH - Your Yahoo! Music Experience
http://launch.yahoo.com