From: Sergey Organov <sorganov@gmail.com>
Date: Tue, 04 Aug 2020 21:58:03 GMT
Subject: Re: [PATCH v2 0/7] making log --first-parent imply -m
Message-ID: <87imdyxedw.fsf@osv.gnss.ru>
In-Reply-To: <20200804195830.GA2014743@coredump.intra.peff.net>

Jeff King <peff@peff.net> writes:

> On Tue, Aug 04, 2020 at 08:50:16PM +0300, Sergey Organov wrote:

[...]

>
>> +		} else {
>> +			die("--diff-merges: unknown value '%s'.", optarg);
>> +		}
>
> A few nits:

I missed this the first time, sorry!

>
>   - we usually don't have a period at the end of our error messages

Oops, I got the dot from

  die("--unpacked=<packfile> no longer supported.");

in the same file. Will fix.

>   - this should probably be marked for translation, i.e.,
>     die(_("translated message"), optarg)

OK, will do.

>
>   - I think other similar messages are more like:
>
>       unknown value for --diff-merges: %s

Thanks, I'll change wording to this one.

-- Sergey