chiark / gitweb /
Revert "fix vercodes."
authorDaniel Martí <mvdan@mvdan.cc>
Mon, 20 May 2013 13:48:55 +0000 (15:48 +0200)
committerDaniel Martí <mvdan@mvdan.cc>
Mon, 20 May 2013 13:48:55 +0000 (15:48 +0200)
This (partially) reverts commit 05735e9fca73168cf19a96665de484d7fbb0effd.

fdroidserver/common.py
fdroidserver/update.py

index 856c3965fc1d6e51b5e34438772f596ae3f1b157..606111cb4bbd772e91199131d8a656d83437114b 100644 (file)
@@ -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")
 
index f0818cf90b8a08f2e32204c1a38f67b4c2270321..9cd6d166beffac520ec47d87696b82879190fd96 100644 (file)
@@ -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:]
                     })