Ilya Maximets <i.maxim...@ovn.org> writes:
> 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.
Fixed on apply.
>> 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>
Thanks Ilya - applied with the nit above corrected.
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
