Skip to content
GitLab
  • Menu
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
    • Switch to GitLab Next
  • Sign in / Register
  • fdroidserver fdroidserver
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 307
    • Issues 307
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
  • Merge requests 52
    • Merge requests 52
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
    • Test Cases
  • Deployments
    • Deployments
    • Releases
  • Packages & Registries
    • Packages & Registries
    • Container Registry
  • External wiki
    • External wiki
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • F-DroidF-Droid
  • fdroidserverfdroidserver
  • Merge requests
  • !591

Still aapt output parsing, adapting regex in updated.py to fix 'fdroid update' for build-tools 28.0.3

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Francesco Cervigni requested to merge neonsoftware/fdroidserver:master into master Nov 13, 2018
  • Overview 14
  • Commits 2
  • Pipelines 10
  • Changes 18

update.py: Still aapt output parsing, setting regex for catching 'name=' to catch only first occurrence in aapt output for build-tools 28.0.3

As mentioned in other issues, build-tools 28.0.3 added a new field in the end 'compileSdkVersionCodename=', which also accidentally ends with the string 'name='. (see sample output below) The purpose of this regex was to catch the 'packageName' field, which is now the first occurence of 'name=', therefore added with non-greedy symbol.

sample aapt output (28.0.3): package: name='com.a.b.app' versionCode='1' versionName='1.0' compileSdkVersion='28' compileSdkVersionCodename='9'

previously, regex was catching second occurence, so '9', then fdroid update fails.

live regex: before after

See merge request !582 (merged) and !582 (merged)

Edited Jan 03, 2019 by Hans-Christoph Steiner
Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: master