Re: Make prep_status() message translatable

On 2025/04/07 15:55, Kyotaro Horiguchi wrote:
> Hello,
>
> The recent commit 173c97812ff made the following change:
>
> - prep_status("Adding \".old\" suffix to old global/pg_control");
> + prep_status("Adding \".old\" suffix to old " XLOG_CONTROL_FILE);
>
> This change results in a message that is untranslatable, at least into
> Japanese. In addition, the file name should be quoted.
>
> The attached patch modifies the message to use %s for XLOG_CONTROL_FILE,
> making it properly translatable.

Thanks for the report and patch!

The fix looks good to me.

pg_log(PG_REPORT, "\n"
"If you want to start the old cluster, you will need to remove\n"
"the \".old\" suffix from %s/%s.old.\n"
"Because \"link\" mode was used, the old cluster cannot be safely\n"
"started once the new cluster has been started.",
old_cluster.pgdata, XLOG_CONTROL_FILE);

Commit 173c97812ff also updated the above part of disable_old_cluster()
and replaced the hardcoded "global/pg_control" with %s using XLOG_CONTROL_FILE.
Maybe we should also add a translator: comment and wrap %s/%s.old in
double quotes there?

Regards,

--
Fujii Masao
Advanced Computing Technology Center
Research and Development Headquarters
NTT DATA CORPORATION