Path: utzoo!utgpu!news-server.csri.toronto.edu!helios.physics.utoronto.ca!ists!yunexus!hydroesm!jtsv16!torsqnt!tmsoft!eci386!woods
From: woods@eci386.uucp (Greg A. Woods)
Newsgroups: comp.sources.d
Subject: Re: mbase is missing something - ie remove function
Message-ID: <1991Jan8.172820.10036@eci386.uucp>
Date: 8 Jan 91 17:28:20 GMT
References: <7389@plains.NoDak.edu> <49906@prls.UUCP>
Reply-To: woods@eci386.UUCP (Greg A. Woods)
Organization: Elegant Communications, Inc.
Lines: 32

In article <49906@prls.UUCP> gordon@prls.UUCP (Gordon Vickers) writes:
>       I also made the following "corrections" needed (at least on) Ultrix :
>         o  File: Makefile
>                added compiler flag -DUNIX
>                 purpose: build.c requires UNIX to be defined is you are
>                           running Unix. The -DUNIX flag for the compiler
>                           will do this.  I could have put a #define UNIX
>                           into build.c but it really belongs in Makefile
>                           should it be needed later by other files that
>                           are patched or enhanced.

This isn't necessarily directed at Gordon, since he's only working
around a "bug"....

I've *NEVER* seen anything resembling UNIX that doesn't implicitly
"-dunix" in the preprocessor and I've even seen some things that
aren't really UNIX that do so too.

If you want to use "#ifdef UNIX" instead of "#ifdef unix", you can always:

	#ifdef unix
	# define UNIX	1
	#endif

in some global header file.

IMHO, The fix does NOT belong in the Makefile.
-- 
							Greg A. Woods
woods@{eci386,gate,robohack,ontmoh,tmsoft}.UUCP		ECI and UniForum Canada
+1-416-443-1734 [h]  +1-416-595-5425 [w]  VE3TCP	Toronto, Ontario CANADA
Political speech and writing are largely the defense of the indefensible-ORWELL