| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
| |
http://lists.freebsd.org/pipermail/freebsd-current/2006-March/061725.html
The src.conf(5) manpage is to follow in a few days.
Brought to you by: imp, jhb, kris, phk, ru (all bugs are mine)
|
|
|
|
|
|
|
| |
comment listing the file name, owner, and group.
MFC after: 1 week
Submitted by: Jan Srzednicki <w at expro dot pl>
|
|
|
|
|
|
| |
mdoc(7) according to ru.
Requested by: ru
|
|
|
|
|
|
| |
by pjd.
MFC after: 3 weeks
|
|
|
|
|
|
|
|
|
| |
output-format keyword, and the keyword they picked is an alias to
some other keyword. E.g.: ps -o stat=Zustand $$
('stat' is defined as an alias for 'state')
PR: bin/57833
MFC after: 3 weeks
|
|
|
|
| |
Discussed with: ru
|
|
|
|
|
|
| |
Mention that the setting of securelevel may affect one's ability to alter flags.
Xref security.7.
Bump doc date.
|
|
|
|
|
|
| |
up always printing "pts".
Submitted by: Michal Mertl <mime at traveller dot cz>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The tcsh 6.14 uses libiconv to convert catalogs to appropriate charset
dynamically. However, we don't have libiconv in our tree. So, I made
some hack to load libiconv dynamically. If libiconv is available, you
can use catalogs for more locales than the locales which catalog is
actually installed.
To use this feature, you need to symlink catalogs to appropriate
locales. You can do this by installing ports/shells/tcsh_nls.
Reviewed by: arch (no objection)
MFC after: 1 week
|
|
|
|
|
|
|
| |
creating symbolic links.
PR: bin/92149
Submitted by: Eugene Grosbein <eugen grosbein.pp.ru>
|
|
|
|
|
|
| |
environment or set it only when changing directories with cd).
PR: standards/92640
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
| |
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@
|