| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
| |
value < 0. errno is only updated in this case. Change all (most?)
callers of syscalls to follow this better, and let's see if this strictness
helps us in the future.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
never do substitution (neither parameter, nor command, nor arithmetic,
nor tilde substitution) on the values of any variables encountered
inside the expression, but do recursively perform arithmetical
evaluation of subexpressions as required. This makes behaviour
more consistent, without hindering any behaviour promised in the
manual page.
A quirk originally reported by Andy Chu <andychup at gmail dot com>
was that in the past, when encountering an array index, the shell
would not only do evaluation, but also substitution on the array
index, even though substitution would not be done on the expression
in general.
tobias@ contributed to initial efforts of understanding the quirk.
patch tested in a bulk build by naddy@
"please commit" deraadt@
|
| |
|
|
|
|
| |
a missing NULL check added by jca@ which fixes the segfault in the installer.
ok jca@ tb@
|
| |
|
|
|
|
| |
install.sub's ${*:+$*} substitution in addel().
ok jca
|
| |
|
|
|
|
|
|
|
|
|
| |
variables; such as positional arguments. Since global() returns a pointer to
static storage for read-only variables, the memory pointed to needs to be copied
to prevent any subsequent call to global() to override the previously accessed
variable.
Bug reported by Andreas Kusalananda Kähäri on bugs@
ok benno@ jca@ tb@
|
| |
|
|
|
|
|
| |
No binary change on amd64 and there should be no differences on any
other 64 bit architecture either (because long = int64_t).
ok cheloha, tb
|
| |
|
|
|
| |
This helps tools like scan-build, and follows the example of warningf()
and errorf(). ok anton@
|
| | |
|
| |
|
|
| |
that fixes the issues seen on hppa. OK deraadt@ otto@
|
| |
|
|
| |
Requested by deraadt@
|
| |
|
|
| |
OK tb@
|
| |
|
|
|
| |
Report and diff by David Crosby <dave@dafyddcrosby.com>
OK otto@
|
| |
|
|
| |
and is not an easy fix for now.
|
| |
|
|
|
|
| |
borrowed from ksh93.
Survived a ports build performed by naddy@ and encouraged by many.
|
| |
|
|
| |
warning from clang.
|
| |
|
|
|
|
| |
All work done by Martijn Dekker
OK millert@
|
| |
|
|
| |
ok millert nicm tb
|
| |
|
|
| |
ok mmcc@ a while ago
|
| |
|
|
| |
ok nicm@. Also discussed with millert@ and guenther@.
|
| |
|
|
|
|
|
|
| |
"The ^ is used in regular expressions and many versions of fnmatch(3)
accept both ! and ^. However, we are never going to accept ^ instead of
! so I think this makes sense" -millert@
"go for it" -nicm@
|
| |
|
|
| |
ok nicm@
|
| |
|
|
| |
ok nicm@
|
| |
|
|
| |
change.
|
| | |
|
| |
|
|
| |
ok md5
|
| |
|
|
|
| |
actually should inherit "set -e" status according to POSIX.
OK jca@
|
| |
|
|
|
|
| |
wrt. errors (do not unwind and do not treat this as fatal if set -e is
used). This matches what bash does. Tweak regress tests while here.
ok millert@, jasper@ agrees
|
| |
|
|
|
|
| |
status. We can't use XERROK for this (since the command might set
-e itself) so just save & restore the value of FERREXIT for the
comsub() call to execute(). OK jca@
|
| | |
|
| |
|
|
| |
noticed by and ok kevlo@
|
| |
|
|
|
|
| |
when they shouldn't. originally from mksh but modified to handle the case
when the expr is quoted as noticed by halex@.
input from guenther@ and halex@, millert@ ok
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
match what both POSIX and ksh.1 already describe in regards to set
-e/errexit's behavior in determining when to exit from nonzero return
values.
specifically, the truth values tested as operands to `&&' and `||', as
well as the resulting compound expression itself, along with the truth
value resulting from a negated command (i.e. a pipeline prefixed `!'),
should not make the shell exit when -e is in effect.
issue reported by matthieu.
testing matthieu, naddy.
ok miod (earlier version), otto.
man page ok jmc.
|
| |
|
|
| |
prompted by deraadt@
|
| | |
|
| |
|
|
| |
millert@ ok
|
| | |
|
| |
|
|
|
| |
behavior the default. Almost all uses of savefd() are followed
by an implicit or explicit close. OK otto@
|
| | |
|
| |
|
|
| |
double checked to make sure no binaries change, and eyed by niallo
|
| | |
|
| |
|
|
|
|
|
| |
expressions. Work by Matthias Kilian, based on an old diff by myself.
Note: MAKEDEV should be updated. Tested by many, thanks.
ok millert@ deraadt@
|
| | |
|
| |
|
|
| |
Bug noticed by otto@.
|
| | |
|
| |
|
|
|
| |
sparc64, binary change in lex.o on macppc due to introduction of
dopprompt() prototype. ok millert@
|
| | |
|
| |
|
|
| |
First step in making the ksh code easier to read. From Matthias Kilian
|
| |
|
|
| |
the actual expression that caused the error. OK otto@
|
| | |
|
| |
|
|
| |
and OK otto@. Closes PR 4023.
|