| Message ID | 20220316140007.184425-1-michael.opdenacker@bootlin.com |
|---|---|
| State | New |
| Headers | show |
| Series | convert-variable-renames.py: add more problematic words | expand |
On Wed, 2022-03-16 at 15:00 +0100, Michael Opdenacker via lists.openembedded.org wrote: > Listed on https://inclusivenaming.org/ > > Signed-off-by: Michael Opdenacker <michael.opdenacker@bootlin.com> > --- > scripts/contrib/convert-variable-renames.py | 7 ++++++- > 1 file changed, 6 insertions(+), 1 deletion(-) Not sure we want to add a whole load more work at this point in the release cycle? This wasn't what we agreed as the first set of priorities :/. Cheers, Richard
On 3/16/22 15:00, Michael Opdenacker wrote: > Listed on https://inclusivenaming.org/ > > Signed-off-by: Michael Opdenacker <michael.opdenacker@bootlin.com> > --- > scripts/contrib/convert-variable-renames.py | 7 ++++++- > 1 file changed, 6 insertions(+), 1 deletion(-) > > diff --git a/scripts/contrib/convert-variable-renames.py b/scripts/contrib/convert-variable-renames.py > index 856c001e11..304f5484ed 100755 > --- a/scripts/contrib/convert-variable-renames.py > +++ b/scripts/contrib/convert-variable-renames.py > @@ -53,9 +53,14 @@ removed_list = [ > ] > > context_check_list = [ > +"abort", > "blacklist", > +"slave", > +"sanity check", > +"sanity test", > +"segregate", > +"segregation", > "whitelist", > -"abort", Oops, I meant to send this as an RFC, but forgot when I tweaked the description line. Cheers Michael.
On 3/16/22 15:02, Richard Purdie wrote: > On Wed, 2022-03-16 at 15:00 +0100, Michael Opdenacker via lists.openembedded.org > wrote: >> Listed on https://inclusivenaming.org/ >> >> Signed-off-by: Michael Opdenacker <michael.opdenacker@bootlin.com> >> --- >> scripts/contrib/convert-variable-renames.py | 7 ++++++- >> 1 file changed, 6 insertions(+), 1 deletion(-) > Not sure we want to add a whole load more work at this point in the release > cycle? This wasn't what we agreed as the first set of priorities :/. The word checks are just warnings by the way, but no problem at all for me. I figured I'd send a patch rather than a question ;-) Thanks Michael.
diff --git a/scripts/contrib/convert-variable-renames.py b/scripts/contrib/convert-variable-renames.py index 856c001e11..304f5484ed 100755 --- a/scripts/contrib/convert-variable-renames.py +++ b/scripts/contrib/convert-variable-renames.py @@ -53,9 +53,14 @@ removed_list = [ ] context_check_list = [ +"abort", "blacklist", +"slave", +"sanity check", +"sanity test", +"segregate", +"segregation", "whitelist", -"abort", ] def processfile(fn):
Listed on https://inclusivenaming.org/ Signed-off-by: Michael Opdenacker <michael.opdenacker@bootlin.com> --- scripts/contrib/convert-variable-renames.py | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-)