From 2800d516da58091f1b37736900a804f57918ad03 Mon Sep 17 00:00:00 2001 From: Alexander Taylor Date: Sun, 24 Nov 2013 17:47:15 +0000 Subject: [PATCH] Rearranged package.name check to avoid crash Buildozer would previously crash if a package name was '', as it checked for zero length *and* tried to check if the first character was a number. --- buildozer/__init__.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/buildozer/__init__.py b/buildozer/__init__.py index 7bf167d..c872656 100644 --- a/buildozer/__init__.py +++ b/buildozer/__init__.py @@ -300,13 +300,13 @@ class Buildozer(object): adderror = errors.append if not get('app', 'title', ''): adderror('[app] "title" is missing') - if not get('app', 'package.name', ''): - adderror('[app] "package.name" is missing') if not get('app', 'source.dir', ''): adderror('[app] "source.dir" is missing') package_name = get('app', 'package.name', '') - if package_name[0] in map(str, range(10)): + if not package_name: + adderror('[app] "package.name" is missing') + elif package_name[0] in map(str, range(10)): adderror('[app] "package.name" may not start with a number.') version = get('app', 'version', '')