lynx-dev Lynx 2.8.2dev 20 for vms - bug fixes

[Andy Harper]

There are a few problems in lynx2.8.2dev20 (17 mar 1999) which prevent a
succesfull build on OpenVMS, using OpenVMS 6.2, DECC and selecting the
socketshr network interface. I've found and fixed all those that seem to
matter; here's a summary:


[]BUILD.COM
[]DESCRIP.MMS
[.WWW.LIBRARY.IMPLEMENTATION]LIBMAKE.COM
Each of these define the symbol DEBUG for compilation. This should be removed
as it is now defined in one of the header files, causing a series of definition
conflicts in every module.


[.WWW.LIBRARY.IMPLEMENTATION]HTAAPROT.C:
Compilation fails because the declarations etc for getpwuid and related
routines aren't properly found under VMS.  The symbol NOUSERS needs to be
defined and added to the VMS section of:
  USERDEFS.H


[.WWW.LIBRARY.IMPLEMENTATION]HTTCP.C:
There seems to be an extra assignment that's not needed, in which both sides
are undeclared variables hence causing compilation failure. This line should be
removed:
  666           h_errno = NO_RECOVERY;



[.SRC]LYCURSES.C:
The very long multi-line string in gettext at line 1333 requires a closing
double quote at the end of each continuation line, jsut prior to the
terminating backslash. So rather than:
    ..... text\n\

they should be
    ..... text\n"\

As far as I can tell, the first is not valid ANSI C but the corrected version
is.



[.SRC]LYUTILS.c
If the symbol DISP_PARTIAL is not defined, then there are several compilation
errors, including the fact that 'keymap' is undefined because the include file
is not included in this case.  A define for DISP_PARTIAL needs to be added to
   BUILD.COM
   DESCRIP.MMS

The symbol COPY_PATH is not defined for VMS and this needs to be added to the
VMS section in:
   USERDEFS.H


[.SRC]LYMAIN.C:
The variable lynx_version_putenv_command is not defined in VMS, so it needs
enclosing in an ifdef:

************
File DISK$COMPCENT:[UDAA055.TEMP.LYNX2-8-2.SRC]LYMAIN.C;2
  523   #ifndef VMS
  524       FREE(lynx_version_putenv_command);
  525   #endif
  526       FREE(lynx_temp_space);
******
File DISK$COMPCENT:[UDAA055.TEMP.LYNX2-8-2.SRC]LYMAIN.C;1
  523       FREE(lynx_version_putenv_command);
  524       FREE(lynx_temp_space);
************



[.SRC]LYMAIL.C
An extra bracket in a printf needs removing:

************
File DISK$COMPCENT:[UDAA055.TEMP.LYNX2-8-2.SRC]LYMAIL.C;2
  555       printf("%s\n\n$ %s\n\n%s", SENDING_FORM_CONTENT, command,
PLEASE_WAI
  556       LYSystem(command);
******
File DISK$COMPCENT:[UDAA055.TEMP.LYNX2-8-2.SRC]LYMAIL.C;1
  555       printf("%s\n\n$ %s\n\n%s"), SENDING_FORM_CONTENT, command,
PLEASE_WA
  556       LYSystem(command);
************



With these changes, it builds and runs on both a VAX 4100 and an alpha 2100
running Open VMS 6.2 with Socketshr. There a few compilation warnings to do
with type mismatches but nothing that affects operation. They should be removed
though as they make MMS and MMK bail out on the affected modules.

I hope VMS lynx developers find these changes helpful and will include them in
the next development release.


Regards,

Andy Harper
Kings College London