Page MenuHomeFreeBSD

Mk/bsd.options.mk: Do not enable options that do not exist.
ClosedPublic

Authored by mat on Jan 17 2018, 4:44 PM.
Tags
None
Referenced Files
Unknown Object (File)
Jan 30 2024, 5:32 PM
Unknown Object (File)
Jan 13 2024, 9:46 AM
Unknown Object (File)
Dec 29 2023, 11:12 PM
Unknown Object (File)
Dec 28 2023, 5:14 AM
Unknown Object (File)
Dec 28 2023, 5:14 AM
Unknown Object (File)
Dec 28 2023, 5:14 AM
Unknown Object (File)
Dec 28 2023, 5:14 AM
Unknown Object (File)
Dec 28 2023, 5:14 AM

Details

Reviewers
None
Group Reviewers
portmgr
Commits
rP479428: Do not enable options that do not exist.
Summary

Many bits of the framework check for PORT_OPTIONS:MDOCS, which is on even if
the option is not defined.

Diff Detail

Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 14455
Build 14602: arc lint + arc unit

Event Timeline

won't this change all the rubygem packages?

all r-cran and drupal packages too

won't this change all the rubygem packages?

Almost all, some have DOCS options, but I think it's OK. I doubt anyone will miss the docs anyway.

  • Allow those options to be in a group, or something.

won't this change all the rubygem packages?
all r-cran and drupal packages too

Those that do not have a DOCS option, yes. Same for all ports that use PORTDOCS/PORTEXAMPLES or %%PORTDOCS%%/%%PORTEXAMPLES%%.

The "new" options framework was commited in rP297660 on May 29 2012, 161k commits ago. It is way past time to remove the "compat" layer for how it was before.

I'd only be ok with this after proof is posted that nothing is referring to ${PORT_OPTIONS:MDOCS} or <target>-DOCS-* without the _DOCS_ option in the list.

I'd only be ok with this after proof is posted that nothing is referring to ${PORT_OPTIONS:MDOCS} or <target>-DOCS-* without the _DOCS_ option in the list.

Well, people using the DOCS option without actually using it are doing it wrong, I am not sure we want to fix everyone's mess.

I have D13651 that somewhat takes it one step further and has a lot of OPTIONS_DEFINE=DOCS added.

linimon retitled this revision from Do not enable options that do not exist. to Mk/bsd.options.mk: Do not enable options that do not exist..Aug 2 2018, 10:24 PM

So, all the ports who were missing DOCS and EXAMPLES options have been fixed in rP479406 and rP479407.

rebase

  • Add IPV6 option to ports using it but not defining it.
  • Add NLS option to ports using it but not defining it.
  • Do not enable options that do not exist.
This revision was not accepted when it landed; it landed in state Needs Review.Sep 10 2018, 2:41 PM
This revision was automatically updated to reflect the committed changes.