From 6808d5d5ed3bf444af5bfe79aeda60d7016decc3 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Daniel=20Mart=C3=AD?= Date: Mon, 20 May 2013 15:48:55 +0200 Subject: [PATCH] Revert "fix vercodes." This (partially) reverts commit 05735e9fca73168cf19a96665de484d7fbb0effd. --- fdroidserver/common.py | 8 ++++---- fdroidserver/update.py | 3 ++- 2 files changed, 6 insertions(+), 5 deletions(-) diff --git a/fdroidserver/common.py b/fdroidserver/common.py index 856c3965..606111cb 100644 --- a/fdroidserver/common.py +++ b/fdroidserver/common.py @@ -416,7 +416,7 @@ def parse_metadata(metafile, **kw): thisbuild['version'] = parts[0] thisbuild['vercode'] = parts[1] try: - thisbuild['vercode'] = int(thisbuild['vercode']) + testvercode = int(thisbuild['vercode']) except: raise MetaDataException("Invalid version code for build in " + metafile.name) thisbuild['commit'] = parts[2] @@ -617,7 +617,7 @@ def write_metadata(dest, app): # Keeping the original formatting if we loaded it from a file... mf.write('\\\n'.join(build['origlines']) + '\n') else: - mf.write(build['version'] + ',' + str(build['vercode']) + ',' + + mf.write(build['version'] + ',' + build['vercode'] + ',' + build['commit']) for key,value in build.iteritems(): if key not in ['version', 'vercode', 'commit']: @@ -861,7 +861,7 @@ def parse_androidmanifest(app_dir): if not vercode: matches = vcsearch(line) if matches: - vercode = int(matches.group(1)) + vercode = matches.group(1) if version: return (version, vercode, package) for xmlfile in glob.glob(app_dir + '/res/values/strings*transl*.xml'): @@ -1116,7 +1116,7 @@ def prepare_source(vcs, app, build, build_dir, srclib_dir, extlib_dir, sdk_path, raise BuildException("Failed to amend manifest") if 'forcevercode' in build: if subprocess.call(['sed','-r','-i', - 's/android:versionCode="[^"]+"/android:versionCode="' + str(build['vercode']) + '"/g', + 's/android:versionCode="[^"]+"/android:versionCode="' + build['vercode'] + '"/g', 'AndroidManifest.xml'], cwd=root_dir) !=0: raise BuildException("Failed to amend manifest") diff --git a/fdroidserver/update.py b/fdroidserver/update.py index f0818cf9..9cd6d166 100644 --- a/fdroidserver/update.py +++ b/fdroidserver/update.py @@ -86,7 +86,8 @@ def update_wiki(apps, apks, verbose=False): if thisbuild['vercode'] == app['Current Version Code']: cantupdate = True apklist.append({ - 'versioncode': thisbuild['vercode'], + #TODO: Nasty: vercode is a string in the build, and an int elsewhere + 'versioncode': int(thisbuild['vercode']), 'version': thisbuild['version'], 'buildproblem': thisbuild['commit'][1:] })