Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 12 Sep 2021 18:32:46 GMT
From:      Warner Losh <imp@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-branches@FreeBSD.org
Subject:   git: b455c74d83e3 - stable/12 - mk: WITH_FOO=no now generates a warning
Message-ID:  <202109121832.18CIWkh4033980@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
The branch stable/12 has been updated by imp:

URL: https://cgit.FreeBSD.org/src/commit/?id=b455c74d83e3274c69adbf2e740b49ff875ac0e9

commit b455c74d83e3274c69adbf2e740b49ff875ac0e9
Author:     Warner Losh <imp@FreeBSD.org>
AuthorDate: 2021-06-10 00:10:12 +0000
Commit:     Warner Losh <imp@FreeBSD.org>
CommitDate: 2021-09-12 16:33:14 +0000

    mk: WITH_FOO=no now generates a warning
    
    Many people are used to gnu configure's behavior of changing
    --with-foo=no to --without-foo. At the same time, several folks have
    WITH_FOO=no in their config files to enable this ironic form of the
    option because of an old meme from IRC, a mailing list or the forums (I
    forget which). Add a warning to allow to alert people w/o breaking POLA.
    
    Reviewed by:            allanjude, bdrewery, manu
    MFC After:              2 weeks
    Sponsored by:           Netflix
    Differential Revision:  https://reviews.freebsd.org/D30684
    
    (cherry picked from commit f4d987cd137cb2d0d54a3e35d9258ca7c175d291)
---
 share/mk/bsd.mkopt.mk | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/share/mk/bsd.mkopt.mk b/share/mk/bsd.mkopt.mk
index 5a9cf1b2f1be..98d23dd46c2a 100644
--- a/share/mk/bsd.mkopt.mk
+++ b/share/mk/bsd.mkopt.mk
@@ -36,6 +36,9 @@
 #
 .for var in ${__DEFAULT_YES_OPTIONS}
 .if !defined(MK_${var})
+.if defined(WITH_${var}) && ${WITH_${var}} == "no"
+.warning "Use WITHOUT_${var}=1 insetad of WITH_${var}=no"
+.endif
 .if defined(WITHOUT_${var})			# WITHOUT always wins
 MK_${var}:=	no
 .else
@@ -54,6 +57,9 @@ MK_${var}:=	yes
 #
 .for var in ${__DEFAULT_NO_OPTIONS}
 .if !defined(MK_${var})
+.if defined(WITH_${var}) && ${WITH_${var}} == "no"
+.warning "Use WITHOUT_${var}=1 insetad of WITH_${var}=no"
+.endif
 .if defined(WITH_${var}) && !defined(WITHOUT_${var}) # WITHOUT always wins
 MK_${var}:=	yes
 .else



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202109121832.18CIWkh4033980>