| Age | Commit message (Collapse) | Author |
|---|
|
mode, to avoid potential confusion pointed out by andreas dot kahari
at icm dot uu dot se on misc@.
Patch minimally tweaked based on a comment from jmc@.
OK anton@ jca@ jmc@ on the previous version without the tweak.
|
|
and NUL
ok millert krw tb
|
|
|
|
|
|
More informative than the usage message.
While here, no need to return EINVAL: just return 1 if we overflow.
ok jca@ schwarze@
|
|
Makes us compliant with POSIX.1-2008, which requires that sleep(1) support
up to 2147483647 seconds.
Bounced off of tb@ and jca@.
ok tb@ millert@ jca@
|
|
also took the time to provide an excellent explanation on why the code isn't
useful on tech@.
ok jca@
|
|
This helps tools like scan-build, and follows the example of warningf()
and errorf(). ok anton@
|
|
it's rarely subject to change. While here, unifdef RLIMIT_VMEM.
ok jca@
|
|
emacs.c:1041:2: warning: Value stored to 'cp' is never read
|
|
str_val returns null, not NULL, if the variable isn't set. The
erroneous check means that we later tried to open("").
ok millert@ tb@ anton@ benno@
|
|
ok anton@
|
|
|
|
patch from Michael W. Bombardieri <mb at ii dot net>;
OK tb@
|
|
- it's safe to assume that there's no plan to write "an `agent' to
execute unreadable/setuid/setgid shell scripts"
- the BUG-REPORTS file has been removed
- update the entry for FPATH, ksh93 also documents the described behavior
- kill entry about octal and hex notation in arithmetic expressions
(supported)
- typos
Feedback from Klemens Nanni
|
|
|
|
from Klemens Nanni, thanks.
|
|
Namely POSIX character classes and tilde expansion within parameter
substitution. While here, kill a useless line and fix a typo.
From Klemens Nanni
|
|
OK anton@ jca@
|
|
Pointed out by Klemens Nanni who had ~ the same diff
|
|
|
|
that fixes the issues seen on hppa. OK deraadt@ otto@
|
|
Prompted by a mail from Klemens Nanni, who also had the same diff.
ok deraadt@ millert@
|
|
from Klemens Nanni
|
|
Requested by deraadt@
|
|
from erroring out with "dd: skip: Undefined error: 0", mostly for
consistency and to avoid the unidiomatic, wrong looking code.
Patch from Bulat Musin <mbulatka at yandex dot ru>.
OK guenther@ millert@
|
|
OK tb@
|
|
from kshe
|
|
the warnings are due to the use of globals with generic names,
specifically "options" and "path". I've renamed "options" to
"sh_options" since it holds the shell options and "path" to
"search_path". OK jca@ tb@
|
|
long time ago. Update the comments and replace test_stat() calls
with stat() since test_stat() now just calls stat() directly. Also
rename the "mode" parameter to "amode" in test_eaccess() to match
access(2) and make it clear that this is the access mode and not
the file mode. OK jca@
|
|
While here, use err(3) instead of errx(3) if adjtime(2) fails.
Discussed/tweaked with/by tb@ and jca@.
ok tb@ jca@
|
|
it out prior to continuing down the recursive call chain.
Initial diff from Michael W. Bombardieri, later tweaked; ok tb@
|
|
redundant call to free() since name is always freed upon return.
From Michael W. Bombardieri, with some tweaks by myself.
|
|
ok jca@
|
|
|
|
|
|
Discussed with and ok jmc
|
|
With tmux, ssh and vmd, we tend to open shells on many different hosts
simultaneously and the default prompts '$ ' and '# ' for {,k}sh as well
as '% ' and '# ' for csh become dangerous: it's very easy to issue a
command on the wrong host.
This can easily be avoided by displaying the hostname in the prompt.
Everything beyond "hostname{$,#,%} " is going to be a matter of taste,
so we left it at that. If you use an FQDN, only the first part (the
output of 'hostname -s') will be printed.
requested by and ok deraadt; mostly positive feedback many
ok anton, brynet, bcallah and others
|
|
provides retry on short-write file descriptors.
ok florian, previous versions seen by millert
|
|
early special casing of ! and !! in the PS1 expansion. This was removed
from set_prompt() as part of the implementaion of the character count
toggles \[ and \] back in 2004.
ok jca
|
|
ok jca, anton (who both had the same diff) and deraadt
|
|
Not bound by default and not very useful. ok schwarze@ anton@
|
|
- Keep the order of bindings in sync between the manual and implementation
- Fix wrongly documented bindings in the manual
- Break out commands without a default binding in the manual
ok jmc@ tb@
|
|
found by anton. To reproduce, run "env EDITOR=emacs MALLOC_OPTIONS=J ksh"
then press "^[_".
ok anton, jca
|
|
argument but I forgot to adapt one conditional. Still no functional change since
tenex() only has one call site where `inputline_size == sizeof(buf)`.
|
|
|
|
by the plaintext history diff, but the code that may have depended on
this was since removed from history.c. None of the code introduced
between the plaintext history commit and now depends on calloc(3).
This way we can again use malloc.conf(5)'s J option to recognize use
of uninitialized memory.
ok jca
|
|
exit with a non-zero EXIT STATUS, as required by POSIX,
consistent with what other operating systems do, consistent
with how other OpenBSD programs behave, and making wait(2)
consistent with what happens when dd(1) dies from other signals.
Patch from <ScottCheloha at gmail dot com>.
OK florian@ tb@.
|
|
We may have read two bytes from the file, but history_load() needs to
call rewind(3) anyway. ok benno@
|
|
With much input from Ori Bernstein and anton@; ok anton@ (and benno@ on
a previous version).
|
