ubuntu-packaging-guide-team team mailing list archive

[Daniel Holbach <daniel.holbach@xxxxxxxxxx>]

Daniel Holbach has proposed merging lp:~dholbach/ubuntu-packaging-guide/fix-add-languages-script into lp:ubuntu-packaging-guide.

Requested reviews:
  Ubuntu Packaging Guide Team (ubuntu-packaging-guide-team)

For more details, see:
https://code.launchpad.net/~dholbach/ubuntu-packaging-guide/fix-add-languages-script/+merge/122535
-- 
https://code.launchpad.net/~dholbach/ubuntu-packaging-guide/fix-add-languages-script/+merge/122535
Your team Ubuntu Packaging Guide Team is requested to review the proposed merge of lp:~dholbach/ubuntu-packaging-guide/fix-add-languages-script into lp:ubuntu-packaging-guide.

=== modified file 'debian/scripts/add-languages'
--- debian/scripts/add-languages	2012-08-21 19:52:28 +0000
+++ debian/scripts/add-languages	2012-09-03 14:44:21 +0000
@@ -62,8 +62,13 @@
         complete = []
         for language in languages:
             pofile = polib.pofile(language)
-            if pofile.percent_translated() >= PERCENTAGE:
+            percentage = pofile.percent_translated()
+            print os.path.basename(language), ("(%s" % percentage)+"%),",
+            if percentage >= PERCENTAGE:
+                print "adding."
                 complete += [ language ]
+            else:
+                print "NOT adding."
         return complete
 
     def get_semi_complete_languages(self):
@@ -73,8 +78,8 @@
     def get_available_binary_packages(self):
         BLACKLIST = [u'ubuntu-packaging-guide']     # metapackages and stuff
         self.stanzas = deb822.Packages.iter_paragraphs(open(self.control_file))
-        return filter(lambda a: a.has_key("Package") and \
-                                a["Package"] not in BLACKLIST, self.stanzas)
+        return filter(lambda a: "Package" in a.keys() and \
+                                a.get("Package") not in BLACKLIST, self.stanzas)
 
     def get_available_binary_package_names(self):
         return map(lambda a: a["Package"],