From owner-cvs-usrbin  Sat Aug 26 15:05:06 1995
Return-Path: cvs-usrbin-owner
Received: (from majordom@localhost)
          by freefall.FreeBSD.org (8.6.11/8.6.6) id PAA04128
          for cvs-usrbin-outgoing; Sat, 26 Aug 1995 15:05:06 -0700
Received: from gndrsh.aac.dev.com (gndrsh.aac.dev.com [198.145.92.241])
          by freefall.FreeBSD.org (8.6.11/8.6.6) with ESMTP id PAA04111
          ; Sat, 26 Aug 1995 15:04:57 -0700
Received: (from rgrimes@localhost) by gndrsh.aac.dev.com (8.6.11/8.6.9) id PAA06282; Sat, 26 Aug 1995 15:04:45 -0700
From: "Rodney W. Grimes" <rgrimes@gndrsh.aac.dev.com>
Message-Id: <199508262204.PAA06282@gndrsh.aac.dev.com>
Subject: Re: cvs commit: src/usr.bin/xinstall xinstall.c
To: davidg@freefall.FreeBSD.org (David Greenman)
Date: Sat, 26 Aug 1995 15:04:45 -0700 (PDT)
Cc: CVS-commiters@freefall.FreeBSD.org, cvs-usrbin@freefall.FreeBSD.org
In-Reply-To: <199508261035.DAA08737@freefall.FreeBSD.org> from "David Greenman" at Aug 26, 95 03:35:46 am
X-Mailer: ELM [version 2.4 PL24]
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Content-Length: 652       
Sender: cvs-usrbin-owner@FreeBSD.org
Precedence: bulk

> 
> davidg      95/08/26 03:35:45
> 
>   Branch:      usr.bin/xinstall  RELENG_2_1_0
>   Modified:    usr.bin/xinstall  xinstall.c
>   Log:
>   Brought in change from rev 1.2: don't unlink target on failure of strip.

Both Bruce and I objected strongly to this hack, it is a hack, it should be
backed out of both branches and redone correctly.

It was added to fix the abuse of install -s to install non binary files.
Installs specifically says in its man page it is for installing binary
files.

-- 
Rod Grimes                                      rgrimes@gndrsh.aac.dev.com
Accurate Automation Company                 Reliable computers for FreeBSD