[CF-Devel] Some points for discussion!

[crossfire-devel at archives.real-time.com]

Apologies for writing this so late at night.

Tinkering mainly with the client, I've got some discussion
items:

 * Late November last year, it was said that should commands
   like 'eat' for food or 'unwield' for weapons be made,
   they should go in the client.

   Unfortunately, the server has access to several different
   sorts of names (base name, short name, custom_name) for items,
   used in item_matched_string (arch.c line 212). 

   The client only gets one or two 'cooked' strings.

   Opinions?

   (It's worse for 'turn' for levers; levers don't get a client_type.
   Perhaps guess by face?)

 * A minor inconsistency in script_tell (client/common/script.c:851);
   most things the client sends to its scripts are terminated
   with just "\n", but script_tell ends with "\r\n".

   I was going to ask about sending function bodies to a
   running script.
   
   I suppose I could just send

     * 'function_start\n'
     * 'function_line %s\n'
     * 'function_end\n'

   as a group, and reassemble it at the script end; then it doesn't
   matter if the pipes mangle line-endings.
     
 * Installing from CVS, I had to copy the various bits and pieces
   in lib (bmaps, faces, animations, formulae...) by hand after
   doing make install. 

   Running autogen.sh, "it" complains about "invalid variables"
   in the doc/ and lib/ directories. (I guess I didn't notice
   the docs...). It might just be a version mismatch on my part 
   (see big appendix at the end).

"Invalid variables"
====================

  
     
     krudat at kevin
     
     :~/fiddling/crossfire/crossfire/crossfire$ automake 
  automake: both `configure.ac' and `configure.in' present: ignoring
  `configure.in'
  configure.ac: 8: required file `./[include/autoconf.h].in' not found
  configure.ac: 8: required file `./[include/stamp-h.in' not found
  Makefile.am:6: variable `CROSSEDIT' not defined
  doc/Makefile.am:29: invalid variable `dist_noinst_SCRIPTS'
  doc/Makefile.am:29: invalid variable `dist_noinst_SCRIPTS'
  doc/Makefile.am:28: invalid variable `dist_noinst_DATA'
  doc/playbook/Makefile.am:6: invalid variable `dist_noinst_SCRIPTS'
  doc/playbook/Makefile.am:6: invalid variable `dist_noinst_SCRIPTS'
  doc/scripts/Makefile.am:2: invalid variable `dist_noinst_SCRIPTS'
  doc/scripts/Makefile.am:2: invalid variable `dist_noinst_SCRIPTS'
  lib/Makefile.am:29: invalid variable `dist_noinst_SCRIPTS'
  lib/Makefile.am:77: invalid variable `dist_adm_SCRIPTS'
  lib/Makefile.am:29: invalid variable `dist_noinst_SCRIPTS'
  lib/Makefile.am:77: invalid variable `dist_adm_SCRIPTS'
  lib/Makefile.am:30: invalid variable `dist_pkgdata_DATA'
  lib/Makefile.am:55: invalid variable `dist_wizhelp_DATA'
  lib/Makefile.am:39: invalid variable `dist_help_DATA'
  utils/Makefile.am:2: invalid variable `nodist_bin_SCRIPTS'
  utils/Makefile.am:4: invalid variable `nodist_pkglib_SCRIPTS'
  utils/Makefile.am:6: invalid variable `dist_noinst_SCRIPTS'
  utils/Makefile.am:5: invalid variable `dist_pkglib_SCRIPTS'
  utils/Makefile.am:3: invalid variable `dist_bin_SCRIPTS'
  utils/Makefile.am:2: invalid variable `nodist_bin_SCRIPTS'
  utils/Makefile.am:4: invalid variable `nodist_pkglib_SCRIPTS'
  utils/Makefile.am:6: invalid variable `dist_noinst_SCRIPTS'
  utils/Makefile.am:5: invalid variable `dist_pkglib_SCRIPTS'
  utils/Makefile.am:3: invalid variable `dist_bin_SCRIPTS'
  
     
     krudat at kevin
     
     :~/fiddling/crossfire/crossfire/crossfire$ autoconf --version
  autoconf (GNU Autoconf) 2.59
  Written by David J. MacKenzie and Akim Demaille.

  Copyright (C) 2003 Free Software Foundation, Inc.
  This is free software; see the source for copying conditions.  There is NO
  warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
  
     
     krudat at kevin
     
     :~/fiddling/crossfire/crossfire/crossfire$ automake --version
  automake (GNU automake) 1.4-p6

  Copyright (C) 1999, 2001 Free Software Foundation, Inc.
  This is free software; see the source for copying conditions.  There is NO
  warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

  Written by Tom Tromey <
     
     tromey at redhat.com
     
     >

Running on Debian testing. The server otherwise runs fine.

-- 
  Thanks,

 -- Kevin "Third Time's an Evacuation" Rudat <
     
     krudat at origin.net.au
     
     >

Has been bitten by semantics before.   Sci-Fi/Fantasy ... type.
Pretends to know a bit of programming. Very bad manners.
Terrible posture and co-ordination.    A forgetful ... wha?

<Kaa-Hahhnn> Humans are an example of a species rushed to life without
          proper testing... they have LOTS of bugs to fix. ;>

_______________________________________________
crossfire-devel mailing list
     
     crossfire-devel at lists.real-time.com
     
     
     https://mailman.real-time.com/mailman/listinfo/crossfire-devel