Às 20:23 de 08-04-2016, Junio C Hamano escreveu:
> Vasco Almeida <vascomalmeida@xxxxxxx> writes:
>
>> Mark strings seen by the user inside setup_unpack_trees_porcelain() and
>> display_error_msgs() functions for translation.
>>
>> One case Git outputs the named strings is when the user does some editing to
>> a file on some branch and then checks out other branch, but that file
>> changes by the checkout. This is the case of the first string marked.
>
> The second paragraph puzzling. Other strings are not shown at all?
> Or is it just you didn't bother explaining the others cases?
>
> Either case, it is not immediately clear why you felt that one
> string among others deserve more attention. Care to elaborate?
I meant that as an example, which was the situation I was when I found
the untranslated string.
I should have written that outside the commit message, as a note.
I'm going to remove that paragraph since it isn’t irrelevant and
confusing. Hope I answered you.
>
> In the patch itself I did not find anything questionable, though.
>
> Thanks.
>
>>
>> Signed-off-by: Vasco Almeida <vascomalmeida@xxxxxxx>
>> ---
>> unpack-trees.c | 24 ++++++++++++------------
>> 1 file changed, 12 insertions(+), 12 deletions(-)
>>
>> diff --git a/unpack-trees.c b/unpack-trees.c
>> index 9f55cc2..4bc6b4f 100644
>> --- a/unpack-trees.c
>> +++ b/unpack-trees.c
>> @@ -61,21 +61,21 @@ void setup_unpack_trees_porcelain(struct unpack_trees_options *opts,
>> const char *cmd2 = strcmp(cmd, "checkout") ? cmd : "switch branches";
>>
>> if (advice_commit_before_merge)
>> - msg = "Your local changes to the following files would be overwritten by %s:\n%%s"
>> - "Please, commit your changes or stash them before you can %s.";
>> + msg = _("Your local changes to the following files would be overwritten by %s:\n%%s"
>> + "Please, commit your changes or stash them before you can %s.");
>> else
>> - msg = "Your local changes to the following files would be overwritten by %s:\n%%s";
>> + msg = _("Your local changes to the following files would be overwritten by %s:\n%%s");
>> msgs[ERROR_WOULD_OVERWRITE] = msgs[ERROR_NOT_UPTODATE_FILE] =
>> xstrfmt(msg, cmd, cmd2);
>>
>> msgs[ERROR_NOT_UPTODATE_DIR] =
>> - "Updating the following directories would lose untracked files in it:\n%s";
>> + _("Updating the following directories would lose untracked files in it:\n%s");
>>
>> if (advice_commit_before_merge)
>> - msg = "The following untracked working tree files would be %s by %s:\n%%s"
>> - "Please move or remove them before you can %s.";
>> + msg = _("The following untracked working tree files would be %s by %s:\n%%s"
>> + "Please move or remove them before you can %s.");
>> else
>> - msg = "The following untracked working tree files would be %s by %s:\n%%s";
>> + msg = _("The following untracked working tree files would be %s by %s:\n%%s");
>>
>> msgs[ERROR_WOULD_LOSE_UNTRACKED_REMOVED] = xstrfmt(msg, "removed", cmd, cmd2);
>> msgs[ERROR_WOULD_LOSE_UNTRACKED_OVERWRITTEN] = xstrfmt(msg, "overwritten", cmd, cmd2);
>> @@ -84,14 +84,14 @@ void setup_unpack_trees_porcelain(struct unpack_trees_options *opts,
>> * Special case: ERROR_BIND_OVERLAP refers to a pair of paths, we
>> * cannot easily display it as a list.
>> */
>> - msgs[ERROR_BIND_OVERLAP] = "Entry '%s' overlaps with '%s'. Cannot bind.";
>> + msgs[ERROR_BIND_OVERLAP] = _("Entry '%s' overlaps with '%s'. Cannot bind.");
>>
>> msgs[ERROR_SPARSE_NOT_UPTODATE_FILE] =
>> - "Cannot update sparse checkout: the following entries are not up-to-date:\n%s";
>> + _("Cannot update sparse checkout: the following entries are not up-to-date:\n%s");
>> msgs[ERROR_WOULD_LOSE_ORPHANED_OVERWRITTEN] =
>> - "The following Working tree files would be overwritten by sparse checkout update:\n%s";
>> + _("The following Working tree files would be overwritten by sparse checkout update:\n%s");
>> msgs[ERROR_WOULD_LOSE_ORPHANED_REMOVED] =
>> - "The following Working tree files would be removed by sparse checkout update:\n%s";
>> + _("The following Working tree files would be removed by sparse checkout update:\n%s");
>>
>> opts->show_all_errors = 1;
>> /* rejected paths may not have a static buffer */
>> @@ -168,7 +168,7 @@ static void display_error_msgs(struct unpack_trees_options *o)
>> string_list_clear(rejects, 0);
>> }
>> if (something_displayed)
>> - fprintf(stderr, "Aborting\n");
>> + fprintf(stderr, _("Aborting\n"));
>> }
>>
>> /*
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html