Re: [PATCH v8 04/37] hook: include hookdir hook in list

On Wed, Mar 24, 2021 at 3:12 PM Junio C Hamano [off-list ref] wrote:

Not just we do not want "hookdir" placed inside _(),

                        printf("%s: %s\n",
+                              (item->from_hookdir
+                               ? "hookdir"
+                               : config_scope_name(item->origin)),
                               item->command.buf);

we do not want the "%s: %s\n" to be placed inside _() and get munged
into "%2$s: %1$s\n" for languages that want the order swapped, for
example.

So perhaps the comment should be about the entire output, i.e.
"don't translate the output from this helper, as it is meant to be
machine parseable", or something?

Having the word "translate" in the comment automatically implies
localization, which confuses the issue. It would be clearer to avoid
that word altogether. Perhaps something along the lines of:

    /* machine-parseable output; do not apply _() localization */