From d0996a25bd4030875fbc98cabea815dd6741dd0d Mon Sep 17 00:00:00 2001 From: Laurent Bercot Date: Fri, 14 Aug 2015 23:29:04 +0000 Subject: Add withstdinas --- doc/backtick.html | 8 +-- doc/index.html | 3 +- doc/upgrade.html | 3 + doc/withstdinas.html | 67 ++++++++++++++++++++ package/deps.mak | 7 ++- package/modes | 1 + package/targets.mak | 3 +- src/execline/backtick.c | 125 ++++++++++++++++++-------------------- src/execline/deps-exe/backtick | 1 - src/execline/deps-exe/withstdinas | 1 + src/execline/forbacktickx.c | 2 +- src/execline/withstdinas.c | 93 ++++++++++++++++++++++++++++ 12 files changed, 237 insertions(+), 77 deletions(-) create mode 100644 doc/withstdinas.html create mode 100644 src/execline/deps-exe/withstdinas create mode 100644 src/execline/withstdinas.c diff --git a/doc/backtick.html b/doc/backtick.html index df351f2..6aa2f01 100644 --- a/doc/backtick.html +++ b/doc/backtick.html @@ -19,8 +19,8 @@

The backtick program

-backtick runs a program and uses its output as the argument of -another program. +backtick runs a program and stores its output in an +environment variable, then executes another program.

Interface

@@ -47,8 +47,8 @@ output as a value.

(Basic block management) @@ -150,6 +150,7 @@ to your installation: the shebang lines for your system might be something like

  • The ifte program
  • The ifthenelse program
  • The backtick program
  • +
  • The pipeline program
  • The runblock program
  • diff --git a/doc/upgrade.html b/doc/upgrade.html index 11ecd0a..6013a41 100644 --- a/doc/upgrade.html +++ b/doc/upgrade.html @@ -22,6 +22,9 @@

    in 2.1.3.1

    diff --git a/doc/withstdinas.html b/doc/withstdinas.html new file mode 100644 index 0000000..29ad1b6 --- /dev/null +++ b/doc/withstdinas.html @@ -0,0 +1,67 @@ + + + + + + execline: the withstdinas command + + + + + + +

    +execline
    +Software
    +skarnet.org +

    + +

    The withstdinas program

    + +

    +withstdinas reads the entirety of its standard input in an +environment variable, and runs another program with that additional +environment variable. +

    + +

    Interface

    + +

    + In an execlineb script: +

    + +
    +     withstdinas [ -i | -D default ] [ -n ] variable prog...
    +
    + + + +

    Options

    + + + +

    Notes

    + + + + + diff --git a/package/deps.mak b/package/deps.mak index a558885..d5b5fe9 100644 --- a/package/deps.mak +++ b/package/deps.mak @@ -3,7 +3,7 @@ # src/execline/background.o src/execline/background.lo: src/execline/background.c src/include/execline/execline.h -src/execline/backtick.o src/execline/backtick.lo: src/execline/backtick.c src/include/execline/execline.h +src/execline/backtick.o src/execline/backtick.lo: src/execline/backtick.c src/include/execline/config.h src/include/execline/execline.h src/execline/cd.o src/execline/cd.lo: src/execline/cd.c src/execline/define.o src/execline/define.lo: src/execline/define.c src/include-local/exlsn.h src/execline/dollarat.o src/execline/dollarat.lo: src/execline/dollarat.c @@ -46,6 +46,7 @@ src/execline/tryexec.o src/execline/tryexec.lo: src/execline/tryexec.c src/inclu src/execline/umask.o src/execline/umask.lo: src/execline/umask.c src/execline/unexport.o src/execline/unexport.lo: src/execline/unexport.c src/execline/wait.o src/execline/wait.lo: src/execline/wait.c src/include/execline/execline.h +src/execline/withstdinas.o src/execline/withstdinas.lo: src/execline/withstdinas.c src/libexecline/el_execsequence.o src/libexecline/el_execsequence.lo: src/libexecline/el_execsequence.c src/include/execline/execline.h src/libexecline/el_getstrict.o src/libexecline/el_getstrict.lo: src/libexecline/el_getstrict.c src/include/execline/execline.h src/libexecline/el_parse.o src/libexecline/el_parse.lo: src/libexecline/el_parse.c src/include/execline/execline.h @@ -73,7 +74,7 @@ src/libexecline/exlsn_multidefine.o src/libexecline/exlsn_multidefine.lo: src/li background: private EXTRA_LIBS := background: src/execline/background.o ${LIBEXECLINE} -lskarnet backtick: private EXTRA_LIBS := -backtick: src/execline/backtick.o ${LIBEXECLINE} -lskarnet +backtick: src/execline/backtick.o -lskarnet cd: private EXTRA_LIBS := cd: src/execline/cd.o -lskarnet define: private EXTRA_LIBS := @@ -158,6 +159,8 @@ unexport: private EXTRA_LIBS := unexport: src/execline/unexport.o -lskarnet wait: private EXTRA_LIBS := wait: src/execline/wait.o ${LIBEXECLINE} -lskarnet +withstdinas: private EXTRA_LIBS := +withstdinas: src/execline/withstdinas.o -lskarnet libexecline.a.xyzzy: src/libexecline/el_execsequence.o src/libexecline/el_getstrict.o src/libexecline/el_parse.o src/libexecline/el_parse_from_buffer.o src/libexecline/el_parse_from_string.o src/libexecline/el_popenv.o src/libexecline/el_pushenv.o src/libexecline/el_semicolon.o src/libexecline/el_spawn0.o src/libexecline/el_spawn1.o src/libexecline/el_substandrun.o src/libexecline/el_substandrun_str.o src/libexecline/el_substitute.o src/libexecline/el_transform.o src/libexecline/el_vardupl.o src/libexecline/exlsn_define.o src/libexecline/exlsn_elglob.o src/libexecline/exlsn_import.o src/libexecline/exlsn_multidefine.o src/libexecline/exlsn_exlp.o src/libexecline/exlsn_main.o src/libexecline/exlsn_free.o src/libexecline/exlp.o libexecline.so.xyzzy: private EXTRA_LIBS := -lskarnet libexecline.so.xyzzy: src/libexecline/el_execsequence.lo src/libexecline/el_getstrict.lo src/libexecline/el_parse.lo src/libexecline/el_parse_from_buffer.lo src/libexecline/el_parse_from_string.lo src/libexecline/el_popenv.lo src/libexecline/el_pushenv.lo src/libexecline/el_semicolon.lo src/libexecline/el_spawn0.lo src/libexecline/el_spawn1.lo src/libexecline/el_substandrun.lo src/libexecline/el_substandrun_str.lo src/libexecline/el_substitute.lo src/libexecline/el_transform.lo src/libexecline/el_vardupl.lo src/libexecline/exlsn_define.lo src/libexecline/exlsn_elglob.lo src/libexecline/exlsn_import.lo src/libexecline/exlsn_multidefine.lo src/libexecline/exlsn_exlp.lo src/libexecline/exlsn_main.lo src/libexecline/exlsn_free.lo src/libexecline/exlp.lo diff --git a/package/modes b/package/modes index b1d0883..29a5813 100644 --- a/package/modes +++ b/package/modes @@ -42,3 +42,4 @@ tryexec 0755 umask 0755 unexport 0755 wait 0755 +withstdinas 0755 diff --git a/package/targets.mak b/package/targets.mak index b26ed04..09ad2f1 100644 --- a/package/targets.mak +++ b/package/targets.mak @@ -42,7 +42,8 @@ trap \ tryexec \ umask \ unexport \ -wait +wait \ +withstdinas SBIN_TARGETS := LIBEXEC_TARGETS := diff --git a/src/execline/backtick.c b/src/execline/backtick.c index 844882b..41b95d6 100644 --- a/src/execline/backtick.c +++ b/src/execline/backtick.c @@ -1,92 +1,83 @@ /* ISC license. */ -#include -#include #include -#include +#include +#include #include #include -#include #include +#include #include -#define USAGE "backtick [ -i | -D default ] [ -n ] var { prog... } remainder..." +#define USAGE "backtick [ -i | -D default ] [ -n ] var { prog... } command..." #define dieusage() strerr_dieusage(100, USAGE) -int main (int argc, char const **argv, char const *const *envp) +int main (int argc, char const *const *argv, char const *const *envp) { - subgetopt_t localopt = SUBGETOPT_ZERO ; - pid_t pid ; - int argc1, fdwstat ; - stralloc modif = STRALLOC_ZERO ; - unsigned int modifstart ; - int insist = 0, chomp = 0 ; char const *def = 0 ; + int insist = 0, chomp = 0 ; PROG = "backtick" ; - for (;;) { - register int opt = subgetopt_r(argc, argv, "einD:", &localopt) ; - if (opt < 0) break ; - switch (opt) + subgetopt_t l = SUBGETOPT_ZERO ; + for (;;) { - case 'i' : insist = 1 ; break ; - case 'n' : chomp = 1 ; break ; - case 'e' : break ; /* compat */ - case 'D' : def = localopt.arg ; break ; - default : dieusage() ; + register int opt = subgetopt_r(argc, argv, "eniD:", &l) ; + if (opt == -1) break ; + switch (opt) + { + case 'e' : break ; /* compat */ + case 'n' : chomp = 1 ; break ; + case 'i' : insist = 1 ; break ; + case 'D' : def = l.arg ; break ; + default : dieusage() ; + } } + argc -= l.ind ; argv += l.ind ; } - argc -= localopt.ind ; argv += localopt.ind ; - if (argc < 2) dieusage() ; - if (!*argv[0]) strerr_dief1x(100, "empty variable not accepted") ; - if (!stralloc_cats(&modif, argv[0]) || !stralloc_catb(&modif, "=", 1)) - strerr_diefu1sys(111, "stralloc_catb") ; - modifstart = modif.len ; - argc-- ; argv++ ; - argc1 = el_semicolon(argv) ; - if (!argc1) strerr_dief1x(100, "empty block") ; - if (argc1 >= argc) strerr_dief1x(100, "unterminated block") ; - - argv[argc1] = 0 ; - pid = child_spawn1_pipe(argv[0], argv, envp, &fdwstat, 1) ; - if (!pid) strerr_diefu2sys(111, "spawn ", argv[0]) ; - if (!slurp(&modif, fdwstat)) strerr_diefu1sys(111, "slurp") ; - close(fdwstat) ; - if (wait_pid(pid, &fdwstat) < 0) strerr_diefu1sys(111, "wait_pid") ; - - if (wait_status(fdwstat)) + if (!argv[0][0]) dieusage() ; + if (!argv[1][0]) strerr_dief1x(100, "empty block") ; { - if (insist) - if (WIFSIGNALED(fdwstat)) strerr_dief1x(111, "child process crashed") ; - else strerr_dief1x(WEXITSTATUS(fdwstat), "child process exited non-zero") ; - else if (def) + unsigned int m = 0, i = 1 ; + int fd = dup(0) ; + char const *newargv[argc + 15] ; + char fmt[UINT_FMT] ; + if (fd < 0) { - modif.len = modifstart ; - if (!stralloc_cats(&modif, def)) strerr_diefu1sys(111, "stralloc_catb") ; + if (errno != EBADF) strerr_diefu1sys(111, "dup stdin") ; } - } - if (argc == argc1 - 1) return 0 ; - if (!stralloc_0(&modif)) strerr_diefu1sys(111, "stralloc_catb") ; - { - unsigned int reallen = str_len(modif.s) ; - if (reallen < modif.len - 1) + else fmt[uint_fmt(fmt, (unsigned int)fd)] = 0 ; + newargv[m++] = EXECLINE_BINPREFIX "pipeline" ; + newargv[m++] = "--" ; + while (argv[i] && argv[i][0] != EXECLINE_BLOCK_END_CHAR && (!EXECLINE_BLOCK_END_CHAR || (argv[i][0] && argv[i][1]))) + newargv[m++] = argv[i++] ; + if (!argv[i]) strerr_dief1x(100, "unterminated block") ; + newargv[m++] = "" ; i++ ; + newargv[m++] = EXECLINE_BINPREFIX "withstdinas" ; + if (insist) newargv[m++] = "-i" ; + if (chomp) newargv[m++] = "-n" ; + if (def) { - if (insist) - strerr_dief1x(1, "child process output contained a null character") ; - else if (def) - { - modif.len = modifstart ; - if (!stralloc_catb(&modif, def, str_len(def)+1)) - strerr_diefu1sys(111, "stralloc_catb") ; - strerr_warnw2x("child process output contained a null character", " - using default instead") ; - } - else - modif.len = reallen + 1 ; + newargv[m++] = "-D" ; + newargv[m++] = def ; + } + newargv[m++] = "-!" ; + newargv[m++] = "--" ; + newargv[m++] = argv[0] ; + if (fd < 0) + { + newargv[m++] = EXECLINE_BINPREFIX "fdclose" ; + newargv[m++] = "0" ; + } + else + { + newargv[m++] = EXECLINE_BINPREFIX "fdmove" ; + newargv[m++] = "0" ; + newargv[m++] = fmt ; } - if (chomp && (modif.s[modif.len - 2] == '\n')) - modif.s[--modif.len - 1] = 0 ; + while (argv[i]) newargv[m++] = argv[i++] ; + newargv[m++] = 0 ; + pathexec_run(newargv[0], newargv, envp) ; + strerr_dieexec(111, newargv[0]) ; } - pathexec_r(argv + argc1 + 1, envp, env_len(envp), modif.s, modif.len) ; - strerr_dieexec(111, argv[argc1 + 1]) ; } diff --git a/src/execline/deps-exe/backtick b/src/execline/deps-exe/backtick index 97021b5..e7187fe 100644 --- a/src/execline/deps-exe/backtick +++ b/src/execline/deps-exe/backtick @@ -1,2 +1 @@ -${LIBEXECLINE} -lskarnet diff --git a/src/execline/deps-exe/withstdinas b/src/execline/deps-exe/withstdinas new file mode 100644 index 0000000..e7187fe --- /dev/null +++ b/src/execline/deps-exe/withstdinas @@ -0,0 +1 @@ +-lskarnet diff --git a/src/execline/forbacktickx.c b/src/execline/forbacktickx.c index 9301581..c0c617a 100644 --- a/src/execline/forbacktickx.c +++ b/src/execline/forbacktickx.c @@ -16,7 +16,7 @@ #define DELIM_DEFAULT " \n\r\t" -int main (int argc, char const **argv, char const *const *envp) +int main (int argc, char const *const *argv, char const *const *envp) { char const *delim = DELIM_DEFAULT ; char const *codes = 0 ; diff --git a/src/execline/withstdinas.c b/src/execline/withstdinas.c new file mode 100644 index 0000000..c1e3c36 --- /dev/null +++ b/src/execline/withstdinas.c @@ -0,0 +1,93 @@ +/* ISC license. */ + +#include +#include +#include +#include +#include +#include +#include +#include +#include + +#define USAGE "withstdinas [ -i | -D default ] [ -n ] var remainder..." +#define dieusage() strerr_dieusage(100, USAGE) + +int main (int argc, char const **argv, char const *const *envp) +{ + subgetopt_t localopt = SUBGETOPT_ZERO ; + stralloc modif = STRALLOC_ZERO ; + unsigned int modifstart ; + int insist = 0, chomp = 0, reapit = 0 ; + char const *def = 0 ; + PROG = "withstdinas" ; + for (;;) + { + register int opt = subgetopt_r(argc, argv, "einD:!", &localopt) ; + if (opt < 0) break ; + switch (opt) + { + case 'i' : insist = 1 ; break ; + case 'n' : chomp = 1 ; break ; + case 'D' : def = localopt.arg ; break ; + case '!' : reapit = 1 ; break ; + default : dieusage() ; + } + } + argc -= localopt.ind ; argv += localopt.ind ; + + if (!argc) dieusage() ; + if (!*argv[0] || strchr(argv[0], '=')) strerr_dief1x(100, "invalid variable name") ; + if (!stralloc_catb(&modif, "!", 2) + || !stralloc_cats(&modif, argv[0]) + || !stralloc_catb(&modif, "=", 1)) + strerr_diefu1sys(111, "stralloc_catb") ; + modifstart = modif.len ; + if (!slurp(&modif, 0)) strerr_diefu1sys(111, "slurp") ; + if (reapit) + { + char const *x = env_get2(envp, "!") ; + if (x) + { + uint64 pid ; + int wstat ; + if (!uint640_scan(x, &pid)) strerr_dieinvalid(100, "!") ; + if (waitpid(pid, &wstat, 0) < 0) + strerr_diefu1sys(111, "waitpid") ; + if (wait_estatus(wstat)) + { + if (insist) + if (WIFSIGNALED(wstat)) strerr_dief1x(wait_estatus(wstat), "child process crashed") ; + else strerr_dief1x(wait_estatus(wstat), "child process exited non-zero") ; + else if (def) + { + modif.len = modifstart ; + if (!stralloc_cats(&modif, def)) strerr_diefu1sys(111, "stralloc_catb") ; + } + } + } + } + if (!stralloc_0(&modif)) strerr_diefu1sys(111, "stralloc_catb") ; + { + unsigned int reallen = str_len(modif.s + 2) ; + if (reallen < modif.len - 3) + { + if (insist) + strerr_dief1x(1, "stdin contained a null character") ; + else if (def) + { + modif.len = modifstart ; + if (!stralloc_catb(&modif, def, str_len(def)+1)) + strerr_diefu1sys(111, "stralloc_catb") ; + strerr_warnw2x("stdin contained a null character", " - using default instead") ; + } + else + modif.len = reallen + 3 ; + } + if (chomp && (modif.s[modif.len - 2] == '\n')) + modif.s[--modif.len - 1] = 0 ; + } + if (!argv[1]) return 0 ; + pathexec_r(argv + 1, envp, env_len(envp), modif.s + !reapit * 2, modif.len - !reapit * 2) ; + strerr_dieexec(111, argv[1]) ; +} -- cgit v1.2.3