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

mk/aosp_optee.mk: define OPTEE_BIN for path of tee.bin #1922

Merged
merged 1 commit into from
Nov 9, 2017

Conversation

liuyq
Copy link
Contributor

@liuyq liuyq commented Nov 8, 2017

so that other android projects could use OPTEE_BIN
as the dependency instead of the old BUILD_OPTEE_OS target.

Ths is workaround for the problem with following with aosp master:
external/optee_test/Android.mk: error: xtest: LOCAL_ADDITIONAL_DEPENDENCIES must only contain paths (not module names)

Reviewed-by: Victor Chong victor.chong@linaro.org

Tested by: Victor Chong victor.chong@linaro.org (hikey aosp)
Tested-by: Yongqin Liu yongqin.liu@linaro.org (hikey aosp master)

Signed-off-by: Yongqin Liu yongqin.liu@linaro.org

@vchong
Copy link
Contributor

vchong commented Nov 8, 2017

Reviewed-by: Victor Chong <victor.chong@linaro.org>

@vchong
Copy link
Contributor

vchong commented Nov 8, 2017

Tested by: Victor Chong <victor.chong@linaro.org> (hikey aosp)

so that other android projects could use OPTEE_BIN
as the dependency instead of the old BUILD_OPTEE_OS target.

Ths is workaround for the problem with following with aosp master:
external/optee_test/Android.mk: error: xtest: LOCAL_ADDITIONAL_DEPENDENCIES must only contain paths (not module names)

Reviewed-by: Victor Chong <victor.chong@linaro.org>

Tested by: Victor Chong <victor.chong@linaro.org> (hikey aosp)
Tested-by: Yongqin Liu <yongqin.liu@linaro.org> (hikey aosp master)

Signed-off-by: Yongqin Liu <yongqin.liu@linaro.org>
@liuyq
Copy link
Contributor Author

liuyq commented Nov 9, 2017

Hi,
Review Tags from @vchong added. OK for merge?

@jforissier jforissier merged commit 2ef721a into OP-TEE:master Nov 9, 2017
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