PR/57997 CVS commit: src/usr.sbin/certctl

["Taylor R Campbell" <riastradh%netbsd.org@localhost>]

The following reply was made to PR bin/57997; it has been noted by GNATS.

From: "Taylor R Campbell" <riastradh%netbsd.org@localhost>
To: gnats-bugs%gnats.NetBSD.org@localhost
Cc: 
Subject: PR/57997 CVS commit: src/usr.sbin/certctl
Date: Mon, 4 Mar 2024 20:37:32 +0000

 Module Name:	src
 Committed By:	riastradh
 Date:		Mon Mar  4 20:37:32 UTC 2024
 
 Modified Files:
 	src/usr.sbin/certctl: certctl.sh
 
 Log Message:
 certctl(8): Pacify formal POSIX sh syntax.
 
 According to POSIX 2018, the syntax between `then' and `elif' and
 `fi' must be a _non-empty_ list of commands:
 
 compound_list    : linebreak term
                  | linebreak term separator
                  ;
 ...
 if_clause        : If compound_list Then compound_list else_part Fi
                  | If compound_list Then compound_list           Fi
                  ;
 else_part        : Elif compound_list Then compound_list
                  | Elif compound_list Then compound_list else_part
                  | Else compound_list
                  ;
 
 https://pubs.opengroup.org/onlinepubs/9699919799/utilities/V3_chap02.html#tag_18_10_02
 
 NetBSD's sh(1) currently doesn't enforce this and allows an empty
 sequence of commands, but let's not rely on that nonstandard quirk.
 
 Noted in PR 57997.
 
 
 To generate a diff of this commit:
 cvs rdiff -u -r1.6 -r1.7 src/usr.sbin/certctl/certctl.sh
 
 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.