On Mon, 7 Dec 2020 13:39:30 +0100
Antoine Jacoutot <ajacou...@bsdfrog.org>:


> Index: syspatch.8
> ===================================================================
> RCS file: /cvs/src/usr.sbin/syspatch/syspatch.8,v
> retrieving revision 1.21
> diff -u -p -r1.21 syspatch.8
> --- syspatch.8        25 Jul 2020 15:45:44 -0000      1.21
> +++ syspatch.8        7 Dec 2020 12:39:07 -0000
> @@ -64,6 +64,8 @@ of installed patches.
>  .El
>  .Sh EXIT STATUS
>  .Ex -std syspatch
> +In particular, 2 indicates that applying patches was requested but no
> +additional patch was installed.
>  .Sh SEE ALSO
>  .Xr signify 1 ,
>  .Xr installurl 5 ,
> Index: syspatch.sh
> ===================================================================
> RCS file: /cvs/src/usr.sbin/syspatch/syspatch.sh,v
> retrieving revision 1.166
> diff -u -p -r1.166 syspatch.sh
> --- syspatch.sh       27 Oct 2020 17:42:05 -0000      1.166
> +++ syspatch.sh       7 Dec 2020 12:39:07 -0000
> @@ -320,6 +320,7 @@ if ((OPTIND == 1)); then
>                       [[ -f ${_D}/rollback.tgz ]] || rm -r ${_D}
>       done
>       _PATCHES=$(ls_missing) # can't use errexit in a for loop
> +     [[ -n ${_PATCHES} ]] || exit 2
>       for _PATCH in ${_PATCHES}; do
>               apply_patch ${_OSrev}-${_PATCH}
>               _PATCH_APPLIED=true
> 
> 

In my opinion, if committed it deserves an entry in
https://www.openbsd.org/faq/current.html

Reply via email to