Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 14 Dec 2010 04:47:39 +0000
From:      "b. f." <bf1783@googlemail.com>
To:        freebsd-current@FreeBSD.org
Cc:        Alexander Best <arundel@freebsd.org>
Subject:   Re: a few OptionalObsoleteFiles.inc improvements
Message-ID:  <AANLkTin%2Bqr7Y-vMTbKg9ZKE6_zE%2BNVYFEMX%2BPwy4vbym@mail.gmail.com>

next in thread | raw e-mail | index | archive | help
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.

b.



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?AANLkTin%2Bqr7Y-vMTbKg9ZKE6_zE%2BNVYFEMX%2BPwy4vbym>