Re: [PATCH v3][Outreachy] branch -D: allow - as abbreviation of @{-1}

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Apr 5, 2016 at 12:53 AM, Remi Galan Alfonso
<remi.galan-alfonso@xxxxxxxxxxxxxxxxxxxxxxx> wrote:
> elena petrashen <elena.petrashen@xxxxxxxxx> wrote:
>> On Thu, Mar 31, 2016 at 6:31 PM, Remi Galan Alfonso
>> <remi.galan-alfonso@xxxxxxxxxxxxxxxxxxxxxxx> wrote:
>> > Elena Petrashen <elena.petrashen@xxxxxxxxx> wrote:
>> >> +void delete_branch_advice(const char *name, const char *ref)
>> >> +{
>> >> +        const char fmt[] =
>> >> +        "\nNote: to restore the deleted branch:\n\ngit branch %s %s\n";
>> >
>> > Shouldn't that be marked for translation, like is done with the other
>> > strings?
>> >
>> > Thanks,
>> > Rémi
>>
>> Thank you for letting me know about that! Could you please
>> help me out and explain how do I mark it for translation? I tried
>> to do it the same way as with the other strings but evidently
>> didn't quite succeed.
>
> I am not sure.
> I tried to grep similar cases, it seems that you can do the following:
>
>         const char fmt[] = N_("\nNote: to restore [...] \ngit branch %s %s\n");
>         fprintf(stderr, _(fmt), name, ref);
>
> Some similar example in builtin/add.c:
>
>         static const char ignore_error[] =
>         N_("The following paths are ignored by one of your .gitignore files:\n");
>         [...]
>                         fprintf(stderr, _(ignore_error));
>
> Or you can define fmt as a 'const char *' and in that case do the
> following:
>
>         const char *fmt = _("\nNote: to restore [...] \n git branch %s %s\n");
>         fprintf(stderr, fmt, name, ref);
>
>
> In builtin/am.c:
>                 const char *invalid_line = _("Malformed input line: '%s'.");
>         [...]
>                                 ret = error(invalid_line, sb.buf);
>
> I don't know which one is the best way to go though.
>
> Thanks,
> Rémi

Thank you! I'm a bit confused though as the previous implemented advice function
in advice.c (the one I used for reference) doesn't seem to employ this... does
this mean it's also not marked for translation?

void detach_advice(const char *new_name)
{
const char fmt[] =
"Note: checking out '%s'.\n\n"
"You are in 'detached HEAD' state. <.........>Example:\n\n"
"  git checkout -b <new-branch-name>\n\n";

fprintf(stderr, fmt, new_name);
}

Cheers,
Elena
--
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



[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]