Bug 15039 - spelling fixes
Summary: spelling fixes
Status: NEW
Alias: None
Product: Samba 4.1 and newer
Classification: Unclassified
Component: Other (show other bugs)
Version: unspecified
Hardware: All All
: P5 minor (vote)
Target Milestone: ---
Assignee: Samba QA Contact
QA Contact: Samba QA Contact
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-04-01 08:07 UTC by Michael Tokarev
Modified: 2022-04-04 13:19 UTC (History)
1 user (show)

See Also:


Attachments
git patch against current master (23.99 KB, patch)
2022-04-01 08:07 UTC, Michael Tokarev
no flags Details
updated version of the same patch including fixes for authencication progagated (25.76 KB, patch)
2022-04-01 18:15 UTC, Michael Tokarev
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Tokarev 2022-04-01 08:07:26 UTC
Created attachment 17258 [details]
git patch against current master

This is a fix for some common spelling errors in samba code and documentation:

 connnect encrytion exisit expection explicit invalide missmatch
 paramater paramter partion privilige relase reponse seperate unkown verson

The same errors are present in third_party/ too, especially in heimdal, I didn't address these.

This is my first patch submission to samba, I don't yet know the rules :)
Comment 1 Michael Tokarev 2022-04-01 18:15:38 UTC
Created attachment 17261 [details]
updated version of the same patch including fixes for authencication progagated

content-type is binary b/c it contains cr-lf line endings
Comment 2 Michael Tokarev 2022-04-01 18:16:51 UTC
(one should use git am --keep-crlf to apply it)
Comment 3 Michael Tokarev 2022-04-01 18:53:56 UTC
FWIW, it wont work as text/plain due to different line endings in the files it touches.
Comment 4 Andrew Bartlett 2022-04-04 00:48:23 UTC
Please submit patches via GitLab per https://wiki.samba.org/index.php/Contribute#Bugzilla
Comment 5 Andrew Bartlett 2022-04-04 00:49:31 UTC
Please also avoid changing the schema (*.ldf) files, any errors in those files are as provided, and we want to match, spelling errors included.
Comment 6 Michael Tokarev 2022-04-04 13:19:55 UTC
I already have more than enough samba-related accounts, I don't need more, thank you very much. I'm happy to carry this patch locally in debian (-ldf bits) if you guys don't want it.