Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • B buck
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 201
    • Issues 201
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 22
    • Merge requests 22
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Meta
  • buck
  • Merge requests
  • !2140

Fix app bundle support

  • Review changes

  • Download
  • Email patches
  • Plain diff
Closed Administrator requested to merge github/fork/thalescm/thales/add-bundletool into master Dec 13, 2018
  • Overview 40
  • Commits 6
  • Pipelines 0
  • Changes 27

Created by: thalescm

This project shows that the current buck implementation of android app bundles is buggy, and the presented PR fixes it while adding more security and scalability to the feature, as it removes a lot of logic from buck that is already maintained by google (bundletool).

  • Added bundletool to the project, bundle tool is now used to create the final bundle.
    • Bundletool correctly generates android app bundle configuration files (current impl don't)
    • Bundletool correctly adds bundletool version used (on BundleConfig.pb) so playstore knows which one to use when creating apks
    • Bundletool will validate the modules before building the final bundle (this dosn't exist on current implementation, so on success build of android_bundle rule can be a false positive)
  • Moving module files to the right places; as documentation explains: https://developer.android.com/guide/app-bundle/
  • Removed Assets,Native and BundleConfig generation steps, as this is now covered by bundletool itself.
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: github/fork/thalescm/thales/add-bundletool