Inclusive language: Difference between revisions
Line 67: | Line 67: | ||
!Recommended by | !Recommended by | ||
!Approved name | !Approved name | ||
!Developer assigned | |||
!Notes | !Notes | ||
|- | |- | ||
Line 73: | Line 74: | ||
|"HALT", "NO_NEW_TASKS" or "WARN" | |"HALT", "NO_NEW_TASKS" or "WARN" | ||
|Richard Purdie | |Richard Purdie | ||
| | |||
| | | | ||
| | | | ||
Line 80: | Line 82: | ||
|Should be changed in the code | |Should be changed in the code | ||
|Richard Purdie | |Richard Purdie | ||
| | |||
| | | | ||
| | | | ||
Line 87: | Line 90: | ||
|BB_ENV_PASSTHROUGH | |BB_ENV_PASSTHROUGH | ||
|Richard Purdie | |Richard Purdie | ||
| | |||
| | | | ||
| | | | ||
Line 94: | Line 98: | ||
|BB_ENV_PASSTHROUGH_ADDITIONS | |BB_ENV_PASSTHROUGH_ADDITIONS | ||
|Richard Purdie | |Richard Purdie | ||
| | |||
| | | | ||
| | | | ||
Line 102: | Line 107: | ||
|Richard Purdie | |Richard Purdie | ||
| | | | ||
| | |||
| | | | ||
|- | |- | ||
Line 109: | Line 115: | ||
|Richard Purdie | |Richard Purdie | ||
| | | | ||
| | |||
| | | | ||
|- | |- | ||
Line 115: | Line 122: | ||
|BB_SETSCENE_ENFORCE_IGNORE_TASKS | |BB_SETSCENE_ENFORCE_IGNORE_TASKS | ||
|Richard Purdie | |Richard Purdie | ||
| | |||
| | | | ||
| | | | ||
Line 122: | Line 130: | ||
|BB_BASEHASH_IGNORE_VARS | |BB_BASEHASH_IGNORE_VARS | ||
|Richard Purdie | |Richard Purdie | ||
| | |||
| | | | ||
| | | | ||
Line 130: | Line 139: | ||
|Richard Purdie | |Richard Purdie | ||
| | | | ||
| | |||
|- | |- | ||
|Bitbake | |Bitbake | ||
|basewhitelist | |basewhitelist | ||
| | |||
| | | | ||
| | | | ||
Line 140: | Line 151: | ||
|Bitbake | |Bitbake | ||
|taskwhitelist | |taskwhitelist | ||
| | |||
| | | | ||
| | | | ||
Line 149: | Line 161: | ||
|based on context | |based on context | ||
|Saul Wold | |Saul Wold | ||
| | |||
| | | | ||
| | | | ||
Line 156: | Line 169: | ||
|See notes | |See notes | ||
|Richard Purdie | |Richard Purdie | ||
| | |||
| | | | ||
|For PNBLACKLIST and blacklist.bbclass, I think we need to take the opportunity | |For PNBLACKLIST and blacklist.bbclass, I think we need to take the opportunity | ||
Line 165: | Line 179: | ||
|allowlist | |allowlist | ||
|Saul | |Saul | ||
| | |||
| | | | ||
|This could be change based on context | |This could be change based on context | ||
Line 172: | Line 187: | ||
|allowed | |allowed | ||
|Saul | |Saul | ||
| | |||
| | | | ||
|Context sensitive | |Context sensitive | ||
Line 179: | Line 195: | ||
|allowing | |allowing | ||
|Saul | |Saul | ||
| | |||
| | | | ||
| | | | ||
Line 186: | Line 203: | ||
|SSTATE_ALLOW_OVERLAP_FILES | |SSTATE_ALLOW_OVERLAP_FILES | ||
|Richard Purdie | |Richard Purdie | ||
| | |||
| | | | ||
| | | | ||
Line 193: | Line 211: | ||
|CVE_CHECK_SKIPRECIPE | |CVE_CHECK_SKIPRECIPE | ||
|Richard Purdie | |Richard Purdie | ||
| | |||
| | | | ||
| | | | ||
Line 201: | Line 220: | ||
|Richard Purdie | |Richard Purdie | ||
| | | | ||
| | || | ||
|- | |- | ||
|core | |core | ||
Line 209: | Line 228: | ||
| | | | ||
| | | | ||
|Should be removed | |||
|- | |- | ||
|core | |core | ||
Line 216: | Line 236: | ||
| | | | ||
| | | | ||
|Should be removed | |||
|- | |- | ||
|core | |core | ||
Line 223: | Line 244: | ||
| | | | ||
| | | | ||
|Should be removed | |||
|- | |- | ||
|core | |core | ||
Line 228: | Line 250: | ||
|SYSROOT_DIRS_IGNORE | |SYSROOT_DIRS_IGNORE | ||
|Richard Purdie | |Richard Purdie | ||
| | |||
| | | | ||
| | | | ||
Line 235: | Line 258: | ||
|INHERIT_RECIPESKIP | |INHERIT_RECIPESKIP | ||
|Saul | |Saul | ||
| | |||
| | | | ||
|if we drop the blacklist class, we can drop this too (RP) | |if we drop the blacklist class, we can drop this too (RP) | ||
Line 249: | Line 273: | ||
|ESDK_LOCALCONF_ALLOW | |ESDK_LOCALCONF_ALLOW | ||
|Richard Purdie | |Richard Purdie | ||
| | |||
| | | | ||
| | | | ||
Line 256: | Line 281: | ||
|ESDK_CLASS_INHERIT_DISABLE | |ESDK_CLASS_INHERIT_DISABLE | ||
|Richard Purdie | |Richard Purdie | ||
| | |||
| | | | ||
| | | | ||
Line 261: | Line 287: | ||
|core | |core | ||
|WHITELIST_(ANY LICENSE) | |WHITELIST_(ANY LICENSE) | ||
| | |||
| | | | ||
| | | | ||
Line 270: | Line 297: | ||
|LICENSE_FLAGS_ACCEPTED (and BLOCKED) | |LICENSE_FLAGS_ACCEPTED (and BLOCKED) | ||
|Saul | |Saul | ||
| | |||
| | | | ||
| Should be changed to | | Should be changed to | ||
Line 277: | Line 305: | ||
|UNKNOWN_CONFIGURE_OPT_IGNORE | |UNKNOWN_CONFIGURE_OPT_IGNORE | ||
|Richard Purdie | |Richard Purdie | ||
| | |||
| | | | ||
| | | | ||
Line 282: | Line 311: | ||
| | | | ||
|TUNEABI_WHITELIST | |TUNEABI_WHITELIST | ||
| | |||
| | | | ||
| | | |
Revision as of 23:05, 14 January 2022
Inclusive Language
First time topic was explored on mailing list. https://lists.openembedded.org/g/openembedded-architecture/topic/inclusive_language_summary/75821819
This is an attempt to collate Inclusive Language for OpenEmbedded and the Yocto Project in regards to code. This is an attempt to improve these variables, comments, and terminology to be more descriptive and also inclusive/ not offensive.
For reference: https://inclusivenaming.org/word-lists/overview/
Game play
If a BB variable has a local assignment, the rename should be transitive. Example: BB_HASHBASE_WHITELIST is changed to BB_HASHBASE_ALLOWLIST, then self.basewhitelist would become self.baseallowlist
Add an offending name in one of the three tables below. If you have a suggested replacement, please add. If the translation is approved, that will be noted in the same row. Approval process: TBD
Recommendations
Recommended replacements for 'blacklist/whitelist' are:
blacklist | whitelist |
---|---|
denylist/disallowlist | allowlist |
blocklist | passlist or welcomelist |
excludes or excludelist | includes or includelist |
Exceptions
Exceptions for introducing new usage is limited upstream sources outside the control of The Yocto Project and OpenEmbedded.
Branch names, like master, are not being handled on the initial phase of this task
Classes
Repo | Current name | Suggest rename | Recommended by | Approved name |
---|---|---|---|---|
core | blacklist.bbclass | excludelist.bbclass | Armin | |
core | blacklist.bbclass (alt) | forcerecipeskip.bbclass | Saul |
Variables
Repo | Current name | Suggest rename | Recommended by | Approved name | Developer assigned | Notes |
---|---|---|---|---|---|---|
Bitbake | ABORT | "HALT", "NO_NEW_TASKS" or "WARN" | Richard Purdie | |||
Bitbake | abort (comment or variable) | Should be changed in the code | Richard Purdie | |||
Bitbake | BB_ENV_WHITELIST | BB_ENV_PASSTHROUGH | Richard Purdie | |||
Bitbake | BB_ENV_EXTRAWHITE | BB_ENV_PASSTHROUGH_ADDITIONS | Richard Purdie | |||
Bitbake | BB_HASHCONFIG_WHITELIST | BB_HASHCONFIG_IGNORE_VARS | Richard Purdie | |||
Bitbake | BB_STAMP_WHITELIST | remove this along with BB_STAMP_POLICY | Richard Purdie | |||
Bitbake | BB_SETSCENE_ENFORCE_WHITELIST | BB_SETSCENE_ENFORCE_IGNORE_TASKS | Richard Purdie | |||
Bitbake | BB_HASHBASE_WHITELIST | BB_BASEHASH_IGNORE_VARS | Richard Purdie | |||
Bitbake | MULTI_PROVIDER_WHITELIST | BB_MULTI_PROVIDER_ALLOWED | Richard Purdie | |||
Bitbake | basewhitelist | |||||
Bitbake | taskwhitelist | |||||
core | abort (comment or variable) | based on context | Saul Wold | |||
core | PNBLACKLIST | See notes | Richard Purdie | For PNBLACKLIST and blacklist.bbclass, I think we need to take the opportunity
to use "recipe" instead of "pn". For blacklist.bbclass, perhaps we just merge into base.bbclass and use SKIP_RECIPE[pn] ? | ||
core | whitelist | allowlist | Saul | This could be change based on context | ||
core | whitelisted | allowed | Saul | Context sensitive | ||
core | whitelisting | allowing | Saul | |||
core | SSTATE_DUPWHITELIST | SSTATE_ALLOW_OVERLAP_FILES | Richard Purdie | |||
core | CVE_CHECK_PN_WHITELIST | CVE_CHECK_SKIPRECIPE | Richard Purdie | |||
core | CVE_CHECK_WHITELIST | CVE_CHECK_IGNORECVE | Richard Purdie | |||
core | ICECC_USER_CLASS_BL | ICECC_USER_CLASS_EXCLUDES | Michael Opdenacker | Should be removed | ||
core | ICECC_USER_PACKAGE_WL | ICECC_USER_CLASS_ALLOWLIST | Michael Opdenacker | Should be removed | ||
core | ICECC_USER_PACKAGE_BL | ICECC_USER_PACKAGE_EXCLUDES | Michael Opdenacker | Should be removed | ||
core | SYSROOT_DIRS_BLACKLIST | SYSROOT_DIRS_IGNORE | Richard Purdie | |||
core | INHERIT_BLACKLIST | INHERIT_RECIPESKIP | Saul | if we drop the blacklist class, we can drop this too (RP) | ||
core | SDK_LOCAL_CONF_BLACKLIST | ESDK_LOCALCONF_REMOVE | Richard Purdie | |||
core | SDK_LOCAL_CONF_WHITELIST | ESDK_LOCALCONF_ALLOW | Richard Purdie | |||
core | SDK_INHERIT_BLACKLIST | ESDK_CLASS_INHERIT_DISABLE | Richard Purdie | |||
core | WHITELIST_(ANY LICENSE) | Should be removed and replaced with INCOMPATIBLE_LICENSE_ALLOWED_RECIPES, which would be a list of recipes which are of a blocked license that is now being overridden | ||||
core | LICENSE_FLAGS_WHITELIST | LICENSE_FLAGS_ACCEPTED (and BLOCKED) | Saul | Should be changed to | ||
core | UNKNOWN_CONFIGURE_WHITELIST | UNKNOWN_CONFIGURE_OPT_IGNORE | Richard Purdie | |||
TUNEABI_WHITELIST | Removed |
Functions
Repo | Current name | Suggest rename | Recommended by | Approved name |
---|---|---|---|---|
Bitbake | test_setscenewhitelist |
Patch Files
Repo | Recipe | Current name | Suggest rename | Recommended by | Approved name |
---|---|---|---|---|---|
core | tcp-wrappers | 11_tcpd_blacklist.patch | 11_tcpd_blocklist.patch | Saul Wold | |
core | udev-extraconf | mount.blacklist | mount.disallow | Saul Wold | |
oe | lxdm | 0001-lxdm.conf.in-blacklist-root-for-release-images.patch | 0001-lxdm.conf.in-deny-root-for-release-images.patch | Saul Wold | |
oe | multipath-tools | 022-RH-Remove-the-property-blacklist-exception-builtin.patch | 022-RH-Remove-the-default-property-exception-builtin.patch | Saul Wold | |
oe | mozjs | 0001-Cargo.toml-do-not-abort-on-panic.patch | 0001-Cargo.toml-do-not-exit-on-panic.patch | Saul Wold | |
oe | mozjs | 0004-Cargo.toml-do-not-abort-on-panic.patch | 0004-Cargo.toml-do-not-exit-on-panic.patch | Saul Wold | |
meta-integrity | files | ima_signing_blacklist | ima_signing_blocklist | Saul Wold | |
meta-efi-secure-boot | efitools | Fix-the-wrong-dependency-for-blacklist.esl.patch | Fix-the-wrong-dependency-esl-file.patch | Saul Wold | |
meta-raspberrypi | gstreamer1.0-omx | 0005-Don-t-abort-gst_omx_video_dec_set_format-if-there-s-.patch | 0005-Don-t-return-gst_omx_video_dec_set_format-if-there-s-.patch | Saul Wold |
Notes
Some upstream references:
[1][ Linux kernel]
[2][IETF: Terminology, Power and Inclusive Language: Draft v2]
[3][Why changing the terms blacklist and whitelist isn’t as easy as it might seem]