Re: [PATCH 05/21] i18n: sequencer: mark entire sentences for translation
- From
Eric Sunshine <sunshine@sunshineco.com>
- Date
- May 18, 2016, 19:28 UTC
- Message-ID
- <CAPig+cQzsxMMv1XSdStFDs08C3qvb_Ttb4oVeN3T1si85v=f7w@mail.gmail.com>
- In-Reply-To
- <1463585274-9027-6-git-send-email-vascomalmeida@sapo.pt>
On Wed, May 18, 2016 at 11:27 AM, Vasco Almeida <vascomalmeida@sapo.pt> wrote:
Show 22 quoted lines
> Mark entire sentences of error message rather than assembling one using
> placeholders (e.g. "Cannot %s during a %s"). That would facilitate
> translation work.
>
> Signed-off-by: Vasco Almeida <vascomalmeida@sapo.pt>
> ---
> diff --git a/sequencer.c b/sequencer.c
> @@ -697,9 +697,14 @@ static struct commit *parse_insn_line(char *bol, char *eol, struct replay_opts *
> if (action != opts->action) {
> - const char *action_str;
> - action_str = action == REPLAY_REVERT ? "revert" : "cherry-pick";
> - error(_("Cannot %s during a %s"), action_str, action_name(opts));
> + if (action == REPLAY_REVERT)
> + error((opts->action == REPLAY_REVERT)
> + ? _("Cannot revert during a another revert.")
> + : _("Cannot revert during a cherry-pick."));
> + else
> + error((opts->action == REPLAY_REVERT)
> + ? _("Cannot cherry-pick during a revert.")
> + : _("Cannot cherry-pick during another cherry-pick."));
> return NULL;
> }Similar to my comment on patch 3/21, since the "actions" are literal git commands, it's not clear why this change is helpful. Perhaps the commit message needs to do a better job of persuading the reader?