<div dir="ltr"><div class="gmail_quote"><div>Hi,</div><div><br></div><div>This version of the patch is looking good to me. I noticed a few intltool-related things you can remove from .gitignore: /intltool-*, /po/.intltool-merge-cache, /m4/intltool.m4.</div><div dir="ltr"><br></div><div dir="ltr">On Tue, Aug 29, 2017 at 1:55 PM Aleksander Morgado <<a href="mailto:aleksander@aleksander.es">aleksander@aleksander.es</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hey Daiki,<br>
<br>
On Tue, Aug 29, 2017 at 4:20 PM, Daiki Ueno <<a href="mailto:ueno@gnu.org" target="_blank">ueno@gnu.org</a>> wrote:<br>
> Hello,<br>
><br>
> (Sending this off-list, as the downloadable archive is not updated yet:<br>
> <a href="https://lists.freedesktop.org/archives/modemmanager-devel/2017-August.txt.gz" rel="noreferrer" target="_blank">https://lists.freedesktop.org/archives/modemmanager-devel/2017-August.txt.gz</a>)<br>
><br>
<br>
Adding the mailing list in CC of this reply.<br>
<br>
> Here are quick comments:<br>
><br>
>>  data/polkit-policy-i18n.its                     |  6 ++<br>
>>  data/polkit-policy-i18n.loc                     |  4 ++<br>
><br>
> To use custom ITS scripts in your package, you need to put them under<br>
> "its/" directory and adjust GETTEXTDATADIR pointing to the parent<br>
> directory.  In this particular case, you might want to put them under:<br>
><br>
>   data/its/polkit-policy-i18n.its<br>
>   data/its/polkit-policy-i18n.loc<br>
><br>
> and set GETTEXTDATADIR around the xgettext and msgfmt calls:<br>
><br>
>   GETTEXTDATADIR=$(top_srcdir)/data<br>
><br>
<br>
Is this mandatory? It looks like adding them under data/ directly<br>
worked ok, without any explicit GETTEXTDATADIR.<br>
<br>
> By the way, have you seen the upstream ITS scripts?<br>
><br>
> <a href="https://cgit.freedesktop.org/polkit/tree/data/polkit.its" rel="noreferrer" target="_blank">https://cgit.freedesktop.org/polkit/tree/data/polkit.its</a><br>
> <a href="https://cgit.freedesktop.org/polkit/tree/data/polkit.loc" rel="noreferrer" target="_blank">https://cgit.freedesktop.org/polkit/tree/data/polkit.loc</a><br>
><br>
<br>
Oh, I didn't see them, nice to see they're basically the same as what<br>
I did :) What's the way to go with this? Should I import those files<br>
into the MM sources, or reference somehow the ones installed in<br>
/usr/share/gettext/its/, or none of those required?<br></blockquote><div><br></div><div>I would suggest referencing the ones installed in the system, if that's possible. It may not be possible if you need to generate the translated policy files even when configured --without-polkit.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">>> --- a/<a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a><br>
>> +++ b/<a href="http://configure.ac" rel="noreferrer" target="_blank">configure.ac</a><br>
>> @@ -82,15 +82,15 @@ dnl-----------------------------------------------------------------------------<br>
>>  dnl i18n<br>
>>  dnl<br>
>><br>
>> -IT_PROG_INTLTOOL([0.40.0])<br>
>> -<br>
>>  AM_GNU_GETTEXT([external])<br>
>> -AM_GNU_GETTEXT_VERSION([0.19.3])<br>
>> +AM_GNU_GETTEXT_VERSION([0.19.8])<br>
>><br>
>>  GETTEXT_PACKAGE=ModemManager<br>
>>  AC_SUBST(GETTEXT_PACKAGE)<br>
>>  AC_DEFINE_UNQUOTED(GETTEXT_PACKAGE,"$GETTEXT_PACKAGE", [Gettext package])<br>
>><br>
>> +AC_PATH_PROG([MSGFMT], [msgfmt])<br>
><br>
> This should be already set by AM_GNU_GETTEXT.<br>
><br>
<br>
Ok, will remove it<br>
<br>
>> --- a/data/Makefile.am<br>
>> +++ b/data/Makefile.am<br>
>> @@ -61,37 +61,36 @@ diagrams = \<br>
>><br>
>><br>
>>  # Polkit<br>
>> -polkit_policy_in_in_files = <a href="http://org.freedesktop.ModemManager1.policy.in.in" rel="noreferrer" target="_blank">org.freedesktop.ModemManager1.policy.in.in</a><br>
>>  if WITH_POLKIT<br>
>> +org.freedesktop.ModemManager1.policy: <a href="http://org.freedesktop.ModemManager1.policy.in" rel="noreferrer" target="_blank">org.freedesktop.ModemManager1.policy.in</a><br>
>> +     $(AM_V_GEN) $(MSGFMT) --xml -d $(top_srcdir)/po/ -o $@ --template $<<br>
> [...]<br>
>>  DISTCLEANFILES = \<br>
>> +     org.freedesktop.ModemManager1.policy \<br>
><br>
> I see that .<a href="http://policy.in.in" rel="noreferrer" target="_blank">policy.in.in</a> is expanded twice: first by AC_CONFIG_FILES,<br>
> and then by msgfmt.  That would require msgfmt installed on user's<br>
> system.<br>
><br>
> It's probably reasonable to assume msgfmt is available everywhere these<br>
> days, but if you ever want to avoid the dependency, you could:<br>
><br>
> - first expand translations with msgfmt and add resulting .<a href="http://policy.in" rel="noreferrer" target="_blank">policy.in</a> to<br>
>   EXTRA_DIST<br>
> - secondly, expand variables in .<a href="http://policy.in" rel="noreferrer" target="_blank">policy.in</a> with sed at 'make' time,<br>
>   instead of AC_CONFIG_FILES<br>
><br>
<br>
I had that order of generation in the v2 patch of the file actually,<br>
so may just switch back to it.<br>
<br>
Thanks for checking this!<br>
<br>
--<br>
Aleksander<br>
<a href="https://aleksander.es" rel="noreferrer" target="_blank">https://aleksander.es</a><br>
</blockquote></div></div>