Path: utzoo!attcan!uunet!lll-winken!lll-crg.llnl.gov!casey
From: casey@lll-crg.llnl.gov (Casey Leedom)
Newsgroups: comp.windows.x
Subject: Re: Small bug in util/imake.includes/Imake.rules
Message-ID: <14527@lll-winken.llnl.gov>
Date: 10 Dec 88 02:28:06 GMT
References: <14520@lll-winken.llnl.gov> <14522@lll-winken.llnl.gov>
Sender: usenet@lll-winken.llnl.gov
Reply-To: casey@lll-crg.llnl.gov.UUCP (Casey Leedom)
Organization: Lawrence Livermore National Laboratory
Lines: 34

| From: jim@EXPO.LCS.MIT.EDU (Jim Fulton)
| 
| Don't forget that you'll probably also need to fix up the Imakefiles for
| xterm and xload (and any other setuid programs) that bash the
| INSTALLFLAGS.  Thats why I didn't do it for R3.

  Ack!!!  Jim is of course 100% right.  I had no idea there were so many
Imakefiles that munged INSTALLFLAGS.  That, together with the
imake.includes/README file which explicitly states that that's what
INSTALLFLAGS is for (YACORTFM (Yet Another Case Of RTFM)).

  I hereby hastily withdraw my fixes for util/imake.includes/Imake.rules
and clients/xload/Imakefile.  Instead I offer the following as a possible
suggestion (I'm trying it right now to see if it will do what I want):
make the default INSTALLFLAGS = INSTBINFLAGS.  This is accomplished with
the following simple change:

*** util/imake.includes/Imake.tmpl-dist	Sun Oct 23 19:37:17 1988
--- util/imake.includes/Imake.tmpl	Fri Dec  9 18:16:28 1988
***************
*** 403,408 ****
--- 403,409 ----
        LIBMANDIR = LibmanDir		/* man pages for library routines */
      XAPPLOADDIR = XAppLoadDir		/* application default resources */
  
+    INSTALLFLAGS = InstBinFlags		/* default is to install a binary */
     INSTBINFLAGS = InstBinFlags		/* install flags for programs */
     INSTUIDFLAGS = InstUidFlags		/* install flags for setuid programs */
     INSTLIBFLAGS = InstLibFlags		/* install flags for libraries */

  Any Imakefile that want's to install anything other than a binary
should already be munging INSTALLFLAGS, so this should get us into
trouble.  You'll of course see a retraction from me if I'm wrong ... :-(

Casey