-
Notifications
You must be signed in to change notification settings - Fork 16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Spelling errors spotted by lintian #66
Comments
@Fantu thanks for the report. I'll have to check the specifications to see if they are typos from me or from the specification. I recall several definitely came from the specification, but I certainly could have introduced some. At the barest minimum I should add comments, documenting where the spelling errors came from OR if we'll leave them there to avoid programming breakage. |
I just did a check on all of the errors you listed above, and AFAICT (which maybe I missed one), every one of these typos comes from the IANA database of manufacturers http://www.iana.org/assignments/enterprise-numbers. In some cases I think the typo is in their database, in other cases the spelling is probably correct (e.g. someone's name) or an alternate spelling in non-English native countries (Interestingly, "Nam" appears to be a spacing typo from "Viet Nam"). I'm not sure there's anything I can do about this, as I'm going off what the database says. They may have corrected some since the last time I updated it. Is there any type of Lintian "ignore this file for spelling errors" that could be added? I'd be happy to add the config or comments into the repo. |
for check spelling error I saw for example codespell as useful, for example with: |
I did a PR with codespell config file: #72 |
Hi, lintian spotted some spelling errors:
https://salsa.debian.org/debian/freeipmi/-/jobs/4214071
Some probably must remain as they are because they are part of code or specifications while some may be correctable errors, I have posted them in case someone want to check and correct where needed and possible
EDIT:
I checked some with grep and are all related to specification files, some seems almost surely a typo, it would be useful to know if they have actually been added as it is in the specifications and their modification would cause functioning problems or if there are strings used for "display only" and the correction of typo would not compromise the functioning
The text was updated successfully, but these errors were encountered: