Date: Thu, 21 Nov 2019 16:23:11 +0000 From: Glen Barber <gjb@freebsd.org> To: Brooks Davis <brooks@freebsd.org> Cc: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org, FreeBSD Ports Management Team <portmgr@freebsd.org>, Colin Percival <cperciva@freebsd.org> Subject: Re: svn commit: r354909 - in head: . share/mk Message-ID: <20191121162311.GN5599@FreeBSD.org> In-Reply-To: <20191121161337.GA92849@FreeBSD.org> References: <201911201836.xAKIawWt053874@repo.freebsd.org> <20191121161337.GA92849@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
--mzDsWmoYTs+XRn5Z Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Thu, Nov 21, 2019 at 04:13:37PM +0000, Glen Barber wrote: > On Wed, Nov 20, 2019 at 06:36:58PM +0000, Brooks Davis wrote: > > Author: brooks > > Date: Wed Nov 20 18:36:58 2019 > > New Revision: 354909 > > URL: https://svnweb.freebsd.org/changeset/base/354909 > >=20 > > Log: > > Make the warning for deprecated NO_ variables an error. > > =20 > > Support for NO_CTF, NO_DEBUG_FILES, NO_INSTALLLIB, NO_MAN, NO_PROFILE, > > and NO_WARNS as deprecated in 2014 with a warning added for each one > > found. Turn these into error in preperation for removal of compatabil= ity > > support before FreeBSD 13. > > =20 > > Reviewed by: imp > > Relnotes: yes > > Sponsored by: DARPA, AFRL > > Differential Revision: https://reviews.freebsd.org/D22448 > >=20 >=20 > > Modified: head/share/mk/bsd.opts.mk > > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D > > --- head/share/mk/bsd.opts.mk Wed Nov 20 18:12:01 2019 (r354908) > > +++ head/share/mk/bsd.opts.mk Wed Nov 20 18:36:58 2019 (r354909) > > @@ -100,7 +100,7 @@ __DEFAULT_DEPENDENT_OPTIONS =3D \ > > PROFILE \ > > WARNS > > .if defined(NO_${var}) > > -.warning "NO_${var} is defined, but deprecated. Please use MK_${var}= =3Dno instead." > > +.error "NO_${var} is defined, but deprecated. Please use MK_${var}=3Dn= o instead." > > MK_${var}:=3Dno > > .endif > > .endfor > >=20 >=20 > Had this been exp-run tested? This breaks at least the > net/bsdec2-image-upload port, which does *not* set NO_MAN explicitly in > its Makefile. I suspect at least several other ports are now implicitly > broken as result of this change. >=20 In fact, on a quick search through ports/*/*/Makefile, I do not see NO_MAN defined *anywhere*. So, this is pulled in from somewhere else, which now I have a feeling more ports than I initially thought are now broken. Glen --mzDsWmoYTs+XRn5Z Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAEBCAAdFiEEjRJAPC5sqwhs9k2jAxRYpUeP4pMFAl3Wue8ACgkQAxRYpUeP 4pPRGg//bDDnF4LD6gYlF0cnkgFt+WB3i9gOvZff3dghn4mb1AXyF3GD3BA2W+RY ai5m+bX33CVXQ5tvq/Okk8b+tafGZSBLRidghn8qwa7BQ1IesGbv4Sd1aTvn7lb0 mmKWRfgeI+EJ7Fo6iexUGiz+PpQZxU5B/ROUIvOgSgzGtu3RQMMzEwX4PVUHQW8I 3OjaagyXVDB57iDDT1b/Jje52LfKkIHhaI7D708lhDQRAjso807uTBjeJN/zMWZ1 3kuhegS0VrMBGAm5XAk+M2r6DikhuKk+f3Jof6/1BtFLFhM8Gt11GvOkswSi771T wpRqgARYEb/ad/B1OqxnD99On69xwe5GNboVWQq5Cwx0PpyFJaS1UyJGfHBWUAQo I/DTjAzfn6mfkrWecZFazVCepIU5UqkyZ98CVea35z1nRfwBv6VrFz/8d1w9ds9Z 56bi6WF7qvmyZD1yd3BrU3a/FYFWC//0mf49LuG6szkYwaQT3t/kfwa3oQbkR9AG 3qJ0eAHlkH03kVBF/qmic3z2tUjrjGDQbCeGz2o1X6Z4hbbF98ZV5juMGiZ2owC/ eqQzNKgRIUpxI4qNdOfEQ+yS9t/WJzguKHp4B5SASMEamomSma/OUh7xon+3Xwx4 gCEew0Il7dGgEKc5+Ny1iD8iQRZUsVAAwK0MhDaoBbEmC3IJLGg= =iJyz -----END PGP SIGNATURE----- --mzDsWmoYTs+XRn5Z--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20191121162311.GN5599>