Age | Commit message (Collapse) | Author |
|
OK deraadt@
|
|
other edge cases found by gsoares@. OK tb@ gsoares@
|
|
when ksh is compiled with -Os (done in distrib/special): move the
bit using XPptrv(l) (which expands to l.beg) where it is actually
used and clearly initialized.
ok tedu, tweak + ok millert
|
|
|
|
make commands starting with the escape key (^[) work as documented.
Long-standing bug, this time reported by Dave minus Cohen dot com.
OK natano@ halex@
|
|
that i misunderstood something when i wrote it;
found by guenther and naddy
|
|
with help from otto
|
|
this before previous release, and I plain forgot about it.
|
|
from Andras Farkas <deepbluemistake at gmail dot com>.
|
|
|
|
traditional mode. This aligns ksh's behavior with bash and FreeBSD sh.
The interpretation of the POSIX text is disputed, but it is unlikely
that a change from the traditional behavior was intended.
ok millert@
|
|
correctly - logically complete that now by removing MLINKS from base;
authors need only to ensure there is an entry in NAME for any function/
util being added. MLINKS will still work, and remain for perl to ease
upgrades;
ok nicm (curses) bcook (ssl)
ok schwarze, who provided a lot of feedback and assistance
ok tb natano jung
|
|
This simple patch makes the 'command' builtin POSIX-compliant and
consistent with other current shells. It fixes two things:
a) 'command -v' does not find shell reserved words (a.k.a. keywords).
For instance, 'command -v select' outputs nothing but should output
'select'.
b) 'command -pv' always outputs the path of an external command, even
if 'command -p' would execute a builtin. For instance, 'command -p kill'
executes the 'kill' builtin, as expected, but 'command -pv kill' outputs
'/bin/kill'. The '-v' option is supposed to reflect what would actually
be executed, so 'command -pv kill' should output 'kill'. The -p option
sets the PATH to a default system value before doing the search, but
that has no bearing on the fact that builtins take precedence over
external commands.
The patch fixes both issues for 'command' without affecting the
behaviour of the ksh-specific builtin 'whence', which is handled by the
same C function.
Regression test added to obsd-regress.t.
Issues found and fixed by Martijn Dekker, ok millert@
|
|
idioms.
Adopt the more concise fcntl(n, F_GETFL) over fcntl(n, F_GETFL, 0)
where it is obvious further investigation will not yield and
even better way.
Obviousness evaluation and ok guenther@
|
|
|
|
|
|
It should be possible to use set +o to save the current shell options,
then modify them at will and later restore them to the saved value.
Therefore the output must not only contain the options which are
currently set, but also the ones that are _not_ set, so a restore
disables them if they are enabled.
from Martijn Dekker (martijn (at) inlv (dot) org)
ok sthen@, tb@
|
|
All work done by Martijn Dekker
OK millert@
|
|
is "getconf PATH" not "getconf CS_PATH".
|
|
accelerate install/upgrade time MAKEDEV runs. That allows ramdisk ksh
to be pledged, without needing "dpath". We'll solve the performance issues
a different way (soon).
ok otto espie natano tb tobiasu rpe
|
|
fixes posix compatibility issue
OK millert@, nicm@, otto@
|
|
from ray@, ok jmc@
|
|
ok nicm@
|
|
instead of scattering their initialization through the body. This makes
the body simpler and avoids an uninitialized use false positive.
ok nicm@
|
|
1. Improve all functions involving words by allowing non-ASCII
characters to be part of words.
2. Allow insertion of non-ASCII characters without screwing up the
display, by backing up to the start byte after inserting a continuation
byte, and starting to re-print there.
3. Fix forward movement which i didn't get quite right in my previous
commit: Always advance to a start byte, never to a final continuation
byte, or the next insertion would split the character in the middle.
OK mpi@
|
|
ok bmercer gsoares nicm sthen
|
|
ok millert nicm tb
|
|
years.
From Michael Reed. ok halex@
|
|
ok halex@
|
|
resilient against being altered by a subsequent shift operation
tweak and ok semarie@
|
|
|
|
|
|
"seems sound" deraadt@
|
|
ok mmcc@ a while ago
|
|
|
|
right can only move by whole characters, not into the middle of a
character, and that deleting characters can only delete characters
whole, not individual bytes out of characters.
Based on parts of a patch by Frederic Nowak <fnwk at mailbox dot org>,
tweaked by me.
OK tedu@ semarie@ mpi@
|
|
|
|
longer exist.
|
|
|
|
|
|
|
|
|
|
halex@, sthen@, jmc@.
|
|
bad writing style. Also, add a paragraph break and split up a
megasentence.
|
|
context.
|
|
upstream.
|
|
|
|
ok mmcc@
|
|
|
|
from Ricardo Mestre
|