Vanilla List Mailing List Archive
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [VANILLA-LIST:2917] MAXPLAYER in config.h.in



On Sam, 26 Feb 2000 you wrote:
>I see that Carlos has removed MAXPLAYER from config.h.in, stating that
>it is redundant.  I do not believe that it is.
>
>If you look at the file, MAXPLAYER is under the "COW only defines".
>These are not defined unless building the client.  I don't know why
>COW and the server share the same config.h.in (Kurt has answered this
>question before, but I've forgotten the answer).

In earlier times (bronco source), the server as well as the client shared
all the importaint includes (like defs.h) to guaranty that client and server
has the same view of the universe.

After client and server development got seperated, individual changes
in the universe hasn't been in sync anymore, which gives us problems
like most clients nowadays display ATT ships with Galaxy bitmaps, whydead
messages are intepreted differently on client an server side, even clients
crashed, because of some new whydead messages not known by the client.

As for the config.h file. Originally I wrote the autoconfig stuff to be able
to detect all system specific parts in netrek on both client and server side.
Since both are very similar and for making maintainance easier it has been
written in a way to detect the code base and be able to be shared between
client and server. To keep all improovements on both side the config stuff
is shared.

As for the location for settings:

config.h.in is supposed to contain all system and site dependend config
settings. That are mainly the parts set with the -D define in the makefile
in the earlier days.

def.h contains the global settings of the netrek universe. Actually it really
should be shared by client and server. 

COW has MAXPLAYER in the site specific config.h file, since servers
usually had 20, but we supported more for convinience.

If the comunity decides MAXPLAYER 32 is a fixed unchangeable part
in the Netrek universe for client and servers it should go to defs.h

If it's a site specific thing config.h is the place.

In any case we really should try to merge client and server config files
again, to have an unique view of the Netrek universe again.

Having clients and server on the same CVS site might be a good time
for such an project.

Kurt (007)

-- 
Kurt Siegl / Franzberg 4, A-4483 Hargelsberg, Austria
Email: Kurt.Siegl@freemail.at  Tel (ISDN):   *(7225)7017
URL:   http://members.aon.at/presents/siegl/kurt/