[Freeciv-Dev] Re: CMA 3.0

[Mike Kaufman <mkaufman@xxxxxxxxxxxxxx>]

On Thu, Dec 20, 2001 at 10:18:42AM +0100, Christian Knoke wrote:
> On Thu, Dec 20, 2001 at 09:37:36AM +0100, Raimar Falke wrote:
> > On Wed, Dec 19, 2001 at 10:41:55PM -0600, Mike Kaufman wrote:
> > 
> > > Please note that this patch required my previous 
> > > save_options patch (PR #1097) which saves the client's options using
> > > secfile_* (see how useful it is? :) It's in there so you don't have to 
> > > go find it to apply this one.
> 
> The client options are the presets, right?

yes.

> 
> > 
> > This should be solved by attributes. There should be an extra
> > attribute flag which is either: SAVE_ON_SERVER, SAVE_ON_CLIENT,
> > SAVE_ON_BOTH. Note that you also have to specify from which source you
> 
> Why should the presets be saved on the server?

I think Raimar's on crack here. He needs to look at the patch. The
_presets_ should not be on the server (just like I think that global
worklists should not be on the server, but that's for another thread)

> 
> > want to load attributes because this will lead to clashes
> > otherwise. The client attributes could be saved in a simple binary
> > file ".civclient.attributes". Yes this isn't nicely editable but the
> 
> bad

The actual agent attributes _still_ go to the server (if that was the
behavior before) I haven't changed anything there (except the name of
the attribute). Whether we want to change that behavior is a different
story.

btw, after only brief exposure, I think attributes kick a lot of ass.

> 
> > other option would be to have two interfaces: the attribute one and
> > another one for the textual client side.
> 
> I think its better to keep the agent's stuff on the client side
> as much as possible. I've learned that's it's necessary to have
> some common code, but presets are a user's matter, no?
> 
> Christian
> 

-mike

> -- 
> * Christian Knoke                           +49 4852 92248 *
> * D-25541 Brunsbuettel                  Wurtleutetweute 49 *
> * * * * * * * * *  Ceterum censeo Microsoft esse dividendum.