On 7/15/25 3:41 PM, Aaron Conole wrote:
> Checkpatch contains a large list of spelling words, currently 326. The
> in-built list is quite cumbersome, and includes duplicate entries
> ('netdev' and 'revalidation'). Managing this list will continue to grow
> difficult over time, and really shouldn't be so difficult. To make the
> maintenance of OVS specific spelling words easier, create a new file
> that will contain the spelling words and move the list into that file
> (sorted). This patch adds 'FreeBSD' and 'gcloud' to the list as it
> was tested against
> 80d723736b64 ("cirrus: Update to FreeBSD 14.3 and 13.5."). It also
> adds "apis", "enablement", "ifdef", and "tnl" after the patch proposed
> by Eelco at the referenced link.
>
> Ref:
> https://patchwork.ozlabs.org/project/openvswitch/patch/d6fecfc9491893b852ff9de3ef8b7ef068535d6d.1750328980.git.echau...@redhat.com/
Nit: I'd use a 'Link:' here.
> Acked-by: Eelco Chaudron <echau...@redhat.com>
> Signed-off-by: Aaron Conole <acon...@redhat.com>
> ---
> v4->v5: Incorporate Ilya's feedback for the testsuite
>
> tests/checkpatch.at | 52 +++++-
> utilities/automake.mk | 1 +
> utilities/checkpatch.py | 73 +-------
> utilities/checkpatch_dict.txt | 330 ++++++++++++++++++++++++++++++++++
> 4 files changed, 382 insertions(+), 74 deletions(-)
> create mode 100644 utilities/checkpatch_dict.txt
Thanks for the update! This version looks good to me.
(CI failed for unrelated reason.)
Acked-by: Ilya Maximets <i.maxim...@ovn.org>
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
