FTBFS when merging latest translations

Bug #1360610 reported by Hans Joachim Desserud
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
widelands
Fix Released
High
Hans Joachim Desserud

Bug Description

I saw that the daily builds PPA [1] is currently failing to build, even though there hasn't been any code changes since the last successful build.

This narrows down the possible causes quite a bit, so I experimented a bit in my test PPA where I was able to build the latest revision, but it would fail if I merged in the translations branch [2].

I then tried locally what would happen if I attempted to merge the translation branch and rebuild:
[307/647] Generating translations for locale it category world_blackland
FAILED: cd /home/user/widelands/build/po && cmake -E make_directory /home/user/widelands/build/locale/it/LC_MESSAGES && msgmerge -q --no-wrap /home/user/widelands/po/world_blackland/it.po /home/user/widelands/po/world_blackland/world_blackland.pot -o /home/user/widelands/build/po/it_world_blackland.tmp.po && msgfmt -o /home/user/widelands/build/locale/it/LC_MESSAGES/world_blackland.mo /home/user/widelands/build/po/it_world_blackland.tmp.po && cmake -E remove /home/user/widelands/build/po/it_world_blackland.tmp.po
msgmerge: feil under åpning av «/home/user/widelands/po/world_blackland/world_blackland.pot» for lesing: Ingen slik fil eller filkatalog
[307/647] Generating translations for locale it category texts
ninja: build stopped: subcommand failed.

So it looks like it fails due to translations of the old blackland world which doesn't exist in the repo. Not quite sure how to resolve this, but in the mean time I'll remove
merge translations lp:~sirver/widelands/translations
from the recipe so the PPA won't fail everytime it tries to build something.

[1] https://code.launchpad.net/~widelands-dev/+recipe/widelands-daily
[2] https://code.launchpad.net/~hjd/+recipe/widelands-test

Revision history for this message
SirVer (sirver) wrote :

I fixed the errors in the translations branch and verified that trunk now builds again without any problems. Could you please reenable merging the translations in the PPAs?

Changed in widelands:
status: New → Triaged
assignee: nobody → Hans Joachim Desserud (hjd)
Revision history for this message
Hans Joachim Desserud (hjd) wrote :

Thanks for sorting this out.

>Could you please reenable merging the translations in the PPAs?

I would like to, but I'm worried that translations might break the build in the future too. Won't it simply fail the same way the next time someone adds a translation to one of the "don't translate" templates? One thing is trunk breaking due to code changes, but I don't see why unmerged translations should break the build. (Naturally those changes need to be fixed before they are merged to trunk, but still...)

Revision history for this message
SirVer (sirver) wrote :

Yes, I think that could indeed be the case. So are you suggesting that we should not merge the translations automatically, but instead merge them more often into trunk or so?

Revision history for this message
GunChleoc (gunchleoc) wrote :

I have renamed and deactivated the templates. I can't delete them, so I've asked a Lanchpad question.

https://answers.launchpad.net/launchpad/+question/254133

Revision history for this message
Hans Joachim Desserud (hjd) wrote :

GunChleoc: Ok. I saw from your question that it's not possible to remove them completely, though since they aren't visible among the other translation templates it would require some effort to find them.

I've re-added merging translations to the PPAs after verifying the current translation branch worked fine in my test PPA.

SirVer: I thinking merging translations more often would be nice. Sometimes it can take months before the latest changes makes their way into trunk which means it takes longer before people can try how the translations work in practice/context of the game, spot typos etc. So I think that would be an improvement in general. We can still merge the latest translations automatically into the PPA, but I didn't want them to break a build which would otherwise be fine without them.

Changed in widelands:
status: Triaged → Fix Committed
Revision history for this message
SirVer (sirver) wrote :

I put something on my calendar. I try to merge translations once per month now.

Revision history for this message
GunChleoc (gunchleoc) wrote :

Let me know if I can do anything to help with the merging.

GunChleoc (gunchleoc)
tags: added: linux
removed: ppa
GunChleoc (gunchleoc)
Changed in widelands:
status: Fix Committed → Fix Released
Revision history for this message
GunChleoc (gunchleoc) wrote :

Fixed in build19-rc1.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Related questions

Remote bug watches

Bug watches keep track of this bug in other bug trackers.