Building FFmpeg in tree for arm

Building FFmpeg in tree for arm

Changes yasm to be built for host instead of target.
Also don't need to depend on yasm if build ing for arm.

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=31929

[Alpha Left Google, 2009-11-11 18:33:44]

Will need some simple fix in FFmpeg's code to build after this change.

[Alpha Left Google, 2009-11-11 18:34:04]

Reviewers: awong,

Message:
Will need some simple fix in FFmpeg's code to build after this change.


Description:
Building FFmpeg in tree for arm

Changes yasm to be built for host instead of target.
Also don't need to depend on yasm if build ing for arm.

Please review this at http://codereview.chromium.org/387023

Affected files:
   M build/all.gyp
   M build/common.gypi
   M third_party/ffmpeg/ffmpeg.gyp
   A third_party/ffmpeg/source/config/Chromium/linux/arm/config.h
   A third_party/ffmpeg/source/config/Chromium/linux/arm/version.h
   M third_party/yasm/yasm.gyp
   M webkit/tools/test_shell/test_shell.gyp

[awong, 2009-11-12 23:11:00]

couple of small changes needed.

http://codereview.chromium.org/387023/diff/2001/2004
File third_party/ffmpeg/ffmpeg.gyp (right):

http://codereview.chromium.org/387023/diff/2001/2004#newcode62
Line 62: 'dependencies': [
Move this dependency into the x64 & ia32 dependencies.

http://codereview.chromium.org/387023/diff/2001/2004#newcode187
Line 187: 'sources': [
Put ths in a common section for target_arch==ia32 or target_arch==x64

[Alpha Left Google, 2009-11-12 23:49:02]

On 2009/11/12 23:11:00, awong wrote:
> couple of small changes needed.
> 
> http://codereview.chromium.org/387023/diff/2001/2004
> File third_party/ffmpeg/ffmpeg.gyp (right):
> 
> http://codereview.chromium.org/387023/diff/2001/2004#newcode62
> Line 62: 'dependencies': [
> Move this dependency into the x64 & ia32 dependencies.
> 
> http://codereview.chromium.org/387023/diff/2001/2004#newcode187
> Line 187: 'sources': [
> Put ths in a common section for target_arch==ia32 or target_arch==x64

all done.

[Alpha Left Google, 2009-11-13 19:04:43]

ping.

[awong, 2009-11-13 19:08:06]

LGTM