From: Peter Stephenson <p.stephenson@samsung.com>
To: Eric Freese <ericdfreese@gmail.com>, Zsh Users <zsh-users@zsh.org>
Subject: Re: Anyone familiar with auto-fu.zsh project?
Date: Tue, 12 Jun 2018 12:32:55 +0100 [thread overview]
Message-ID: <20180612113257eucas1p2342f735bcea27bb1009aee1b23c682f0~3ZmPbMrh61839118391eucas1p2G@eucas1p2.samsung.com> (raw)
In-Reply-To: <CAAikoA+93yDK4F-=ogf8HpheVeLG59LeeHGEgk4dCURoGhFZfw@mail.gmail.com>
On Mon, 11 Jun 2018 19:55:32 -0600
Eric Freese <ericdfreese@gmail.com> wrote:
> Do you know if there is any way to get the PID of the process spawned
> for process substitution? It looks like `$!` doesn't hold it as it
> does in bash.
Hmm, this is obviously useful but I'd be loath simply to make $! do this
as the fact there's an asynchronous process involved is a bit hidden and
the mechanism is rather different from creating a background job, so
that could be confusing.
We could add it to the $sysparams hash provided by the zsh/system
module? This already contains some PID information. (It's
not strictly a system parameter but I don't think anyone's
going to hold a gun to my head and tell me this hash can only
return the result of immediate system calls.)
pws
diff --git a/Doc/Zsh/mod_system.yo b/Doc/Zsh/mod_system.yo
index 7f2009b..a27bab4 100644
--- a/Doc/Zsh/mod_system.yo
+++ b/Doc/Zsh/mod_system.yo
@@ -255,6 +255,11 @@ Returns the process ID of the parent of the current process, even in
subshells. Compare tt($PPID), which returns the process ID of the parent
of the main shell process.
)
+item(tt(procsubstpid))(
+Returns the process ID of the last process started for process
+substitution, i.e. the tt(<LPAR())var(...)tt(RPAR()) and
+tt(>LPAR())var(...)tt(RPAR()) expansions.
+)
enditem()
)
enditem()
diff --git a/Src/Modules/system.c b/Src/Modules/system.c
index 9fd4d25..7a4f4ee 100644
--- a/Src/Modules/system.c
+++ b/Src/Modules/system.c
@@ -772,6 +772,8 @@ fillpmsysparams(Param pm, const char *name)
num = (int)getpid();
} else if (!strcmp(name, "ppid")) {
num = (int)getppid();
+ } else if (!strcmp(name, "procsubstpid")) {
+ num = (int)procsubstpid;
} else {
pm->u.str = dupstring("");
pm->node.flags |= PM_UNSET;
@@ -805,6 +807,8 @@ scanpmsysparams(UNUSED(HashTable ht), ScanFunc func, int flags)
func(&spm.node, flags);
fillpmsysparams(&spm, "ppid");
func(&spm.node, flags);
+ fillpmsysparams(&spm, "procsubstpid");
+ func(&spm.node, flags);
}
static struct mathfunc mftab[] = {
diff --git a/Src/exec.c b/Src/exec.c
index 963b0a5..d445278 100644
--- a/Src/exec.c
+++ b/Src/exec.c
@@ -174,6 +174,11 @@ mod_export int zleactive;
/**/
pid_t cmdoutpid;
+/* pid of last process started by <(...), >(...) */
+
+/**/
+mod_export pid_t procsubstpid;
+
/* exit status of process undergoing 'process substitution' */
/**/
@@ -4850,6 +4855,7 @@ getproc(char *cmd, char **eptr)
return NULL;
if (!out)
addproc(pid, NULL, 1, &bgtime);
+ procsubstpid = pid;
return pnam;
}
closem(FDT_UNUSED, 0);
@@ -4887,6 +4893,7 @@ getproc(char *cmd, char **eptr)
{
addproc(pid, NULL, 1, &bgtime);
}
+ procsubstpid = pid;
return pnam;
}
entersubsh(ESUB_ASYNC|ESUB_PGRP);
@@ -4937,6 +4944,7 @@ getpipe(char *cmd, int nullexec)
}
if (!nullexec)
addproc(pid, NULL, 1, &bgtime);
+ procsubstpid = pid;
return pipes[!out];
}
entersubsh(ESUB_PGRP);
@@ -6172,6 +6180,7 @@ execsave(void)
es->cmdoutpid = cmdoutpid;
es->cmdoutval = cmdoutval;
es->use_cmdoutval = use_cmdoutval;
+ es->procsubstpid = procsubstpid;
es->trap_return = trap_return;
es->trap_state = trap_state;
es->trapisfunc = trapisfunc;
@@ -6207,6 +6216,7 @@ execrestore(void)
cmdoutpid = en->cmdoutpid;
cmdoutval = en->cmdoutval;
use_cmdoutval = en->use_cmdoutval;
+ procsubstpid = en->procsubstpid;
trap_return = en->trap_return;
trap_state = en->trap_state;
trapisfunc = en->trapisfunc;
diff --git a/Src/zsh.h b/Src/zsh.h
index 8535d48..8e7f20b 100644
--- a/Src/zsh.h
+++ b/Src/zsh.h
@@ -1095,6 +1095,7 @@ struct execstack {
pid_t cmdoutpid;
int cmdoutval;
int use_cmdoutval;
+ pid_t procsubstpid;
int trap_return;
int trap_state;
int trapisfunc;
next prev parent reply other threads:[~2018-06-12 11:33 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-06-06 17:42 Sebastian Gniazdowski
2018-06-06 18:35 ` Eric Freese
2018-06-09 15:30 ` Sebastian Gniazdowski
2018-06-10 0:58 ` Eric Freese
2018-06-11 5:24 ` Sebastian Gniazdowski
2018-06-11 8:25 ` Eric Freese
2018-06-11 9:46 ` Sebastian Gniazdowski
2018-06-12 1:55 ` Eric Freese
2018-06-12 10:23 ` Sebastian Gniazdowski
2018-06-12 11:32 ` Peter Stephenson [this message]
2018-06-13 4:09 ` Eric Freese
2018-06-13 7:26 ` dana
2018-06-13 7:32 ` Sebastian Gniazdowski
2018-06-13 8:28 ` Sebastian Gniazdowski
2018-06-13 9:17 ` dana
2018-06-13 10:48 ` Sebastian Gniazdowski
2018-06-13 11:43 ` dana
2018-06-14 8:55 ` Peter Stephenson
2018-06-14 9:48 ` dana
2018-06-14 10:00 ` Peter Stephenson
2018-06-13 7:27 ` Sebastian Gniazdowski
2018-06-06 23:45 ` Takeshi Banse
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='20180612113257eucas1p2342f735bcea27bb1009aee1b23c682f0~3ZmPbMrh61839118391eucas1p2G@eucas1p2.samsung.com' \
--to=p.stephenson@samsung.com \
--cc=ericdfreese@gmail.com \
--cc=zsh-users@zsh.org \
/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).