Re: [PATCH 0/3] [doc] Remove gittutorial-2

[Junio C Hamano <gitster@pobox.com>]

"Julia Evans via GitGitGadget" <gitgitgadget@gmail.com> writes:

>  Documentation/MyFirstObjectWalk.adoc |   2 +-
>  Documentation/git.adoc               |   2 +-
>  Documentation/gitcore-tutorial.adoc  |   1 -
>  Documentation/gitcvs-migration.adoc  |   2 +-
>  Documentation/gitglossary.adoc       |   1 -
>  Documentation/gittutorial-2.adoc     | 422 +--------------------------
>  Documentation/gittutorial.adoc       |  23 +-
>  command-list.txt                     |   1 -
>  po/bg.po                             |   3 -
>  po/ca.po                             |   4 -
>  po/de.po                             |   3 -
>  po/el.po                             |   4 -
>  po/es.po                             |   3 -
>  po/fr.po                             |   3 -
>  po/ga.po                             |   3 -
>  po/id.po                             |   3 -
>  po/it.po                             |   4 -
>  po/ko.po                             |   3 -
>  po/pl.po                             |   3 -
>  po/pt_PT.po                          |   4 -
>  po/ru.po                             |   3 -
>  po/sv.po                             |   3 -
>  po/tr.po                             |   3 -
>  po/uk.po                             |   3 -
>  po/vi.po                             |   3 -
>  po/zh_CN.po                          |   4 -
>  po/zh_TW.po                          |   4 -
>  27 files changed, 14 insertions(+), 503 deletions(-)

One thing I forgot to mention.

I think we try to stay out of the po/ directory unless the patch is
about updating translated text to catch up with translatable text
that was updated by code or doc changes.  IOW, unless you are
working on the patch set as a member of the l10n team, you do not
touch these files.  This is to avoid unnecessary churn and
burdening the l10n teams with synchronization pain.

The above is my understanding of the workflow agreed on by people
inside and outside the l10n group, but I'd like to double-check
with the i18n coordinator (Cc'ed).  If I understand correctly, the
recent workflow used by the l10n teams gives more autonomy to
individual teams than before, which may have changed the equation.

The removals we see in the diffstat touch lines in the early part
that appears in the manual page, namely ...

    gittutorial-2(7)
    ================

    NAME
    ----
    gittutorial-2 - A tutorial introduction to Git: part two

    SYNOPSIS

the string used for "NAME", and one of them looks like this:

        diff --git c/po/es.po w/po/es.po
        index aa1bb9bf90..dcdcbf5360 100644
        --- c/po/es.po
        +++ w/po/es.po
        @@ -14046,9 +14046,6 @@ msgstr "Montar un repositorio dentro de otro"
         msgid "A tutorial introduction to Git"
         msgstr "Un tutorial de introducción a Git"
        -msgid "A tutorial introduction to Git: part two"
        -msgstr "Un tutorial de introducción a Git: parte dos"
        -
         msgid "Git web interface (web frontend to Git repositories)"
         msgstr "Interfaz web Git (interfaz web para repositorios Git)"

I'd say a patch set like this one, which is not about updating the
localization, should just leave po/ intact, since the removal does
not help without a corresponding addition from the same "NAME" in the
file that replaces this "gittutorial-2", which reads like

    gittutorial-2(7)
    ================

    NAME
    ----
    gittutorial-2 - Obsolete tutorial

    SYNOPSIS

and as the patch set stands, we are leaving it to the l10n teams
anyway to add "Obsolete tutorial" to the set of translatable
strings.

Jiang Xin (i18n/l10n coordinator), what do you and the l10n teams
want to see in a patch like this one?

Thanks.