Complete.Org: Mailing Lists: Archives: freeciv-dev: April 2003:
[Freeciv-Dev] Re: (PR#4044) Revised multiple veteran patch
Home

[Freeciv-Dev] Re: (PR#4044) Revised multiple veteran patch

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: undisclosed-recipients:;
Subject: [Freeciv-Dev] Re: (PR#4044) Revised multiple veteran patch
From: "James" <james.blewitt@xxxxxxxxxxxxxx>
Date: Fri, 25 Apr 2003 09:25:38 -0700
Reply-to: rt@xxxxxxxxxxxxxx

> No, don't do that! Sure, you can't add new unit classes in Freeciv CVS
> yet (this is still waiting... and waiting... for inclusion, but it's in
> FreecivAC, http://freecivac.sf.net/, already) but city improvement effects
> don't do anything at all in Freeciv CVS anyway, so adding new effects won't
> help you. FreecivAC also does (2) already.
> 
> I say again, don't bother trying to reimplement improvement effects just
> for veteran levels. It's non-trivial code (if it weren't, the FreecivAC
> code to do it would've gone into Freeciv 2 years ago when I wrote it...)
> but it has been written already. I suggest you concentrate on getting
> the other stuff sorted out (e.g. the graphics) so that the code can get
> in to CVS; improvement effects will get there too eventually...
> 
>       Ben

Thats a weight off my mind :-)  I'll just leave out veteran caravan support 
completely.

When do you think that the integration of your code into the Freeciv CVS will 
take place?

James



[Prev in Thread] Current Thread [Next in Thread]