Complete.Org: Mailing Lists: Archives: freeciv-dev: June 2006:
[Freeciv-Dev] Re: (PR#18113) caravan with no homecity crashes server
Home

[Freeciv-Dev] Re: (PR#18113) caravan with no homecity crashes server

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: evyscr@xxxxxxxxx
Subject: [Freeciv-Dev] Re: (PR#18113) caravan with no homecity crashes server
From: "Jason Dorje Short" <jdorje@xxxxxxxxx>
Date: Mon, 26 Jun 2006 17:24:04 -0700
Reply-to: bugs@xxxxxxxxxxx

<URL: http://bugs.freeciv.org/Ticket/Display.html?id=18113 >

Egor Vyscrebentsov wrote:
> <URL: http://bugs.freeciv.org/Ticket/Display.html?id=18113 >
> 
> Good daytime!
> 
> If one load a .sav file which contain caravan with no homecity, server
> will crash next turn. Problem is that aicore caravan code assume that
> caravan always has homecity, and server does not check for this when
> load a game. Also, it seems to me that editor allows to create caravan
> without homecity.
> I'm not sure who own this bug: server, editor, or both.
> 
> There are several ways to fix this behavior, so I want to listen which
> is better [before try to make a patch]:
> 1. Disallow caravans with no homecity, reject a game if it has them.
> 2. Allow caravan with no homecity to change it once. Until changing
> caravan could not provide caravan bonuses/help building wonders.
> 3. Something else? (Removing caravans totally is not a way for me.)
> 
> [Chosing second variant needs to answer several additional questions in
> AI part.]
> 
> Bug was discovered while dancing with manually converted civ2 scenario.

I think the easiest way is either of

1.  Allow homeless caravans to change homecity (homed caravans cannot).
2.  Don't allow homeless caravans to create trade route/enter 
marketplace at all.

Both should be pretty easy patches.  Either is okay with me really.

-jason





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