-
Notifications
You must be signed in to change notification settings - Fork 216
Description
Hi guys,
Tested on rk3399 evb board(arm64), the component build is broken now.
1/ libmedia_gpu.so failed to link, which is because v4l2_jpeg_encode_accelerator.cc depends on chromeos_camera:
https://chromium.googlesource.com/chromium/src/+blame/HEAD/media/gpu/v4l2/BUILD.gn#102
This patch works:
+++ b/media/gpu/v4l2/BUILD.gn
@@ -32,8 +32,6 @@ source_set("v4l2") {
"v4l2_device.h",
"v4l2_image_processor.cc",
"v4l2_image_processor.h",
- "v4l2_jpeg_encode_accelerator.cc",
- "v4l2_jpeg_encode_accelerator.h",
"v4l2_video_decode_accelerator.cc",
"v4l2_video_decode_accelerator.h",
"v4l2_video_encode_accelerator.cc",
@@ -94,6 +92,8 @@ source_set("v4l2") {
if (is_chromeos) {
sources += [
+ "v4l2_jpeg_encode_accelerator.cc", <--- this depends on chromeos_camera
+ "v4l2_jpeg_encode_accelerator.h",
2/ Failed to generate v8 bin:
out/Release/libfreetype_harfbuzz.so: undefined symbol: uscript_getShortName_64
Which is because the generated libfreetype_harfbuzz.so depends on generated libicuuc.so, and we override LD_LIBRARY_PATH to prefer searching STAGING_DIR_HOST instead of output dir in:
https://github.com/OSSystems/meta-browser/blob/master/recipes-browser/chromium/chromium-gn.inc#L311
Maybe we should add $(B) (output dir) in front of the LD_LIBRARY_PATH.