Skip to content
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

Fix typos in man pages. #351

Closed
wants to merge 1 commit into from
Closed

Conversation

catull
Copy link
Contributor

@catull catull commented Jan 12, 2018

Apologies for the 'final white-space' in many lines.

@krader1961
Copy link
Contributor

Merged. Thx.

@krader1961 krader1961 closed this Jan 12, 2018
@catull catull deleted the fix-typos-in-man-pages branch January 13, 2018 00:32
JohnoKing added a commit to JohnoKing/ksh that referenced this pull request Aug 6, 2020
Most of these fixes are for typos and extra whitespace at the
end of lines. These are the notable changes:
- Fixed a compatibility issue with how asterisks are displayed
  using certain fonts. Bug report: att#764
- Fixed a bug in the man page that caused searches for the '|'
  character to fail. Bug report: att#871
- Removed a duplicate description of 'set -B' from the man
  page. Bug report: att#789
- Added documentation for options missing from the ksh man
  page (applies to 'hist -N', 'sleep -s', 'whence -q' and
  many of ulimit's options). Bug reports:
  att#948
  att#503 (comment)
  att#507 (comment)
- Applied the following ksh2020 man page fixes:
  att#351
  att#352
- Fixed a minor GCC -Wformat warning in procopen.c by changing
  NiL to NULL.
JohnoKing added a commit to JohnoKing/ksh that referenced this pull request Aug 6, 2020
Most of these fixes are for typos and extra whitespace at the
end of lines. These are the notable changes:
- Fixed a compatibility issue with how asterisks are displayed
  using certain fonts. Bug report: att#764
- Fixed a bug in the man page that caused searches for the '|'
  character to fail. Bug report: att#871
- Removed a duplicate description of 'set -B' from the man
  page. Bug report: att#789
- Added documentation for options missing from the ksh man
  page (applies to 'hist -N', 'sleep -s', 'whence -q' and
  many of ulimit's options). Bug reports:
  att#948
  att#503 (comment)
  att#507 (comment)
- Applied the following ksh2020 man page fixes:
  att#351
  att#352
- Fixed a minor GCC -Wformat warning in procopen.c by changing
  a sentinel NULL.
JohnoKing added a commit to JohnoKing/ksh that referenced this pull request Aug 6, 2020
Most of these fixes are for typos and extra whitespace at the
end of lines. These are the notable changes:
- Fixed a compatibility issue with how asterisks are displayed
  using certain fonts. Bug report: att#764
- Fixed a bug in the man page that caused searches for the '|'
  character to fail. Bug report: att#871
- Removed a duplicate description of 'set -B' from the man
  page. Bug report: att#789
- Added documentation for options missing from the ksh man
  page (applies to 'hist -N', 'sleep -s', 'whence -q' and
  many of ulimit's options). Bug reports:
  att#948
  att#503 (comment)
  att#507 (comment)
- Applied the following ksh2020 man page fixes:
  att#351
  att#352
- Fixed a minor GCC -Wformat warning in procopen.c by changing
  a sentinel NULL.
JohnoKing added a commit to JohnoKing/ksh that referenced this pull request Aug 6, 2020
Most of these fixes are for typos and extra whitespace at the
end of lines. These are the notable changes:
- Fixed a compatibility issue with how asterisks are displayed
  using certain fonts. Bug report: att#764
- Fixed a bug in the man page that caused searches for the '|'
  character to fail. Bug report: att#871
- Removed a duplicate description of 'set -B' from the man
  page. Bug report: att#789
- Added documentation for options missing from the ksh man
  page (applies to 'hist -N', 'sleep -s', 'whence -q' and
  many of ulimit's options). Bug reports:
  att#948
  att#503 (comment)
  att#507 (comment)
- Applied the following ksh2020 man page fixes:
  att#351
  att#352
- Fixed a minor GCC -Wformat warning in procopen.c by changing
  a sentinel NULL.
JohnoKing added a commit to JohnoKing/ksh that referenced this pull request Aug 6, 2020
Most of these fixes are for typos and extra whitespace at the
end of lines. These are the notable changes:
- Fixed a compatibility issue with how asterisks are displayed
  using certain fonts. Bug report: att#764
- Fixed a bug in the man page that caused searches for the '|'
  character to fail. Bug report: att#871
- Removed a duplicate description of 'set -B' from the man
  page. Bug report: att#789
- Added documentation for options missing from the ksh man
  page (applies to 'hist -N', 'sleep -s', 'whence -q' and
  many of ulimit's options). Bug reports:
  att#948
  att#503 (comment)
  att#507 (comment)
- Applied the following ksh2020 documentation fixes:
  att#351
  att#352
- Fixed a minor GCC -Wformat warning in procopen.c by changing
  a sentinel NULL.
JohnoKing added a commit to JohnoKing/ksh that referenced this pull request Aug 6, 2020
Most of these fixes are for typos and extra whitespace at the
end of lines. These are the notable changes:
- Fixed a compatibility issue with how asterisks are displayed
  using certain fonts. Bug report: att#764
- Fixed a bug in the man page that caused searches for the '|'
  character to fail. Bug report: att#871
- Removed a duplicate description of 'set -B' from the man
  page. Bug report: att#789
- Added documentation for options missing from the ksh man
  page (applies to 'hist -N', 'sleep -s', 'whence -q' and
  many of ulimit's options). Bug reports:
  att#948
  att#503 (comment)
  att#507 (comment)
- Applied the following ksh2020 documentation fixes:
  att#351
  att#352
- Fixed a minor GCC -Wformat warning in procopen.c by changing
  a sentinel to NULL.
McDutchie pushed a commit to ksh93/ksh that referenced this pull request Aug 6, 2020
Most of these fixes are for typos and extra whitespace at the
end of lines. These are the notable changes:
- Fixed a compatibility issue with how asterisks are displayed
  using certain fonts. Bug report: att#764
- Fixed a bug in the man page that caused searches for the '|'
  character to fail. Bug report: att#871
- Removed a duplicate description of 'set -B' from the man
  page. Bug report: att#789
- Added documentation for options missing from the ksh man
  page (applies to 'hist -N', 'sleep -s', 'whence -q' and
  many of ulimit's options). Bug reports:
  att#948
  att#503 (comment)
  att#507 (comment)
- Applied the following ksh2020 documentation fixes:
  att#351
  att#352
- Fixed a minor GCC -Wformat warning in procopen.c by changing
  a sentinel to NULL.
citrus-it pushed a commit to citrus-it/ast that referenced this pull request Dec 4, 2021
This bug was first reported in <https://www.illumos.org/issues/7694>.
The time keyword currently overrides the errexit shell option,
allowing failing scripts to continue after an error:

  $ cat 1.sh
  #!/bin/sh
  time false   # This should cause the script to exit
  echo FAILURE
  true
  $ ksh -o errexit 1.sh

  real    0m0.00s
  user    0m0.00s
  sys     0m0.00s
  FAILURE

src/cmd/ksh93/sh/xec.c:
- When the time keyword runs a command, pass the errexit state flag
  to the sh_exec call. This state flag is required for ksh to exit
  when a command fails while the errexit option is on.

src/cmd/ksh93/tests/basic.sh:
- Add a regression test based on the reproducer.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants