From: Junio C Hamano Date: Thu, 19 Jul 2018 18:29:28 GMT Subject: Re: [PATCH v2 01/23] Update messages in preparation for i18n Message-ID: In-Reply-To: Junio C Hamano writes: >> @@ -622,7 +626,7 @@ int cmd_config(int argc, const char **argv, const char *prefix) >> * location; error out even if XDG_CONFIG_HOME >> * is set and points at a sane location. >> */ >> - die("$HOME not set"); >> + die("$HOME is not set"); > > Meh. There are many verb-less messages e.g. "only one X at a time" > that are not given a new verb in this patch. Just to save wasted work due to misunderstanding, this is merely "Meh" (I do not think it is worth it, and I wouldn't have included this hunk if I were doing the patch, but since the patch has already been written, I do not mind having it, either), not "Reject" (drop this and resend). Thanks.