| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
| |
PR: 85103
Submitted by: garys
Obtained from: pdksh manual
Patch from: Daniel Gerzo (with changes by me)
|
|
|
|
|
|
|
|
|
|
|
|
| |
automatically it is possible wait4(2) returns -1 and sets
errno = ECHILD if there were forked children. A user can
set such signal handler e.g. via ``trap "" 20'', see a PR
for the test case. Deal with this case and mark a job as
JOBDONE.
PR: bin/90334
Submitted by: bde
MFC after: 4 weeks
|
| |
|
|
|
|
|
|
|
|
| |
dash.
- Discourage the omission of the action.
PR: 70985 [1]
Submitted by: Martin Kammerhofer
|
|
|
|
|
|
|
| |
This makes a difference for the trap builtin, where after "trap '' 0" we
printed "trap -- quit". This is wrong, because an empty action means to reset
the action to the default. A side effect of this commit is that empty
variables are now printed as "variable=''" instead of just "variable=".
|
|
|
|
|
| |
PR: 74043
Submitted by: Jilles Tjoelker
|
|
|
|
| |
Mention that spaces are printed between the strings.
|
| |
|
|
|
|
| |
implementing them were never part of FreeBSD.
|
|
|
|
|
|
| |
itself and its children. Instead of calling times() (as implied by POSIX) this
implementation directly calls getrusage() to get the times because this is more
convenient.
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
files is too much and hard to follow. Instead, make the -I option
just mean "do not automatically set -A for root". That is, if -A
is explicitly set, -I is ignored. Also, document -I in usage().
(The ls.c diff is better viewed relative to rev. 1.80.)
No objection: mux
Silence from: mnag
MFC after: 3 days
|
|
|
|
|
| |
Spotted by: ru
MFC after: 3 days
|
|
|
|
|
|
| |
PR: bin/86710
Submitted by: Marcus Alves Grando
MFC after: 3 days
|
|
|
|
|
| |
Submitted by: garys
Approved by: trhodes (mentor)
|
|
|
|
|
|
|
| |
Xref locale(1) and bump document date.
Reviewed by: keramida
No Objections: -standards
|
|
|
|
|
|
|
|
|
|
|
|
| |
set +o can be used to reload previous settings, for this to work disabled
options must be printed as well or otherwise options that were set in the mean
time won't be turned off.
To avoid an excessively long output line I formatted the output to print only
six options per line.
Submitted by: Jilles Tjoelker
PR: 73500
|
|
|
|
| |
Glanced at by: simon
|
| |
|
|
|
|
|
|
|
| |
type of their argument, if it is a shell function, an alias, a builtin, etc.
-V is more verbose than -v.
PR: 77259, 84539
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
ckrealloc and ckfree (added), respectively. sh jumps out of the signal handler
using longjmp which is obviously a bad idea during malloc calls.
Note: I think there is still a small race here because volatile sig_atomic_t
only guarantees atomic reads and writes while we're doing increments and
decrements.
Protect a setmode call with INT{ON,OFF} as it calls malloc internally.
PR: 45478
Patch from: Nate Eldredge
|
|
|
|
|
|
| |
size.
PR: 86355
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
earlier, and more gracefully. Previously, this combination would be
ignored early in the code where permissions are tested and fail later
with a very unhelpful "permission denied" error.
Instead, test for this flag in the same block that generates the
"override?" messages for read-only files, but instead of trying
to guess what the user has in mind, generate an error and exit.
Update the man page to reflect this new behavior.
Not objected to by: freebsd-hackers@
|
|
|
|
|
|
| |
ports builds in the cluster to work correctly.
Tested by: kris
|
|
|
|
| |
MFC after: 1 week
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
termination with set -e if a command fails in a loop body inside a function
with an explicitely tested exit status, eg
f() {
for i in 1 2 3; do
false
done
}
f || true
Briefly reviewed by: cracauer
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
two cases of unwanted termination with set -e:
* if-commands containing several commands separated by semicolons, eg
if false; false; then [...]
* functions with an explicitely tested exit status that contain a failing
command which is not the last one, eg
f() {
false
false
}
f || true
PR: 77067, 85267
Briefly reviewed by: cracauer
|
|
|
|
|
|
|
| |
active. Use a separate flag to avoid adding another condition to the
if-statement at the end of evaltree().
Briefly reviewed by: cracauer
|
|
|
|
| |
names.
|
|
|
|
| |
- Use argv rather than argptr since getopt() is used here.
|
|
|
|
| |
non-digits.
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
and linting procedure:
1. Remove useless sub-expression:
- if (*start || (!ifsspc && start > string && (nulonly || 1))) {
+ if (*start || (!ifsspc && start > string)) {
The sub-expression "(nulonly || 1)" always evaluates to true and
according to CVS logs seems to be just a left-over from some
debugging and introduced by accident. Removing the sub-expression
doesn't change semantics and a code inspection showed that the
variable "nulonly" is also not necessary here in any way (and the
expression would require fixing instead of removing).
2. Remove dead code:
- if (backslash && c == '\\') {
- if (read(STDIN_FILENO, &c, 1) != 1) {
- status = 1;
- break;
- }
- STPUTC(c, p);
- } else if (ap[1] != NULL && strchr(ifs, c) != NULL) {
+ if (ap[1] != NULL && strchr(ifs, c) != NULL) {
Inspection of the control and data flow showed that variable
"backslash" is always false (0) when the "if"-expression is
evaluated, hence the whole block is effectively dead code.
Additionally, the skipping of characters after a backslash is already
performed correctly a few lines above, so this code is also not
needed at all. According to the CVS logs and the ASH 0.2 sources,
this code existed in this way already since its early days.
3. Cleanup Style:
- ! trap[signo][0] == '\0' &&
+ ! (trap[signo][0] == '\0') &&
The expression wants to ensure the trap is not assigned the empty
string. But the "!" operator has higher precedence than "==", so the
comparison should be put into parenthesis to form the intended way of
expression. Nevertheless the code was effectively not really broken
as both particular NUL comparisons are semantically equal, of course.
But the parenthesized version is a lot more intuitive.
4. Remove shadowing variable declaration:
- char *q;
The declaration of symbol "q" hides another identical declaration of
"q" in the same context. As the other "q" is already reused multiple
times and also can be reused again without negative side-effects,
just remove the shadowing declaration.
5. Just small cosmetics:
- if (ifsset() != 0)
+ if (ifsset())
The ifsset() macro is already coded by returning the boolean result
of a comparison operator, so no need to compare this boolean result
again against a numerical value. This also aligns the macros usage to
the remaining existing code.
Reviewed by: stefanf@
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This includes adding support for ACLs into cp(1) and mv(1) userspace
utilities.
For mv(1), if _PC_ACL_EXTENDED is in effect for the source AND destination
operands, the destination file's ACLs shall reflect the source.
For cp(1), if _PC_ACL_EXTENDED is in effect for both source and destination
operands, and -p has been specified, the ACLs from the source shall be
preserved on the destination.
MFC after: 1 month
|
| |
|
|
|
|
| |
one.
|
| |
|
| |
|
|
|
|
|
|
|
| |
PR: docs/84765
Submitted by: garys
Approved by: keramida
MFC after: 3 days
|
|
|
|
|
| |
Noticed by: simon
Approved by: trhodes (mentor)
|
|
|
|
|
|
|
|
|
|
| |
o Fix the alias syntax lines.
o Refer to the Aliases subsection.
PR: docs/84914
Submitted by: garys
Approved by: trhodes (mentor)
MFC after: 3 days
|
| |
|
| |
|
| |
|
|
|
|
| |
character comes from a string.
|