Re: [PATCH v2 1/2] test: Add unary operator -E: test that a file is an empty directory

2016-04-13 Thread Mattias Andrée
On Wed, 13 Apr 2016 16:16:54 +0100 Stephane Chazelas wrote: > 2016-04-07 10:34:56 +0200, Mattias Andrée: > [] > > + if (!dir) > > +{ > > + error (0, errno, "%s", dirname); > > + test_exit (TEST_FAILURE); > > +} > > Note that it means it

Re: [PATCH v2 1/2] test: Add unary operator -E: test that a file is an empty directory

2016-04-13 Thread Stephane Chazelas
2016-04-07 10:34:56 +0200, Mattias Andrée: [] > + if (!dir) > +{ > + error (0, errno, "%s", dirname); > + test_exit (TEST_FAILURE); > +} Note that it means it makes it the first operator that would actually cause "test" to output something. A test for non-empty without

Re: [PATCH v2 1/2] test: Add unary operator -E: test that a file is an empty directory

2016-04-07 Thread Mattias Andrée
On Thu, 7 Apr 2016 11:47:36 +0200 Bernhard Voelker wrote: > On 04/07/2016 10:34 AM, Mattias Andrée wrote: > > diff --git a/src/test.c b/src/test.c > > index 8ac7467..eb9c43a 100644 > > --- a/src/test.c > > +++ b/src/test.c > > @@ -27,6 +27,7 @@ > > #endif > > > >

Re: [PATCH v2 1/2] test: Add unary operator -E: test that a file is an empty directory

2016-04-07 Thread Bernhard Voelker
On 04/07/2016 10:34 AM, Mattias Andrée wrote: > diff --git a/src/test.c b/src/test.c > index 8ac7467..eb9c43a 100644 > --- a/src/test.c > +++ b/src/test.c > @@ -27,6 +27,7 @@ > #endif > > #include > +#include > #include > #include > > @@ -179,6 +180,39 @@ get_mtime (char const

[PATCH v2 1/2] test: Add unary operator -E: test that a file is an empty directory

2016-04-07 Thread Mattias Andrée
--- NEWS | 5 + doc/coreutils.texi | 7 ++- src/test.c | 44 +++- 3 files changed, 54 insertions(+), 2 deletions(-) diff --git a/NEWS b/NEWS index dd3ee9c..3445b17 100644 --- a/NEWS +++ b/NEWS @@ -27,6 +27,11 @@ GNU