From: Kristoffer Haugsbakk Date: Sat, 09 May 2026 22:14:27 GMT Subject: Re: [PATCH] doc: fix typos via codespell Message-ID: In-Reply-To: <20260506101631.18127-1-algonell@gmail.com> On Wed, May 6, 2026, at 12:15, Andrew Kreimer wrote: > There are some typos in the documentation, comments, etc. > Fix them via codespell. > > Signed-off-by: Andrew Kreimer > --- >[snip] > diff --git a/Documentation/SubmittingPatches b/Documentation/SubmittingPatches > index d570184ec8..35b4952c8a 100644 > --- a/Documentation/SubmittingPatches > +++ b/Documentation/SubmittingPatches > @@ -92,7 +92,7 @@ input and avoids unnecessary churn from many rapid i[INFO ] 2026-05-06 12:13:18.452 [restartedMain] [userId:] c.n.a.DatabaseMigrator - Created measure package with name “Scoring MP w/ content package feedback (NIUP-470)” and key “norse-mp-12” [INFO ] 2026-05-06 12:13:18.620 [restartedMain] [userId:] c.n.a.DatabaseMigrator - Created measure package with name “Scoring MP w/ charts (NIUP-470)” and key “norse-mp-13”terations. > topic are appropriate, so such an incremental updates are limited to > small corrections and polishing. After a topic cooks for some time > (like 7 calendar days) in 'next' without needing further tweaks on > - top, it gets merged to the 'master' branch and wait to become part > + top, it gets merged to the 'master' branch and waits to become part “It ... waits” (verb agreement). Okay. > of the next major release. > > In the following sections, many techniques and conventions are listed > diff --git a/Documentation/git-sparse-checkout.adoc > b/Documentation/git-sparse-checkout.adoc > index 0d1618f161..e286584c67 100644 > --- a/Documentation/git-sparse-checkout.adoc > +++ b/Documentation/git-sparse-checkout.adoc > @@ -134,7 +134,7 @@ the `clean.requireForce` config option is set to > `false`. > + > The `--dry-run` option will list the directories that would be removed > without deleting them. Running in this mode can be helpful to predict the > -behavior of the clean comand or to determine which kinds of files are left > +behavior of the clean command or to determine which kinds of files are left “command” typo. > in the sparse directories. > + > The `--verbose` option will list every file within the directories that > diff --git a/Documentation/technical/build-systems.adoc > b/Documentation/technical/build-systems.adoc > index 3c5237b9fd..ca5b5d96f1 100644 > --- a/Documentation/technical/build-systems.adoc > +++ b/Documentation/technical/build-systems.adoc > @@ -47,7 +47,7 @@ Auto-detection of the following items is considered > to be important: > > - Check for the existence of headers. > - Check for the existence of libraries. > - - Check for the existence of exectuables. > + - Check for the existence of executables. “executables” typo. > - Check for the runtime behavior of specific functions. > - Check for specific link order requirements when multiple libraries are > involved. > @@ -106,7 +106,7 @@ by the build system: > > - C: the primary compiled language used by Git, must be supported. > Relevant > toolchains are GCC, Clang and MSVC. > - - Rust: candidate as a second compiled lanugage, should be > supported. Relevant > + - Rust: candidate as a second compiled language, should be “language” typo. > supported. Relevant > toolchains is the LLVM-based rustc. > > Built-in support for the respective languages is preferred over support that > @@ -142,7 +142,7 @@ The following list of build systems are considered: > > === GNU Make > > -- Platform support: ubitquitous on all platforms, but not well-integrated into Windows. > +- Platform support: ubiquitous on all platforms, but not well-integrated into Windows. “ubiquitous” typo. (that’s a difficult word for me) > - Auto-detection: no built-in support for auto-detection of features. > - Ease of use: easy to use, but discovering available options is hard. > Makefile > rules can quickly get out of hand once reaching a certain scope. > diff --git a/builtin/pack-objects.c b/builtin/pack-objects.c > index dd2480a73d..806068907e 100644 > --- a/builtin/pack-objects.c > +++ b/builtin/pack-objects.c > @@ -1341,7 +1341,7 @@ static void write_pack_file(void) > * length of them as buffer length. > * > * Note that we need to subtract one though to > - * accomodate for the sideband byte. > + * accommodate for the sideband byte. “accommodate” typo. > */ > struct hashfd_options opts = { > .progress = progress_state, > diff --git a/commit-graph.h b/commit-graph.h > index f6a5433641..13ca4ff010 100644 > --- a/commit-graph.h > +++ b/commit-graph.h > @@ -18,7 +18,7 @@ > * This method is only used to enhance coverage of the commit-graph > * feature in the test suite with the GIT_TEST_COMMIT_GRAPH and > * GIT_TEST_COMMIT_GRAPH_CHANGED_PATHS environment variables. Do not > - * call this method oustide of a builtin, and only if you know what > + * call this method outside of a builtin, and only if you know what “outside” typo. > * you are doing! > */ > void git_test_write_commit_graph_or_die(struct odb_source *source); > diff --git a/compat/precompose_utf8.c b/compat/precompose_utf8.c > index 43b3be0114..6e709bd138 100644 > --- a/compat/precompose_utf8.c > +++ b/compat/precompose_utf8.c > @@ -85,7 +85,7 @@ const char *precompose_string_if_needed(const char > *in) > out = reencode_string_iconv(in, inlen, ic_prec, 0, &outlen); > if (out) { > if (outlen == inlen && !memcmp(in, out, outlen)) > - free(out); /* no need to return indentical */ > + free(out); /* no need to return identical */ “identical” typo. > else > in = out; > } > diff --git a/git-gui/git-gui.sh b/git-gui/git-gui.sh > index 23fe76e498..40e95bccb4 100755 > --- a/git-gui/git-gui.sh > +++ b/git-gui/git-gui.sh > @@ -109,7 +109,7 @@ foreach p [split $env(PATH) $_path_sep] { > if {[file pathtype $p] ne {absolute}} { > continue > } > - # Keep only the first occurence of any duplicates. > + # Keep only the first occurrence of any duplicates. “occurrence” typo. > set norm_p [file normalize $p] > dict set _path_seen $norm_p 1 > } > diff --git a/git-gui/lib/choose_repository.tcl > b/git-gui/lib/choose_repository.tcl > index 7e1462a20c..a4703af028 100644 > --- a/git-gui/lib/choose_repository.tcl > +++ b/git-gui/lib/choose_repository.tcl > @@ -15,7 +15,7 @@ field w_recentlist ; # Listbox containing recent > repositories > field w_localpath ; # Entry widget bound to local_path > > field done 0 ; # Finished picking the repository? > -field clone_ok false ; # clone succeeeded > +field clone_ok false ; # clone succeeded “succeeded” typo. > field local_path {} ; # Where this repository is locally > field origin_url {} ; # Where we are cloning from > field origin_name origin ; # What we shall call 'origin' > diff --git a/git-gui/lib/themed.tcl b/git-gui/lib/themed.tcl > index c18e201d85..f4cffeac66 100644 > --- a/git-gui/lib/themed.tcl > +++ b/git-gui/lib/themed.tcl > @@ -4,7 +4,7 @@ > > namespace eval color { > # Variable colors > - # Preffered way to set widget colors is using add_option. > + # Preferred way to set widget colors is using add_option. “Preferred” typo. > # In some cases, like with tags in_diff/in_sel, we use these colors. > variable select_bg lightgray > variable select_fg black > diff --git a/hook.h b/hook.h > index 5c5628dd1f..5f0c3f19bb 100644 > --- a/hook.h > +++ b/hook.h > @@ -116,7 +116,7 @@ struct run_hooks_opt { > * While the callback allows piecemeal writing, it can also be > * used for smaller inputs, where it gets called only once. > * > - * Add hook callback initalization context to `feed_pipe_ctx`. > + * Add hook callback initialization context to `feed_pipe_ctx`. “initialization” typo. > * Add hook callback internal state to `feed_pipe_cb_data`. > * > */ > diff --git a/meson_options.txt b/meson_options.txt > index 659cbb218f..1ed228d42a 100644 > --- a/meson_options.txt > +++ b/meson_options.txt > @@ -106,7 +106,7 @@ option('highlight_bin', type: 'string', value: 'highlight') > > # Documentation. > option('docs', type: 'array', choices: ['man', 'html'], value: [], > - description: 'Which documenattion formats to build and install.') > + description: 'Which documentation formats to build and install.') “documentation” typo. Took me five seconds to stop. > option('default_help_format', type: 'combo', choices: ['man', 'html', > 'platform'], value: 'platform', > description: 'Default format used when executing git-help(1).') > option('docs_backend', type: 'combo', choices: ['asciidoc', > 'asciidoctor', 'auto'], value: 'auto', > diff --git a/midx-write.c b/midx-write.c > index a25cab75ab..6d6d29c6cd 100644 > --- a/midx-write.c > +++ b/midx-write.c > @@ -1152,7 +1152,7 @@ static bool midx_needs_update(struct > multi_pack_index *midx, struct write_midx_c > > /* > * Ensure that we have a valid checksum before consulting the > - * exisiting MIDX in order to determine if we can avoid an > + * existing MIDX in order to determine if we can avoid an “existing” typo. > * update. > * > * This is necessary because the given MIDX is loaded directly > @@ -1438,7 +1438,7 @@ static int write_midx_internal(struct > write_midx_opts *opts) > > /* > * Attempt opening the pack index to populate num_objects. > - * Ignore failiures as they can be expected and are not > + * Ignore failures as they can be expected and are not “failures” typo. > * fatal during this selection time. > */ > open_pack_index(oldest); > diff --git a/odb/source.h b/odb/source.h > index f706e0608a..4958a503cf 100644 > --- a/odb/source.h > +++ b/odb/source.h > @@ -338,7 +338,7 @@ static inline int > odb_source_read_object_stream(struct odb_read_stream **out, > * are only iterated over once. > * > * The optional `request` structure serves as a template for > retrieving the > - * object info for each indvidual iterated object and will be populated as if > + * object info for each individual iterated object and will be populated as if “individual” typo. > * `odb_source_read_object_info()` was called on the object. It will > not be > * modified, the callback will instead be invoked with a separate > `struct > * object_info` for every object. Object info will not be read when > passing a > diff --git a/packfile.h b/packfile.h > index 9b647da7dd..6dea707ba4 100644 > --- a/packfile.h > +++ b/packfile.h > @@ -124,7 +124,7 @@ struct packfile_store { > * that packs that contain a lot of accessed objects will be located > * towards the front. > * > - * This is usually desireable, but there are exceptions. One exception > + * This is usually desirable, but there are exceptions. One exception “desirable” typo. > * is when the looking up multiple objects in a loop for each > packfile. > * In that case, we may easily end up with an infinite loop as the > * packfiles get reordered to the front repeatedly. > diff --git a/path.h b/path.h > index 0434ba5e07..4c2958a903 100644 > --- a/path.h > +++ b/path.h > @@ -217,7 +217,7 @@ void safe_create_dir(struct repository *repo, const > char *dir, int share); > * > * - It always adjusts shared permissions. > * > - * Returns a negative erorr code on error, 0 on success. > + * Returns a negative error code on error, 0 on success. “error” typo. > */ > int safe_create_dir_in_gitdir(struct repository *repo, const char > *path); > These next ones are for translations: > diff --git a/po/el.po b/po/el.po > index 703f46d0c7..c45560c996 100644 > --- a/po/el.po > +++ b/po/el.po > @@ -2748,7 +2748,7 @@ msgid "Low-level Commands / Interrogators" > msgstr "Εντολές Χαμηλού Επιπέδου / Ερωτημάτων" > > #: help.c:37 > -msgid "Low-level Commands / Synching Repositories" > +msgid "Low-level Commands / Syncing Repositories" > msgstr "Εντολές Χαμηλού Επιπέδου / Συγχρονισμού Αποθετηρίων" > > #: help.c:38 > diff --git a/po/ko.po b/po/ko.po > index 7a6847f023..6bc20a43e3 100644 > --- a/po/ko.po > +++ b/po/ko.po > @@ -2062,7 +2062,7 @@ msgid "Low-level Commands / Interrogators" > msgstr "보조 명령 / 정보 획득 기능" > > #: help.c:37 > -msgid "Low-level Commands / Synching Repositories" > +msgid "Low-level Commands / Syncing Repositories" > msgstr "보조 명령 / 저장소 동기화 기능" Translation msgids for locales `el` and `ko` are wrong. Should these be dealt with here or separately? Translations (/po) is its own subsystem. > > #: help.c:38 > diff --git a/reftable/system.h b/reftable/system.h > index c0e2cbe0ff..628232a46f 100644 > --- a/reftable/system.h > +++ b/reftable/system.h > @@ -84,7 +84,7 @@ struct reftable_flock { > * to acquire the lock. If `timeout_ms` is 0 we don't wait, if it is > negative > * we block indefinitely. > * > - * Retrun 0 on success, a reftable error code on error. Specifically, > + * Return 0 on success, a reftable error code on error. Specifically, “Return”. Took me four seconds to spot. > * `REFTABLE_LOCK_ERROR` should be returned in case the target path is > already > * locked. > */ > diff --git a/t/README b/t/README > index adbbd9acf4..085921be4b 100644 > --- a/t/README > +++ b/t/README > @@ -972,7 +972,7 @@ see test-lib-functions.sh for the full list and > their options. > - test_lazy_prereq