Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

For dynamic partitions devices. Like poco X2(phoenix) and Redmi Note 9s(Curtana) #10

Open
wants to merge 2 commits into
base: android-10
Choose a base branch
from

Conversation

ghost
Copy link

@ghost ghost commented Jul 31, 2020

No description provided.

PeterCxy and others added 2 commits July 31, 2020 01:43
* Before this commit, the generated `dynamic_partitions_op_list` in
  FullOTA packages always tries to remove all partitions and recreate
  them upon flashing. This makes it impossible to have a system-only
  "FullOTA" because vendor partition(s) are always removed.

* This commit detects if a build is vendor-less and disables every
  dynamic partition operation except `resize`, in order to keep the
  original content around after the flash. The change should not affect
  non-dynamic-partition or builds with vendor image included.

Change-Id: I0cded7f3b2958f35103d73d19b7fb5f292f6c17f
Signed-off-by: Jesse Chan <jc@lineageos.org>
Currently only the system and vendor partitions are updated in the
generic block based OTA generation script. Since the product partition
is quite large and consist most of APK files, it would be beneficial to
update it similar to system. Handle the odm and system_ext partitions in
the same way as well.

Bug: 132683080
Test: Run unit tests, generate full and incremental OTA with product partitions.
Change-Id: I13478cf9bd32137c6729b8c9cb102080147093f2
@@ -1680,6 +1754,13 @@ def WriteBlockIncrementalOTAPackage(target_zip, source_zip, output_file):
brotli=OPTIONS.brotli)
else:
vendor_diff = None
=======

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bad Cherry-pick :(
Missed these out

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants