Skip to content

Commit

Permalink
Merge branch 'jc/document-use-of-local'
Browse files Browse the repository at this point in the history
Doc update.

* jc/document-use-of-local:
  doc: note that AT&T ksh does not work with our test suite
  • Loading branch information
gitster committed Aug 14, 2024
2 parents 4a443f0 + 67be8c4 commit dd59778
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions Documentation/CodingGuidelines
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down

0 comments on commit dd59778

Please sign in to comment.