Complete.Org: Mailing Lists: Archives: freeciv-dev: September 2001:
[Freeciv-Dev] Re: [Patch] Multiple bonus tech patch
Home

[Freeciv-Dev] Re: [Patch] Multiple bonus tech patch

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
Cc: freeciv-dev@xxxxxxxxxxx
Subject: [Freeciv-Dev] Re: [Patch] Multiple bonus tech patch
From: Josh Cogliati <jjc@xxxxxxxxxxxxxxxxxxxxxxxxx>
Date: Sat, 1 Sep 2001 19:44:33 -0600
Reply-to: jjc@xxxxxxxxxxxxxxxxxx

On Sat, Sep 01, 2001 at 05:41:13PM +0200, Raimar Falke wrote:
> On Sat, Sep 01, 2001 at 09:49:53AM -0600, Josh Cogliati wrote:
> > On Sat, Sep 01, 2001 at 10:40:35AM +0200, Raimar Falke wrote:
> > > On Fri, Aug 31, 2001 at 08:16:40PM -0600, Josh Cogliati wrote:
> > > > This patch is based off of 1.12.0 and allows multiple bonus techs.
> > > > It adds Computers as a bonus tech in addition to Philosopy.  I think I 
> > > > got all the comments saying their can only be one bonus tech.
<snip>
> > Would something like:
> > "Great scientists from all the world join your civilization; 
> > you get an immediate advance." be good enough, or should a 
> > first_build_message be added to the techs ruleset?
> 
> I think this is good enough.
> 
> > Oh, as a warning this patch changes the network protocol since it
> > removes the sending of the bonus_tech (instead the flags are used). 
> 
> I meant this with capabilities. Just do a search in common/packets.c
> for has_capability().
Added capabilities.
I think the patch should continue to work with older freeciv client
or older servers (so far as I can tell the client never even used the
value anyway).
Changed the message to "Great scientists ...

Here the diff from freeciv 1.12.0.  Check over the network code.


-- 
Josh Cogliati
jjc@xxxxxxxxx
This message created in Linux, the choice of a GNU generation.

Attachment: freeciv_new_bonus.diff
Description: Text document


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