From: Sami Kerola <kerolasa@iki.fi>
To: util-linux@vger.kernel.org
Cc: Sami Kerola <kerolasa@iki.fi>
Subject: [PATCH 00/22] pull: follow-up to chsh, newgrp, and script changes
Date: Sat, 27 Dec 2014 17:17:27 +0000 [thread overview]
Message-ID: <1419700655-19107-1-git-send-email-kerolasa@iki.fi> (raw)
Hi,
As I mentioned in
http://comments.gmane.org/gmane.linux.utilities.util-linux-ng/10390
the previous patch set was not complete. The script(1) had signals and
io-streams race that is now fixed with changes 0015 to 0018.
The patches below can be also found from my github repository branch
'script'.
https://github.com/kerolasa/lelux-utiliteetit/tree/script
Sami Kerola (22):
chsh: remove function prototypes
chfn, chsh: share illegal_passwd_chars() function
chsh: use getline() to support arbitrarily long lines
chsh: set few variables read-only and rename one of them
chsh: allow user to set shell to /bin/sh if none is set
chsh: clean up parse_argv()
chsh: rewrite function interacting with user to get path to new shell
chsh: simplify check_shell()
chsh: fail get_shell_list() check when /etc/shells cannot be opened
newgrp: simplify if else clauses
newgrp: move shell determination closer where it is used
newgrp: set function arguments read-only when possible
script: remove function prototypes
script: add struct script_control and remove global variables
=== end of earlier submission ===
script: use signalfd() to catch signals
script: use poll() rather than select()
script: merge doinput() and output() functions to do_io()
script: remove io vs signal race
script: add 'Script started' line always to capture file
script: move do_io() content to small functions
script: replace strftime() workaround with CFLAGS = -Wno-format-y2k
script: use correct input type, move comment, and so on
login-utils/Makemodule.am | 4 +-
login-utils/ch-common.c | 34 ++
login-utils/ch-common.h | 6 +
login-utils/chfn.c | 16 +-
login-utils/chsh.c | 365 +++++++++----------
login-utils/newgrp.c | 25 +-
term-utils/Makemodule.am | 1 +
term-utils/script.c | 866 ++++++++++++++++++++--------------------------
tests/ts/script/race | 4 -
9 files changed, 594 insertions(+), 727 deletions(-)
create mode 100644 login-utils/ch-common.c
create mode 100644 login-utils/ch-common.h
--
2.2.1
next reply other threads:[~2014-12-27 17:17 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-12-27 17:17 Sami Kerola [this message]
2014-12-27 17:17 ` [PATCH 15/22] script: use signalfd() to catch signals Sami Kerola
2014-12-27 17:17 ` [PATCH 16/22] script: use poll() rather than select() Sami Kerola
2014-12-27 17:17 ` [PATCH 17/22] script: merge doinput() and output() functions to do_io() Sami Kerola
2014-12-27 17:17 ` [PATCH 18/22] script: remove io vs signal race Sami Kerola
2014-12-27 17:17 ` [PATCH 19/22] script: add 'Script started' line always to capture file Sami Kerola
2014-12-27 17:17 ` [PATCH 20/22] script: move do_io() content to small functions Sami Kerola
2014-12-27 17:17 ` [PATCH 21/22] script: replace strftime() workaround with CFLAGS = -Wno-format-y2k Sami Kerola
2014-12-27 17:17 ` [PATCH 22/22] script: use correct input type, move comment, and so on Sami Kerola
2015-01-06 11:11 ` Karel Zak
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=1419700655-19107-1-git-send-email-kerolasa@iki.fi \
--to=kerolasa@iki.fi \
--cc=util-linux@vger.kernel.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.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.