The checkpatch spelling test uses the sentence "an intentionaly misspelt word" and expects only "intentionaly" to be flagged. However, "misspelt" is the British English past tense of "misspell" and is not present in all en_US enchant dictionaries. On systems where the en_US dictionary does not include "misspelt" (e.g. Fedora with hunspell-en-US), it gets flagged as a misspelling too, causing the test to fail.
Replace "misspelt" with "misspelled", which is the standard American English form and is universally recognized by en_US dictionaries. CC: Eli Oliver <[email protected]> Fixes: d2f1ca089ed1 ("checkpatch: Add simple python comment spell checking.") Signed-off-by: Timothy Redaelli <[email protected]> --- tests/checkpatch.at | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/checkpatch.at b/tests/checkpatch.at index b63d7005e..7d80ffbed 100755 --- a/tests/checkpatch.at +++ b/tests/checkpatch.at @@ -795,11 +795,11 @@ try_checkpatch_c_file \ "/^Did you mean:/d" try_checkpatch_py_file \ - "# This is a python file with an intentionaly misspelt word. + "# This is a python file with an intentionaly misspelled word. # The user wants to check if it's working." \ "WARNING: Possible misspelled word: \"intentionaly\" test.py:1: - # This is a python file with an intentionaly misspelt word." \ + # This is a python file with an intentionaly misspelled word." \ "-S" \ "/^Did you mean:/d" -- 2.53.0 _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
