Date: Sun, 12 May 2024 09:30:28 +0200 (CEST) From: Ronald Klop <ronald-lists@klop.ws> To: Warner Losh <imp@FreeBSD.org> Cc: dev-commits-src-main@FreeBSD.org, src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org Subject: Re: git: 9bcc1b18c119 - main - /bin/rmdir: Exit with status 2 for invalid arguments Message-ID: <1570892010.7412.1715499028011@localhost> In-Reply-To: <202405111916.44BJGJ2I010959@gitrepo.freebsd.org>
index | next in thread | previous in thread | raw e-mail
[-- Attachment #1 --] Wouldn’t EX_USAGE fit really well? https://man.freebsd.org/cgi/man.cgi?query=sysexits&manpath=FreeBSD+4.3-RELEASE Regards, Ronald Van: Warner Losh <imp@FreeBSD.org> Datum: 11 mei 2024 21:16 Aan: src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org Onderwerp: git: 9bcc1b18c119 - main - /bin/rmdir: Exit with status 2 for invalid arguments > > > The branch main has been updated by imp: > > URL: https://cgit.FreeBSD.org/src/commit/?id=9bcc1b18c119148e4455e548c90b2bc9cef16d1b > > commit 9bcc1b18c119148e4455e548c90b2bc9cef16d1b > Author: Henrich Hartzer > AuthorDate: 2024-05-10 17:53:49 +0000 > Commit: Warner Losh > CommitDate: 2024-05-11 19:13:28 +0000 > > /bin/rmdir: Exit with status 2 for invalid arguments > > PR: 277677 > > Signed-off-by: Henrich Hartzer > Reviewed by: imp > Pull Request: https://github.com/freebsd/freebsd-src/pull/1161 > --- > bin/rmdir/rmdir.1 | 15 ++++++++------- > bin/rmdir/rmdir.c | 2 +- > bin/rmdir/tests/rmdir_test.sh | 6 +++--- > 3 files changed, 12 insertions(+), 11 deletions(-) > > diff --git a/bin/rmdir/rmdir.1 b/bin/rmdir/rmdir.1 > index 87ca1f1547f0..eb23c605050c 100644 > --- a/bin/rmdir/rmdir.1 > +++ b/bin/rmdir/rmdir.1 > @@ -74,14 +74,14 @@ The > .Nm > utility exits with one of the following values: > .Bl -tag -width indent > -.It Li 0 > -Each directory entry specified by a > +.It Er 0 > +Each > .Ar directory > -operand > -referred to an empty directory and was removed > -successfully. > -.It Li >0 > -An error occurred. > +referred to an empty directory and was removed successfully. > +.It Er 1 > +An error occurred while attempting to remove one or more directories. > +.It Er 2 > +Invalid arguments. > .El > .Sh EXAMPLES > Remove the directory > @@ -97,6 +97,7 @@ stopping at the first non-empty directory (if any): > .Dl $ rmdir -p cow/horse/monkey > .Sh SEE ALSO > .Xr rm 1 > +.Xr rmdir 2 > .Sh STANDARDS > The > .Nm > diff --git a/bin/rmdir/rmdir.c b/bin/rmdir/rmdir.c > index c5d3db831309..0a495018495c 100644 > --- a/bin/rmdir/rmdir.c > +++ b/bin/rmdir/rmdir.c > @@ -112,5 +112,5 @@ usage(void) > { > > (void)fprintf(stderr, "usage: rmdir [-pv] directory ..."); > - exit(1); > + exit(2); > } > diff --git a/bin/rmdir/tests/rmdir_test.sh b/bin/rmdir/tests/rmdir_test.sh > index d443849258b6..ba80ac6204be 100644 > --- a/bin/rmdir/tests/rmdir_test.sh > +++ b/bin/rmdir/tests/rmdir_test.sh > @@ -35,8 +35,8 @@ invalid_usage_head() > > invalid_usage_body() > { > - atf_check -s not-exit:0 -e match:"$usage_output" rmdir -p > - atf_check -s not-exit:0 -e match:"$usage_output" rmdir -v > + atf_check -s exit:2 -e match:"$usage_output" rmdir -p > + atf_check -s exit:2 -e match:"$usage_output" rmdir -v > } > > atf_test_case no_arguments > @@ -47,7 +47,7 @@ no_arguments_head() > > no_arguments_body() > { > - atf_check -s not-exit:0 -e match:"$usage_output" rmdir > + atf_check -s exit:2 -e match:"$usage_output" rmdir > } > > atf_init_test_cases() > > > > > [-- Attachment #2 --] <html><head></head><body>Wouldn’t EX_USAGE fit really well?<div><br></div><div><a href="https://man.freebsd.org/cgi/man.cgi?query=sysexits&manpath=FreeBSD+4.3-RELEASE">https://man.freebsd.org/cgi/man.cgi?query=sysexits&manpath=FreeBSD+4.3-RELEASE</a></div><div><br></div><div>Regards,</div><div>Ronald</div><div><br></div><div><p><small><strong>Van:</strong>; Warner Losh <imp@FreeBSD.org><br><strong>Datum:</strong> 11 mei 2024 21:16<br><strong>Aan:</strong> src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org<br><strong>Onderwerp:</strong> git: 9bcc1b18c119 - main - /bin/rmdir: Exit with status 2 for invalid arguments<br></small></p><blockquote style="margin-left: 5px; border-left: 3px solid #ccc; margin-right: 0px; padding-left: 5px;"><div class="MessageRFC822Viewer do_not_remove" id="P"><!-- P --> <!-- processMimeMessage --><div class="TextPlainViewer do_not_remove" id="P.P"><!-- P.P -->The branch main has been updated by imp:<br> <br> URL: <a href="https://cgit.FreeBSD.org/src/commit/?id=9bcc1b18c119148e4455e548c90b2bc9cef16d1b">https://cgit.FreeBSD.org/src/commit/?id=9bcc1b18c119148e4455e548c90b2bc9cef16d1b</a><br>; <br> commit 9bcc1b18c119148e4455e548c90b2bc9cef16d1b<br> Author: Henrich Hartzer <henrichhartzer@tuta.io><br> AuthorDate: 2024-05-10 17:53:49 +0000<br> Commit: Warner Losh <imp@freebsd.org><br> CommitDate: 2024-05-11 19:13:28 +0000<br> <br> /bin/rmdir: Exit with status 2 for invalid arguments<br> <br> PR: 277677<br> <br> Signed-off-by: Henrich Hartzer <henrichhartzer@tuta.io><br> Reviewed by: imp<br> Pull Request: <a href="https://github.com/freebsd/freebsd-src/pull/1161">https://github.com/freebsd/freebsd-src/pull/1161</a><br>; ---<br> bin/rmdir/rmdir.1 | 15 ++++++++-------<br> bin/rmdir/rmdir.c | 2 +-<br> bin/rmdir/tests/rmdir_test.sh | 6 +++---<br> 3 files changed, 12 insertions(+), 11 deletions(-)<br> <br> diff --git a/bin/rmdir/rmdir.1 b/bin/rmdir/rmdir.1<br> index 87ca1f1547f0..eb23c605050c 100644<br> --- a/bin/rmdir/rmdir.1<br> +++ b/bin/rmdir/rmdir.1<br> @@ -74,14 +74,14 @@ The<br> .Nm<br> utility exits with one of the following values:<br> .Bl -tag -width indent<br> -.It Li 0<br> -Each directory entry specified by a<br> +.It Er 0<br> +Each<br> .Ar directory<br> -operand<br> -referred to an empty directory and was removed<br> -successfully.<br> -.It Li >0<br> -An error occurred.<br> +referred to an empty directory and was removed successfully.<br> +.It Er 1<br> +An error occurred while attempting to remove one or more directories.<br> +.It Er 2<br> +Invalid arguments.<br> .El<br> .Sh EXAMPLES<br> Remove the directory<br> @@ -97,6 +97,7 @@ stopping at the first non-empty directory (if any):<br> .Dl $ rmdir -p cow/horse/monkey<br> .Sh SEE ALSO<br> .Xr rm 1<br> +.Xr rmdir 2<br> .Sh STANDARDS<br> The<br> .Nm<br> diff --git a/bin/rmdir/rmdir.c b/bin/rmdir/rmdir.c<br> index c5d3db831309..0a495018495c 100644<br> --- a/bin/rmdir/rmdir.c<br> +++ b/bin/rmdir/rmdir.c<br> @@ -112,5 +112,5 @@ usage(void)<br> {<br> <br> (void)fprintf(stderr, "usage: rmdir [-pv] directory ... ");<br> - exit(1);<br> + exit(2);<br> }<br> diff --git a/bin/rmdir/tests/rmdir_test.sh b/bin/rmdir/tests/rmdir_test.sh<br> index d443849258b6..ba80ac6204be 100644<br> --- a/bin/rmdir/tests/rmdir_test.sh<br> +++ b/bin/rmdir/tests/rmdir_test.sh<br> @@ -35,8 +35,8 @@ invalid_usage_head()<br> <br> invalid_usage_body()<br> {<br> - atf_check -s not-exit:0 -e match:"$usage_output" rmdir -p<br> - atf_check -s not-exit:0 -e match:"$usage_output" rmdir -v<br> + atf_check -s exit:2 -e match:"$usage_output" rmdir -p<br> + atf_check -s exit:2 -e match:"$usage_output" rmdir -v<br> }<br> <br> atf_test_case no_arguments<br> @@ -47,7 +47,7 @@ no_arguments_head()<br> <br> no_arguments_body()<br> {<br> - atf_check -s not-exit:0 -e match:"$usage_output" rmdir<br> + atf_check -s exit:2 -e match:"$usage_output" rmdir<br> }<br> <br> atf_init_test_cases()<br> <br> </henrichhartzer@tuta.io></imp@freebsd.org></henrichhartzer@tuta.io></div><!-- TextPlainViewer --> <hr> </div><!-- MessageRFC822Viewer --> </blockquote><br><br><br></div></body></html>help
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1570892010.7412.1715499028011>