| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When a child is receiving SIGSTOP, eval continues with the next
command. While that is correct for the interactive case (Control-Z
and you get the prompt back), it is wrong for a shellscript, which
just continues with the next command, never again waiting for the
stopped child. Noted when childs from cronjobs were stopped, just to
make more processes (by wosch).
The fix is not to return from a job wait when the wait returned for a
stopped child while in non-interactive mode. This bahaviour seems to
be what bash2 and ksh implement. I tested for correct behaviour for
finnaly killing the child with and without forgrounding it first.
When not foregrouding before killing, the shell continues with the
script, which is what the other shells do as well.
Reviewed by: Silence on -current
|
| |
|
| |
|
| |
|
| |
|
|
|
|
| |
Use _PATH_* where where possible.
|
| |
|
|
|
|
| |
Audited by: markm, cracauer
|
| |
|
|
|
|
|
|
|
|
| |
Serious fix still needed, see discussion on -current
(Subject: /bin/sh dumps core with here-document of 8bit text)
Problem in this code originally spotted by
Jun Kuriyama <kuriyama@FreeBSD.org>
|
|
|
|
|
|
|
|
|
| |
names in them.
Also use a colon in the answer of `type` everytime the questioned item
is not usable.
PR: bin/20567
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
growstackblock() sometimes relocates a stack_block considered empty
without properly relocating stack marks referencing that block.
The first call to popstackmark() with the unrelocated stack mark
as argument then causes sh to abort.
Relocating the relevant stack marks seems to solve this problem.
The patch changes the semantics of popstackmark() somewhat. It can
only be called once after a call to setstackmark(), thus cmdloop() in
main.c needs an extra call to setstackmark().
PR: bin/19983
Submitted by: Tor.Egge@fast.no
Reviewed by: Gerald Pfeifer <pfeifer@dbai.tuwien.ac.at>
|
| |
|
| |
|
|
|
|
| |
Reviewed by: Matthew Hunt <mph@astro.caltech.edu>
|
|
|
|
|
| |
PR: bin/19475
Approved by: Martin Cracauer <cracauer@cons.org> (with conditions)
|
| |
|
|
|
|
|
| |
PR: 18447
Submitted by: Koji Mori <mori@tri.asanuma.co.jp>
|
|
|
|
|
|
| |
PR: bin/6577
Submitted by: Anatoly Vorobey <mellon@pobox.com>
Approved by: silence amoung other sh experts
|
|
|
|
|
|
|
|
|
|
|
| |
case), so that it doesn't clash with the ncurses function of the same
name when linking statically with -ltermcap.
The linker only complains when -static is used, and it is not clear
whether this is a bug.
PR: bin/18104
Submitted by: Anatoly Vorobey <mellon@pobox.com>
|
|
|
|
|
| |
PR: bin/18104
Submitted by: mellon@pobox.com
|
|
|
|
|
|
|
|
| |
spaces reserved by the header files it includes.
mkinit.c still produces C code with redundant declarations, although
they are more harmless since they automatically derived from the right
places.
|
| |
|
|
|
|
|
|
|
| |
Approved-by: jkh
PR: bin/12137
Submitted by: "Danny J. Zerkel" <dzerkel@columbus.rr.com>
|
|
|
|
|
|
| |
PATH=... command
Noted by and fix works for Marcel Moolenaar <marcel@scc.nl>
|
| |
|
| |
|
| |
|
|
|
|
|
| |
PR: docs/15265
Submitted by: takamune@avrl.mei.co.jp
|
|
|
|
| |
discussed on -arch.
|
|
|
|
| |
Fix a warning.
|
|
|
|
|
|
| |
in that revision as well as things I broke in that revision. A note-
worthy instance of the latter case was the inversion of -E and -V in the
subsection on Commandline Editing.
|
|
|
|
|
|
|
| |
1.23. This revision should work for long pipes both in
here-documents/backquote and in normal cases.
Fix works for jmz, bde.
|
|
|
|
|
|
|
| |
example of such a clone.
PR: 14601
Submitted by: Matthias Buelow <mkb@altair.mayn.de>
|
|
|
|
|
|
| |
don't have their stdin closed indisciminantly.
Reviewed by: markm
|
|
|
|
|
|
|
|
| |
than two processes (got that? :-), the stdin fd of the middle
processes that has just been set up was accidetially closed. Don't do
this.
PR: bin/14527
|
|
|
|
| |
PR: bin/14509
|
| |
|
|
|
|
| |
Reviewed by: mpp
|
|
|
|
|
|
|
|
| |
commit and those which cause ugly nroff output have been fixed, since
the purpose of the style guideline which they contravene is to reduce
the sizes of deltas.
Reported by: bde
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Consistently misspell built-in as builtin.
* Add a builtin(1) manpage and create builtin(1) MLINKS for all shell
builtin commands for which no standalone utility exists. These MLINKS
replace those that were created for csh(1).
* Add appropriate xrefs for builtin(1) to the csh(1) and sh(1) manpages,
as well as to the manpages of standalone utilities which are supported
as shell builtin commands in at least one of the shells. In such
manpages, explain that similar functionality may be provided as a
shell builtin command.
* Improve sh(1)'s description of the cd builtin command. Csh(1) already
describes it adequately. Replace the cd(1) manpage with a builtin(1)
MLINKS link.
* Clean up some mdoc problems: use Xr instead of literal "foo(n)"; use
Ic instead of Xr for shell builtin commands.
* Undo English contractions.
Reviewed by: mpp, rgrimes
|
|
|
|
|
|
| |
but that doesn't mean that they will never work.
Requested by: mpp, rgrimes
|
|
|
|
|
|
|
|
|
|
| |
Fix grammar and spelling nits.
Use .Dq and .Qq where appropriate.
Divorce trailing punctuation from quoted elements.
Use .Dq instead of .Xr for builtins.
Remove trailing whitespace and blank lines.
PR: 13340
|
| |
|
|
|
|
|
|
|
| |
the default. Add -r option for the read builtin to reverse this.
PR: 13274
Reviewed by: cpiazza, hoek, sheldonh
|
|
|
|
|
|
|
|
| |
which it should not do.
PR: 12578
Reported by: Gerald Pfeifer <pfeifer@dbai.tuwien.ac.at>
Submitted by: Niall Smart <niall@pobox.com>
|
|
|
|
| |
Submitted by: Philippe Charnier <charnier@xp11.frmug.org>
|
|
|
|
|
|
|
|
|
|
| |
statement if blocks[*] when the else could be ambiguous, not defaulting
to int type and removal of some unused variables.
[*] This is explicitly allowed by style(9) when the single statement
spans more than one line.
Reviewed by: obrien, chuckr
|
|
|
|
| |
PR: bin/9173
|
| |
|