Skip to content

removed TARGET_SYSTEM_PROP, edit TARGET_COPY_OUT_VENDOR#2

Open
DigiGoon wants to merge 1 commit intoOpenKirin:7.xfrom
DigiGoon:patch-1
Open

removed TARGET_SYSTEM_PROP, edit TARGET_COPY_OUT_VENDOR#2
DigiGoon wants to merge 1 commit intoOpenKirin:7.xfrom
DigiGoon:patch-1

Conversation

@DigiGoon
Copy link

  1. no need for TARGET_SYSTEM_PROP after 7.x
  2. vendor is built as separate image, so no need to cook it into system/vendor

1. no need for TARGET_SYSTEM_PROP after 7.x
2. vendor is built as separate image, so no need to cook it into system/vendor
Copy link

@Huawei-Dev Huawei-Dev left a comment

Choose a reason for hiding this comment

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

we don't want to build vendor, as we use a prebuild one, so we still need "TARGET_COPY_OUT_VENDOR := system"

@DigiGoon
Copy link
Author

So it should be "TARGET_COPY_OUT_VENDOR := system/vendor" no?

@Huawei-Dev
Copy link

I did not understand what problems you are experiencing if you keep : TARGET_COPY_OUT_VENDOR := system ?

@DigiGoon
Copy link
Author

DigiGoon commented Aug 29, 2017

Some ROMs like halogenOS needs it to be either system/vendor or just vendor.
https://github.com/halogenOS/android_build/blob/bcf24aa92067b57c18e987d4d8b9d36cbea34c77/core/envsetup.mk#L180

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants