-
Notifications
You must be signed in to change notification settings - Fork 302
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
va/Android.mk: Fixes for compilation #728
base: master
Are you sure you want to change the base?
Conversation
* defined to /system/etc by default, since anyone building libva-android can install a configuration file there.
* These libraries aren't exactly supposed to end up in /vendor, https://source.android.com/docs/core/architecture/vndk defined here. Only vendor specific items are supposed to be in /vendor.
* Allows libva-utils to compile with libva-android as well
@@ -25,8 +25,8 @@ | |||
|
|||
LOCAL_PATH:= $(call my-dir) | |||
|
|||
LIBVA_DRIVERS_PATH_32 := /vendor/lib/dri | |||
LIBVA_DRIVERS_PATH_64 := /vendor/lib64/dri | |||
LIBVA_DRIVERS_PATH_32 := /system/lib/dri |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
libva should be put at vendor partition since system partition is for android AOSP and can be updated seperately with vendor partition.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
libva is a vendor specified module and should be in vendor partition.
@@ -51,14 +51,14 @@ LOCAL_CFLAGS_64 += \ | |||
|
|||
LOCAL_CFLAGS := \ | |||
$(IGNORED_WARNNING) \ | |||
-DSYSCONFDIR=\"/system/etc\" \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this can be put at /vendor/etc/
No description provided.