Commit cacd0b18 authored by Frank Barchard's avatar Frank Barchard Committed by Commit Bot

libyuv roll from 1678 to 1682 for webrtc libyuv_config

Using all_dependent_configs to pass libyuv_config around.

Using public_configs, client projects must rely on public_deps to
propagate configurations up in the build graph. This is bad because
public_deps allows the exposition of headers that live in another
target. This can lead to a really unhealthy build.

On the other side, all_dependent_configs is automatically propagated
up in the build graph but if a target includes a libyuv header it is
forced by GN to declare the dependency (and this will propagate
libyuv_config).

Bug: webrtc:8605, webrtc:8603
Test: None
Change-Id: I590c620b29102f2f7720da36872335f83ea7c453
Reviewed-on: https://chromium-review.googlesource.com/822334
Commit-Queue: Mirko Bonadei <mbonadei@chromium.org>
Reviewed-by: default avatarFrank Barchard <fbarchard@chromium.org>
Cr-Commit-Position: refs/heads/master@{#523702}
parent 3ae2446b
......@@ -480,7 +480,7 @@ deps = {
Var('chromium_git') + '/webm/libwebm.git' + '@' + '4956b2dec65352af32dc71bab553acb631c64177',
'src/third_party/libyuv':
Var('chromium_git') + '/libyuv/libyuv.git' + '@' + '12c904a97c81c3ef4cab0fc8fb1f0485b4ec4e8c', # from r1678
Var('chromium_git') + '/libyuv/libyuv.git' + '@' + 'd94a4867bf8be91e4c0fa8b7a4133ba060fe524f', # from r1682
'src/third_party/lighttpd': {
'url': Var('chromium_git') + '/chromium/deps/lighttpd.git' + '@' + Var('lighttpd_revision'),
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment