People are misusing everything, and of course, it breaks. Try to leverage that.
About half a dozen PHP ports set PHP_DEFAULT for a good reason, so something may need to happen in those.
Differential D8060
Add a warning for people who set foo_DEFAULT in their make.conf or environment. mat on Sep 28 2016, 2:03 PM. Authored by Tags None Referenced Files
Details
People are misusing everything, and of course, it breaks. Try to leverage that. About half a dozen PHP ports set PHP_DEFAULT for a good reason, so something may need to happen in those.
Diff Detail
Event TimelineComment Actions question: how are you going to implement the "removal" ? i.e. if SSL_DEFAULT=libressl is set in /etc/make.conf, are you going to force an error if SSL_DEFAULT is defined before it should be? Comment Actions If you intend to ".undef" if, you need to specify that action exactly in the warning message. Or perhaps emit a notice that the definition was undefined/ignored if it exists too early. Comment Actions I think the date is a bit long for something that probably isn't that common, and I would almost rather go ahead and make it an error now. Comment Actions I started with 2016-12-31 and wondered if it was not too close by, also, I need to figure out how to handle the PHP ports. |