<feed xmlns='http://www.w3.org/2005/Atom'>
<title>src/usr.bin/script, branch release/9.0.0</title>
<subtitle>FreeBSD source tree</subtitle>
<id>https://cgit-dev.freebsd.org/src/atom?h=release%2F9.0.0</id>
<link rel='self' href='https://cgit-dev.freebsd.org/src/atom?h=release%2F9.0.0'/>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src/'/>
<updated>2011-10-21T05:37:40Z</updated>
<entry>
<title>MFC r226403:</title>
<updated>2011-10-21T05:37:40Z</updated>
<author>
<name>Mikolaj Golub</name>
<email>trociny@FreeBSD.org</email>
</author>
<published>2011-10-21T05:37:40Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src/commit/?id=d80d19dbbbd4bc8fff9d4923095740826ffc003f'/>
<id>urn:sha1:d80d19dbbbd4bc8fff9d4923095740826ffc003f</id>
<content type='text'>
In r225809 the intention was to send VEOF only once if STDIN was not a
terminal. Unfortunately the fix was incorrect and for flushtime &gt; 0 it
keept sending VEOF.

Sent VEOF generates ^D\b\b echoed by the terminal, which was reported
in bin/161526. Note, we still send VEOF at least once. Otherwise
commands like below would hang forever:

  echo 1 |script /tmp/script.out cat

PR:		bin/161526
Reported by:	Adrian Wontroba &lt;aw1@stade.co.uk&gt;, Stefan Bethke &lt;stb@lassitu.de&gt;
Tested by:	Stefan Bethke &lt;stb@lassitu.de&gt;
Approved by:	re (kib)
</content>
</entry>
<entry>
<title>MFC r225857:</title>
<updated>2011-10-10T11:05:58Z</updated>
<author>
<name>Glen Barber</name>
<email>gjb@FreeBSD.org</email>
</author>
<published>2011-10-10T11:05:58Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src/commit/?id=025df08fff1dd14fabdb3e001f77f4837cbc0382'/>
<id>urn:sha1:025df08fff1dd14fabdb3e001f77f4837cbc0382</id>
<content type='text'>
 - Fix a few grammar and mdoc nits in script.1

PR:		161088
Approved by:	re (kib)
</content>
</entry>
<entry>
<title>MFC r225809:</title>
<updated>2011-10-04T10:00:28Z</updated>
<author>
<name>Mikolaj Golub</name>
<email>trociny@FreeBSD.org</email>
</author>
<published>2011-10-04T10:00:28Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src/commit/?id=3c85cb7b5942043cb3fea320fa21b36ae43f8d76'/>
<id>urn:sha1:3c85cb7b5942043cb3fea320fa21b36ae43f8d76</id>
<content type='text'>
When script(1) reads EOF from input it starts spinning on zero-byte
reads eating 100% CPU. Fix this by skipping select on STDIN after
reading EOF -- permanently if STDIN is not terminal and for one second
if it is.

Also after reading EOF from STDIN we have to pass it to the program
being scripted. The previous approach was to write zero bytes into the
pseudo-terminal. This does not work because zero-byte write does not
have any effect on read. Fix this by sending VEOF instead.

Submitted by:	Ronald Klop &lt;ronald-freebsd8@klop.yi.org&gt;
Discussed with:	kib, Chris Torek &lt;chris.torek@gmail.com&gt;

Approved by:	re (kib)
</content>
</entry>
<entry>
<title>Remove the advertising clause from UCB copyrighted files in usr.bin.  This</title>
<updated>2010-12-11T08:32:16Z</updated>
<author>
<name>Joel Dahl</name>
<email>joel@FreeBSD.org</email>
</author>
<published>2010-12-11T08:32:16Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src/commit/?id=da52b4caaf187775f6b56a72c6b16e94ad728f7b'/>
<id>urn:sha1:da52b4caaf187775f6b56a72c6b16e94ad728f7b</id>
<content type='text'>
is in accordance with the information provided at
ftp://ftp.cs.berkeley.edu/pub/4bsd/README.Impt.License.Change

Also add $FreeBSD$ to a few files to keep svn happy.

Discussed with:	imp, rwatson
</content>
</entry>
<entry>
<title>Fix the grammar after I added a second environmental variable.</title>
<updated>2010-09-19T00:36:26Z</updated>
<author>
<name>David E. O'Brien</name>
<email>obrien@FreeBSD.org</email>
</author>
<published>2010-09-19T00:36:26Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src/commit/?id=aa2e033466be5ec41d1a3e2b4c00e877f53a374c'/>
<id>urn:sha1:aa2e033466be5ec41d1a3e2b4c00e877f53a374c</id>
<content type='text'>
Submitted by:	wxs
</content>
</entry>
<entry>
<title>+ Add the SCRIPT environmental variable to the sub-shell.  Its value is</title>
<updated>2010-09-16T22:31:03Z</updated>
<author>
<name>David E. O'Brien</name>
<email>obrien@FreeBSD.org</email>
</author>
<published>2010-09-16T22:31:03Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src/commit/?id=9b91846ce46883d5478ab381c71245e12498a931'/>
<id>urn:sha1:9b91846ce46883d5478ab381c71245e12498a931</id>
<content type='text'>
the name of the typescript file.
+ Add the 'command' argument (if supplied on the command line) to the
typescript file.  This creates a more complete typescript when invoked
this way - more equal to invoking script without supplying the 'command'
argument.
</content>
</entry>
<entry>
<title>Mark functions and variables as static.</title>
<updated>2010-08-16T12:19:36Z</updated>
<author>
<name>Ed Schouten</name>
<email>ed@FreeBSD.org</email>
</author>
<published>2010-08-16T12:19:36Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src/commit/?id=e8efeec6c53cb3c97db41a1cc04296e18ddb41fe'/>
<id>urn:sha1:e8efeec6c53cb3c97db41a1cc04296e18ddb41fe</id>
<content type='text'>
All these functions and variables are local to this compilation unit, so
there is no reason why we shouldn't mark them static. This slightly
reduces the binary size.
</content>
</entry>
<entry>
<title>Remove WNOHANG flag from wait3().</title>
<updated>2010-04-30T22:33:49Z</updated>
<author>
<name>Ed Schouten</name>
<email>ed@FreeBSD.org</email>
</author>
<published>2010-04-30T22:33:49Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src/commit/?id=e1e9ba3370d01ab5448943461a07d6d183a15bf1'/>
<id>urn:sha1:e1e9ba3370d01ab5448943461a07d6d183a15bf1</id>
<content type='text'>
Because script(1) now reliably terminates when the TTY is closed, it may
be the case that the call to wait3() occurs just before the child
process exits. This causes error codes to be ignored.

Just change script(1) to use waitpid() instead of wait3(). This makes it
more portable and prevents the need for a loop, since waitpid() only
returns a specified process.

PR:		bin/146189
Tested by:	amdmi3@, older version
MFC after:	2 weeks
</content>
</entry>
<entry>
<title>Improve the change made in the previous commit.</title>
<updated>2010-03-11T11:28:29Z</updated>
<author>
<name>Ed Schouten</name>
<email>ed@FreeBSD.org</email>
</author>
<published>2010-03-11T11:28:29Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src/commit/?id=e8b9127d7e6d7f9a2a3f882fe4af2415419e5781'/>
<id>urn:sha1:e8b9127d7e6d7f9a2a3f882fe4af2415419e5781</id>
<content type='text'>
doshell() never returns, so there is no need to see whether we are the
parent process.
</content>
</entry>
<entry>
<title>Make script(1) a little less broken.</title>
<updated>2010-03-11T11:09:58Z</updated>
<author>
<name>Ed Schouten</name>
<email>ed@FreeBSD.org</email>
</author>
<published>2010-03-11T11:09:58Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src/commit/?id=8d5e4a1416a17965273ceb8acfb5ef649dfc2acf'/>
<id>urn:sha1:8d5e4a1416a17965273ceb8acfb5ef649dfc2acf</id>
<content type='text'>
Close the file descriptor to the TTY. There is no reason why the parent
process should keep track of the descriptor. This ensures that the
application inside properly drains the TTY during exit(2).

Reported by:	alfred
MFC after:	2 weeks
</content>
</entry>
</feed>
