diff options
| author | Junio C Hamano <gitster@pobox.com> | 2024-07-15 17:05:00 +0200 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2024-07-15 19:14:52 +0200 |
| commit | 67be8c4de5255efd95c48c9ddd6cd8537fbf03d1 (patch) | |
| tree | a5e613d83a25dce0aac78eb65a276aa914779fa6 | |
| parent | Git 2.46-rc0 (diff) | |
| download | git-67be8c4de5255efd95c48c9ddd6cd8537fbf03d1.tar.xz git-67be8c4de5255efd95c48c9ddd6cd8537fbf03d1.zip | |
doc: note that AT&T ksh does not work with our test suite
The scripted Porcelain commands do not allow use of "local" because
it is not universally supported, but we use it liberally in our test
scripts, which means some POSIX compliant shells (like "ksh93") can
not be used to run our tests.
Document the status quo, to help the next person who gets perplexed
seeing our tests fail.
Signed-off-by: Junio C Hamano <gitster@pobox.com>
| -rw-r--r-- | Documentation/CodingGuidelines | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/Documentation/CodingGuidelines b/Documentation/CodingGuidelines index 1d92b2da03..94ca5cf7c0 100644 --- a/Documentation/CodingGuidelines +++ b/Documentation/CodingGuidelines @@ -185,8 +185,8 @@ For shell scripts specifically (not exhaustive): - Even though "local" is not part of POSIX, we make heavy use of it in our test suite. We do not use it in scripted Porcelains, and - hopefully nobody starts using "local" before they are reimplemented - in C ;-) + hopefully nobody starts using "local" before all shells that matter + support it (notably, ksh from AT&T Research does not support it yet). - Some versions of shell do not understand "export variable=value", so we write "variable=value" and then "export variable" on two |