Skip to content

Commit

Permalink
build: Add SKIP_SET_METADATA flag
Browse files Browse the repository at this point in the history
On some of the older devices the recovery doesn't support
metadata when setting permissions. This allows to use
set_perm vs set_metadata.
Enable this behavior by setting SKIP_SET_METADATA := true

Change-Id: Ib6cf97b5ca28cee8a87975231f3a61b156e77d29
  • Loading branch information
flintman authored and amartinz committed Jan 5, 2014
1 parent e4757ba commit 437843b
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions core/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -1404,7 +1404,9 @@ ifdef PRODUCT_EXTRA_RECOVERY_KEYS
$(hide) echo "extra_recovery_keys=$(PRODUCT_EXTRA_RECOVERY_KEYS)" >> $(zip_root)/META/misc_info.txt
endif
$(hide) echo "mkbootimg_args=$(BOARD_MKBOOTIMG_ARGS)" >> $(zip_root)/META/misc_info.txt
ifneq ($(SKIP_SET_METADATA),true)

This comment has been minimized.

Copy link
@beegee-tokyo

beegee-tokyo Jun 9, 2014

This single change cause a problem for older devices with Recovery that doesn't support set_metadata.
If possible, please take back "ifneq ($(SKIP_SET_METADATA),true)" for support for older devices. Thank you.

This comment has been minimized.

Copy link
@kasperhettinga

kasperhettinga Jun 9, 2014

I reuploaded this patch to gerrit:
https://gerrit.nameless-rom.org/#/c/6234/

$(hide) echo "use_set_metadata=1" >> $(zip_root)/META/misc_info.txt
endif
$(hide) echo "update_rename_support=1" >> $(zip_root)/META/misc_info.txt
$(call generate-userimage-prop-dictionary, $(zip_root)/META/misc_info.txt)
@# Zip everything up, preserving symlinks
Expand Down

0 comments on commit 437843b

Please sign in to comment.