Page MenuHomeFreeBSD

Add a warning for people who set foo_DEFAULT in their make.conf or environment.
ClosedPublic

Authored by mat on Sep 28 2016, 2:03 PM.
Tags
None
Referenced Files
Unknown Object (File)
Nov 14 2024, 1:55 PM
Unknown Object (File)
Oct 2 2024, 12:08 PM
Unknown Object (File)
Oct 2 2024, 2:26 AM
Unknown Object (File)
Sep 27 2024, 10:07 AM
Unknown Object (File)
Sep 27 2024, 3:05 AM
Unknown Object (File)
Sep 8 2024, 8:53 PM
Unknown Object (File)
Sep 8 2024, 10:47 AM
Unknown Object (File)
Sep 8 2024, 3:52 AM
Subscribers

Details

Summary

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

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 5337
Build 5524: arc lint + arc unit

Event Timeline

mat retitled this revision from to Add a warning for people who set foo_DEFAULT in their make.conf or environment..
mat updated this object.
mat edited the test plan for this revision. (Show Details)
mat added a reviewer: portmgr.

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?

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?

I don't know, it is possible, yes, or I will just .undef it and ignore it.

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.

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.

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.

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.

This revision was automatically updated to reflect the committed changes.