From owner-freebsd-current@FreeBSD.ORG  Fri Dec 17 14:32:22 2010
Return-Path: <owner-freebsd-current@FreeBSD.ORG>
Delivered-To: freebsd-current@FreeBSD.org
Received: by hub.freebsd.org (Postfix, from userid 1233)
	id 3C4B21065670; Fri, 17 Dec 2010 14:32:22 +0000 (UTC)
Date: Fri, 17 Dec 2010 14:32:22 +0000
From: Alexander Best <arundel@freebsd.org>
To: bf1783@gmail.com
Message-ID: <20101217143222.GA28086@freebsd.org>
References: <AANLkTin+qr7Y-vMTbKg9ZKE6_zE+NVYFEMX+Pwy4vbym@mail.gmail.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <AANLkTin+qr7Y-vMTbKg9ZKE6_zE+NVYFEMX+Pwy4vbym@mail.gmail.com>
Cc: freebsd-current@FreeBSD.org
Subject: Re: a few OptionalObsoleteFiles.inc improvements
X-BeenThere: freebsd-current@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: Discussions about the use of FreeBSD-current
	<freebsd-current.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-current>, 
	<mailto:freebsd-current-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-current>
List-Post: <mailto:freebsd-current@freebsd.org>
List-Help: <mailto:freebsd-current-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-current>,
	<mailto:freebsd-current-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Fri, 17 Dec 2010 14:32:22 -0000

On Tue Dec 14 10, b. f. wrote:
> Alexander Best wrote:
> 
> >any thoughts on this patch? it adds files which will be removed when
> >WITHOUT_SYSCONS is set. also it makes sure sysinstall(8) and sade(8) only get
> >installed when WITHOUT_SYSINSTALL wasn't defined and also that any related
> >executables and manual pages get removed if in fact that var is defined.
> 
> ...
> 
> >diff --git a/usr.sbin/Makefile b/usr.sbin/Makefile
> >index f3e853e..2151868 100644
> >--- a/usr.sbin/Makefile
> >+++ b/usr.sbin/Makefile
> >@@ -250,7 +250,6 @@ SUBDIR+=    ftp-proxy
> > SUBDIR+=       pkg_install
> > .endif
> >
> >-# XXX MK_TOOLCHAIN?
> > .if ${MK_PMC} != "no"
> > SUBDIR+=       pmcannotate
> > SUBDIR+=       pmccontrol
> >@@ -283,7 +282,9 @@ SUBDIR+=    praliases
> > SUBDIR+=       sendmail
> > .endif
>  >
> >+.if ${MK_SYSINSTALL} != "no"
> > SUBDIR+=       sysinstall
> >+.endif
> 
> I'm glad to see that you're filling in some of the many missing bits
> in this file.
> 
> The last part of your patch reverts a change that Warner Losh made in
> r212525 as part of his tbemd project merge.  It's possible that this
> change may have been an unintended, but it followed a discussion in
> which Warner rejected a related patch proposed by Garrett Cooper,
> partly because sysinstall is included in build-tools in Makefile.inc1,
> even though some thought that it should not be.  In any event, you
> should probably look into that before committing the last part of your
> patch.

so is csh, but still you can set WITHOUT_TCSH=true and have a world without
(t)csh.

no need to worry i'll commit any changes, since i don't have commit rights. ;)

cheers.
alex

> 
> b.

-- 
a13x