On 6/30/25 11:26 AM, Eelco Chaudron wrote:
>
>
> On 26 Jun 2025, at 14:52, Aaron Conole via dev 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
>> naintenance of OVS specific spelling words easier, create a new file
>> that will contain the spelling words and move the list into that file
>> (sorted). Additionally, add 'FreeBSD' and 'gcloud' to the list as I
>> tested against 80d723736b64 ("cirrus: Update to FreeBSD 14.3 and 13.5.")
>>
>> Signed-off-by: Aaron Conole <acon...@redhat.com>
>
> Thanks for doing this, Aaron. Can you add the words from
> https://patchwork.ozlabs.org/project/openvswitch/list/?series=461629 as well,
> so we can include them with this change? We could probably mark it as such.
>
> Also, not sure if the name and location are the most optimal. When looking
> at your comment on my patch, I was thinking of something like
> utilities/checkpatch_words.txt, but I’m not sure if that’s any better.
> Anyone else have an opinion on this?
No strong opinion on the name, but I would also expect this to be in the
utilities/ directory instead of build-aux, as checkpatch is not part of
the build process.
Aaron, there is also a wrong word split in the current patch that made you
miss the duplicated 'uuid' word:
+uuid
+uuid virtqueue
Best regards, Ilya Maximets.
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
