Relay-Version: version B 2.10 5/3/83; site utzoo.UUCP
Path: utzoo!mnetor!uunet!seismo!rutgers!labrea!aurora!ames!oliveb!sun!gorodish!guy
From: guy%gorodish@Sun.COM (Guy Harris)
Newsgroups: comp.sources.d
Subject: Re: Problems with Nethack Patch #1 on Sys V
Message-ID: <26313@sun.uucp>
Date: Sat, 22-Aug-87 22:36:39 EDT
Article-I.D.: sun.26313
Posted: Sat Aug 22 22:36:39 1987
Date-Received: Sun, 23-Aug-87 21:40:31 EDT
References: <270@kosman.UUCP>
Sender: news@sun.uucp
Lines: 22
Keywords: Grumble grumble....

> I got several errors, as you can see from the output of the process included 
> below.
> 
> I am running Sys V, with 14-byte names.  Patch adapts to this, sh scripts do
> not.  Thus the first error.  The others I don't know about.

Well, two errors, anyway; I presume you meant "the other", not "the others".

The second error is caused by the "here documents" containing the "ed" scripts
not being protected by the end-of-document string being enclosed in quotes.  If
a "here document" isn't so protected, the shell does expansion of variable
names and command-substitution sequences in the document.  This causes nasty
problems if the "here document" contains e.g. Makefile text, since "make" and
the Bourne shell use the same basic syntax for variable references.  It also
causes the shell to get upset about certain syntax errors.

The person who posted the patches posted an update explaining this; the update
also included a new version of "invent.c", since the patch did NOT apply to
the distributed version.
	Guy Harris
	{ihnp4, decvax, seismo, decwrl, ...}!sun!guy
	guy@sun., an