chinese/ibus-pinyin: Fixed build with NLS=off

[yuri]

Summary

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=219603

[tcberner]

chinese/ibus-pinyin/Makefile

20 ↗

(On Diff #37399)

so what is the point of having an NLS option, if you have to depend on gettext anyways?

[yuri]

chinese/ibus-pinyin/Makefile

20 ↗

(On Diff #37399)

It's a bug in GNU toolchain. It has files with double .in suffix (.in.in) that are processed by gettext unconditionally. The point of the NLS option is that translation files aren't installed when NLS=off. One way to handle this is to remove NLS and have it always on. But I would rather see the option that removes files.

[yuri]

This port has been breaking with the blanked NLS=off rule. There is no way we can fix GNU toolchain. So there are two options: kill NLS altogether, or keep it being able to disable the installed files.
I have a preference for the latter choice. Is this perfect? Not at all. This will never be perfect. It's just a pragmatic choice.

There's nothing more to this matter. :-)

[tcberner]

Please double check with the maintainer that this change is what they accepted.

From my side it is fine (although I would rather have the option gone completely ^^).

[yuri]

In D13736#289574, @tcberner wrote:

Please double check with the maintainer that this change is what they accepted.

Yes, I am positive that this is what he accepted. :-)

[mat]

Adding all of gettext when NLS is disabled is most certainly bogus.

This should probably be:

NLS_USES=     gettext-runtime
NLS_USES_OFF= gettext-tools