Merge lp:~dholbach/help-app/prerequisites into lp:~ubuntu-touch-coreapps-drivers/help-app/trunk

[Daniel Holbach]

Give more information and fail more gently if prerequisites like po4a or polib are missing.

[Nicholas Skaggs (nskaggs)]

One minor tweak

review: Needs Fixing

[Nicholas Skaggs (nskaggs)]

There is an unrelated issue with the HACKING doc, I'll ask you to sneak in:

You can find the updated HTML in ./app/www/. For instance, you can browse
the site in firefox with

firefox app/www/index.html

^^ This is incorrect now. We could say

firefox app/www/index.en-us.html

or

firefox app/www/index*.html

or remove the specific recommendation.

[lp:~dholbach/help-app/prerequisites]

58. By Daniel Holbach on 2015-02-26

use shutil.which(), thanks Nick!

[Daniel Holbach (dholbach)]

app/www/index.html will be re-added through 1425025.

[Daniel Holbach (dholbach)]

<dholbach> balloons, the index.html will come back
 in fact I pushed a small branch to start work on bug 1425025
<ubot5> bug 1425025 in Help for Ubuntu for devices "Automatically load translated pages according to the user language" [High,Triaged] https://launchpad.net/bugs/1425025
<dholbach> can I merge the prerequisites branch now?
<balloons> dholbach, yep go for it

review: Approve

=== modified file 'edit-here/translations.py'
--- edit-here/translations.py	2015-02-26 11:59:36 +0000
+++ edit-here/translations.py	2015-02-26 15:56:37 +0000
@@ -2,10 +2,17 @@
 import copy
 import glob
 import os
-import polib
 import re
 import shutil
 import subprocess
+import sys
+
+try:
+    import polib
+except ImportError:
+    print('Missing prerequisites. Please run: '
+          '"sudo apt install python3-polib"')
+    sys.exit(1)
 
 from pelicanconf import PATH
 
@@ -27,6 +34,9 @@
             '-M', 'utf-8',
             ]
         self.pot_file = pot_file
+        if not shutil.which('po4a'):
+            print('Missing prerequisites. Please run: "sudo apt install po4a"')
+            sys.exit(1)
 
     def run(self, po4a_command, additional_args, with_output=False):
         args = copy.copy(self.default_args)