New issue
Advanced search Search tips

Issue 901221 link

Starred by 1 user
Status: Verified
Owner:
hiroh@chromium.org
Closed: Nov 19
Cc:
derat@chromium.org
keiichiw@chromium.org
nya@chromium.org
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug

Blocking:
issue 889496



Sign in to add a comment

Install libvpx in Chrome OS test image

Project Member Reported by hiroh@chromium.org, Nov 2 
video.EncodeAccel* is the test to encode yuv raw data to some codecs (e.g. H264/VP8).
YUV raw data is not compressed and thus the data size tends to be large.
Because tast needs to put them in test image, it is mostly infeasible to put the YUV raw data.

To resolve this problem, we put data compressed by vp9 and decode them to yuv data by "libvpx".
We should install libvpx in test image by default for this task.

Comment 1 by nya@chromium.org, Nov 2 

Sounds good to me. Please remember to make sure in advance libvpx is helpful in your situation :)

Comment 2 by hiroh@chromium.org, Nov 2

Blocking: 889496
Project Member

Comment 3 by bugdroid1@chromium.org, Nov 3 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/portage-stable/+/71174ae98881368831622e509cc8ae30ef9f725b

commit 71174ae98881368831622e509cc8ae30ef9f725b
Author: Hirokazu Honda <hiroh@chromium.org>
Date: Sat Nov 03 10:39:54 2018

media-libs/libvpx: Uprev to 1.7.0

BUG= chromium:901221 
TEST=./build_package && ./build_image

Change-Id: Ia59cb3f4993aadc92293ef2cbe20952e29040871
Reviewed-on: https://chromium-review.googlesource.com/1314274
Commit-Ready: Hirokazu Honda <hiroh@chromium.org>
Tested-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Chirantan Ekbote <chirantan@chromium.org>

[modify] https://crrev.com/71174ae98881368831622e509cc8ae30ef9f725b/media-libs/libvpx/metadata.xml
[delete] https://crrev.com/23d80e3a072d0be91f0970109e7ff324f1f37daf/media-libs/libvpx/files/libvpx-1.1.0-arm.patch
[add] https://crrev.com/71174ae98881368831622e509cc8ae30ef9f725b/media-libs/libvpx/libvpx-1.7.0.ebuild
[delete] https://crrev.com/23d80e3a072d0be91f0970109e7ff324f1f37daf/media-libs/libvpx/files/libvpx-1.1.0-generic-gnu-shared.patch
[delete] https://crrev.com/23d80e3a072d0be91f0970109e7ff324f1f37daf/media-libs/libvpx/libvpx-1.1.0.ebuild
[add] https://crrev.com/71174ae98881368831622e509cc8ae30ef9f725b/media-libs/libvpx/files/libvpx-1.3.0-sparc-configure.patch
[delete] https://crrev.com/23d80e3a072d0be91f0970109e7ff324f1f37daf/media-libs/libvpx/files/libvpx-1.1.0-chost.patch
[modify] https://crrev.com/71174ae98881368831622e509cc8ae30ef9f725b/media-libs/libvpx/Manifest
Project Member

Comment 5 by bugdroid1@chromium.org, Nov 8 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/491b15b4a4a0bd68c21e73eefc0bd425ce3aa582

commit 491b15b4a4a0bd68c21e73eefc0bd425ce3aa582
Author: Hirokazu Honda <hiroh@chromium.org>
Date: Thu Nov 08 08:48:14 2018

license_lib.py: Hard-code license of yasm package

yasm's LICENSE is "BSD-2 BSD || ( Artistic GPL-2 LGPL-2 )"
It is necessary to hard-code the license in license_lib.py, because license_lib.py cannot parse
LICENSE that contains "||" in the middle.

BUG= chromium:901221 
TEST=emerge yasm

Change-Id: Ie72fd74086eae7189b6a1a2fd27d2716e39c3d72
Reviewed-on: https://chromium-review.googlesource.com/1319355
Commit-Ready: Hirokazu Honda <hiroh@chromium.org>
Tested-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Don Garrett <dgarrett@chromium.org>

[modify] https://crrev.com/491b15b4a4a0bd68c21e73eefc0bd425ce3aa582/licensing/licenses_lib.py

Comment 6 by weifangsun@chromium.org, Nov 8 

<UI triage> Bug owners, please add the appropriate component to your bug. Thanks!
Project Member

Comment 7 by bugdroid1@chromium.org, Nov 18 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/6a3c079fb1fab7be450de2b55b40efa74f44de8a

commit 6a3c079fb1fab7be450de2b55b40efa74f44de8a
Author: Hirokazu Honda <hiroh@chromium.org>
Date: Sun Nov 18 07:02:50 2018

Install libvpx to test image

By this change, libvpx.so is put in /usr/local/lib, and vpxdec and vpxenc are
installed in /usr/local/bin.

BUG= chromium:901221 
TEST=./build_package && ./build_image
CQ-DEPEND=CL:1319355

Change-Id: Ibf96366f01ec36a50978f47d135770cbc6024bc5
Reviewed-on: https://chromium-review.googlesource.com/1314082
Commit-Ready: Hirokazu Honda <hiroh@chromium.org>
Tested-by: Hirokazu Honda <hiroh@chromium.org>
Reviewed-by: Hirokazu Honda <hiroh@chromium.org>

[rename] https://crrev.com/6a3c079fb1fab7be450de2b55b40efa74f44de8a/virtual/target-chromium-os-test/target-chromium-os-test-1-r76.ebuild
[modify] https://crrev.com/6a3c079fb1fab7be450de2b55b40efa74f44de8a/virtual/target-chromium-os-test/target-chromium-os-test-1.ebuild

Comment 8 by hiroh@chromium.org, Nov 19

Status: Verified (was: Started)
►

Sign in to add a comment