No. I did not compile it as I have no working environment and not enough time to care for one. These are a suggestion and clearly marked as suggestion. But are you sure that it does not work? Gettext simply uses a text table lookup. So when there is a translation for the text (which is for building names), then a call to the translate function should translate it. Only issue could be the different translation contexts you use in widelands which complicate things.
> I am not sure about your changes in rev 6383, as I am not that deep into lua. Were those lines untranslated without those changes?
These changes are requires. The commit comment tells you why.
> Did you compile and test whether it works?
No. I did not compile it as I have no working environment and not enough time to care for one. These are a suggestion and clearly marked as suggestion. But are you sure that it does not work? Gettext simply uses a text table lookup. So when there is a translation for the text (which is for building names), then a call to the translate function should translate it. Only issue could be the different translation contexts you use in widelands which complicate things.
> I am not sure about your changes in rev 6383, as I am not that deep into lua. Were those lines untranslated without those changes?
These changes are requires. The commit comment tells you why.