[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: new snapshot available: coreutils-8.27.103-db8d1
From: |
Pádraig Brady |
Subject: |
Re: new snapshot available: coreutils-8.27.103-db8d1 |
Date: |
Wed, 30 Aug 2017 20:46:27 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.8.0 |
On 30/08/17 20:41, Pádraig Brady wrote:
>> Looks like solaris doesn't distinguish ENOTTY and EBADF
>> It returns ENOENT in all isatty() failure cases :/
>>
>> I might change things so that tty doesn't distinguish
>> these failure cases on input.
>> I.E. just return 1 for all errors on input
>
> I'm going with the two attached patches to avoid these issues.
Oops, with these doc adjustments on top:
diff --git a/doc/coreutils.texi b/doc/coreutils.texi
index 5a0c476..eb17462 100644
--- a/doc/coreutils.texi
+++ b/doc/coreutils.texi
@@ -15123,7 +15123,6 @@ Exit status:
1 if standard input is a non-terminal file
2 if given incorrect arguments
3 if a write error occurs
-4 if standard input is closed or its type cannot be determined
@end display
diff --git a/src/tty.c b/src/tty.c
index 5e88c87..c02a0d0 100644
--- a/src/tty.c
+++ b/src/tty.c
@@ -17,7 +17,7 @@
/* Displays "not a tty" if stdin is not a terminal.
Displays nothing if -s option is given.
Exit status 0 if stdin is a tty, 1 if not a tty, 2 if usage error,
- 3 if write error, 4 for other stdin errors.
+ 3 if write error.
- Re: [platform-testers] new snapshot available: coreutils-8.27.99-3ebdc3, (continued)
new snapshot available: coreutils-8.27.103-db8d1, Pádraig Brady, 2017/08/30
- Re: new snapshot available: coreutils-8.27.103-db8d1, Erik Auerswald, 2017/08/30
- [PATCH] expr: plug a memory leak in eval6(), Kamil Dudka, 2017/08/30
- Re: new snapshot available: coreutils-8.27.103-db8d1, Assaf Gordon, 2017/08/30
- Re: new snapshot available: coreutils-8.27.103-db8d1, Eric Blake, 2017/08/31
- Re: new snapshot available: coreutils-8.27.103-db8d1, Pádraig Brady, 2017/08/31
new snapshot available: coreutils-8.27.108-2ba96, Pádraig Brady, 2017/08/31