From: "Bart Schaefer" <schaefer@candle.brasslantern.com>
To: zsh-workers@sunsite.auc.dk
Subject: PATCH: Parameter STTY should work as documented
Date: Mon, 25 Sep 2000 16:40:52 +0000 [thread overview]
Message-ID: <1000925164052.ZM11699@candle.brasslantern.com> (raw)
STTY
If this parameter is set in a command's environment, the shell
runs the stty command with the value of this parameter as
arguments in order to set up the terminal before executing the
command. [...] STTY is ignored if the command is run in the
background, or if it is in the environment of the shell but not
explicitly assigned to in the input line. This avoids running stty
at every external command by accidentally exporting it. [...]
A patch for 3.0.8 will go up on the SourceForge patch manager soon.
Index: Src/exec.c
===================================================================
@@ -133,6 +133,7 @@
static LinkList args;
static int doneps4;
+static char *STTYval;
/* Execution functions. */
@@ -444,7 +445,6 @@
execute(Cmdnam not_used_yet, int dash)
{
Cmdnam cn;
- static LinkList exargs;
char buf[MAXCMDLEN], buf2[MAXCMDLEN];
char *s, *z, *arg0;
char **argv, **pp;
@@ -459,17 +459,19 @@
/* If the parameter STTY is set in the command's environment, *
* we first run the stty command with the value of this *
* parameter as it arguments. */
- if (!exargs && (s = zgetenv("STTY")) && isatty(0) &&
- (GETPGRP() == getpid())) {
- char *t;
+ if ((s = STTYval) && isatty(0) && (GETPGRP() == getpid())) {
+ LinkList exargs = args;
+ char *t = tricat("stty", " ", s);
- exargs = args; /* this prevents infinite recursion */
+ STTYval = 0; /* this prevents infinite recursion */
+ zsfree(s);
args = NULL;
- t = tricat("stty", " ", s);
execstring(t, 1, 0);
zsfree(t);
args = exargs;
- exargs = NULL;
+ } else if (s) {
+ STTYval = 0;
+ zsfree(s);
}
cn = (Cmdnam) cmdnamtab->getnode(cmdnamtab, arg0);
@@ -1537,6 +1539,10 @@
state->pc = opc;
return;
}
+ if (strcmp(name, "STTY") == 0) {
+ zsfree(STTYval);
+ STTYval = ztrdup(val);
+ }
allexp = opts[ALLEXPORT];
opts[ALLEXPORT] = 1;
pm = setsparam(name, val);
@@ -2326,6 +2332,10 @@
if (!forked)
setlimits(NULL);
#endif
+ if (how & Z_ASYNC) {
+ zsfree(STTYval);
+ STTYval = 0;
+ }
execute((Cmdnam) hn, cflags & BINF_DASH);
} else { /* ( ... ) */
DPUTS(varspc,
--
Bart Schaefer Brass Lantern Enterprises
http://www.well.com/user/barts http://www.brasslantern.com
Zsh: http://www.zsh.org | PHPerl Project: http://phperl.sourceforge.net
reply other threads:[~2000-09-25 16:42 UTC|newest]
Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1000925164052.ZM11699@candle.brasslantern.com \
--to=schaefer@candle.brasslantern.com \
--cc=zsh-workers@sunsite.auc.dk \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.vuxu.org/mirror/zsh/
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).