Re: a few OptionalObsoleteFiles.inc improvements

From: b. f. <bf1783_at_googlemail.com>
Date: Tue, 14 Dec 2010 04:47:39 +0000
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
>_at__at_ -250,7 +250,6 _at__at_ SUBDIR+=    ftp-proxy
> SUBDIR+=       pkg_install
> .endif
>
>-# XXX MK_TOOLCHAIN?
> .if ${MK_PMC} != "no"
> SUBDIR+=       pmcannotate
> SUBDIR+=       pmccontrol
>_at__at_ -283,7 +282,9 _at__at_ 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.
Received on Tue Dec 14 2010 - 04:12:37 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:40:10 UTC