Suppress libvpx warning

Suppress libvpx warning


BUG=
TEST=

[tzik, 2011-11-08 15:16:37]

Hi.

Recent libvpx.gyp seems to cause build-time warning like below, on building
chrome on linux.
Can I fix it like this patch?

--
third_party/libvpx/simple_encoder.target.mk:27:warning: overriding commands for
target `out/Release/obj/gen/third_party/libvpx/gen_example_code.sh'
third_party/libvpx/simple_decoder.target.mk:27:warning: ignoring old commands
for target `out/Release/obj/gen/third_party/libvpx/gen_example_code.sh'

[Alpha Left Google, 2011-11-08 15:19:53]

LGTM with small changes.

http://codereview.chromium.org/8501011/diff/1/libvpx.gyp
File libvpx.gyp (right):

http://codereview.chromium.org/8501011/diff/1/libvpx.gyp#newcode170
libvpx.gyp:170: 'destination': '<(shared_generated_dir)/encoder',
Please rename this to simple_encoder.

http://codereview.chromium.org/8501011/diff/1/libvpx.gyp#newcode181
libvpx.gyp:181: 'inputs': [
'<(shared_generated_dir)/encoder/gen_example_code.sh', ],
same there.

http://codereview.chromium.org/8501011/diff/1/libvpx.gyp#newcode188
libvpx.gyp:188: '<(shared_generated_dir)/encoder/gen_example_code.sh
<(RULE_INPUT_PATH) > <(shared_generated_dir)/<(RULE_INPUT_ROOT).c',
same as above.

http://codereview.chromium.org/8501011/diff/1/libvpx.gyp#newcode216
libvpx.gyp:216: 'source/libvpx/examples/decoder/gen_example_code.sh',
Please rename this to simple_decoder.

http://codereview.chromium.org/8501011/diff/1/libvpx.gyp#newcode225
libvpx.gyp:225: 'inputs': [
'<(shared_generated_dir)/decoder/gen_example_code.sh', ],
Also this please.

http://codereview.chromium.org/8501011/diff/1/libvpx.gyp#newcode232
libvpx.gyp:232: '<(shared_generated_dir)/decoder/gen_example_code.sh
<(RULE_INPUT_PATH) > <(shared_generated_dir)/<(RULE_INPUT_ROOT).c',
And this please.

[tzik, 2011-11-08 15:34:37]

Thanks for very fast review.

BTW, I don't have permission for this repository.
Could you commit this patch instead of me?

http://codereview.chromium.org/8501011/diff/1/libvpx.gyp
File libvpx.gyp (right):

http://codereview.chromium.org/8501011/diff/1/libvpx.gyp#newcode170
libvpx.gyp:170: 'destination': '<(shared_generated_dir)/encoder',
On 2011/11/08 15:19:53, Alpha wrote:
> Please rename this to simple_encoder.

Done.

http://codereview.chromium.org/8501011/diff/1/libvpx.gyp#newcode181
libvpx.gyp:181: 'inputs': [
'<(shared_generated_dir)/encoder/gen_example_code.sh', ],
On 2011/11/08 15:19:53, Alpha wrote:
> same there.

Done.

http://codereview.chromium.org/8501011/diff/1/libvpx.gyp#newcode188
libvpx.gyp:188: '<(shared_generated_dir)/encoder/gen_example_code.sh
<(RULE_INPUT_PATH) > <(shared_generated_dir)/<(RULE_INPUT_ROOT).c',
On 2011/11/08 15:19:53, Alpha wrote:
> same as above.

Done.

http://codereview.chromium.org/8501011/diff/1/libvpx.gyp#newcode216
libvpx.gyp:216: 'source/libvpx/examples/decoder/gen_example_code.sh',
On 2011/11/08 15:19:53, Alpha wrote:
> Please rename this to simple_decoder.

Done. I misplaced directory name.

http://codereview.chromium.org/8501011/diff/1/libvpx.gyp#newcode225
libvpx.gyp:225: 'inputs': [
'<(shared_generated_dir)/decoder/gen_example_code.sh', ],
On 2011/11/08 15:19:53, Alpha wrote:
> Also this please.

Done.

http://codereview.chromium.org/8501011/diff/1/libvpx.gyp#newcode232
libvpx.gyp:232: '<(shared_generated_dir)/decoder/gen_example_code.sh
<(RULE_INPUT_PATH) > <(shared_generated_dir)/<(RULE_INPUT_ROOT).c',
On 2011/11/08 15:19:53, Alpha wrote:
> And this please.

Done.

[Alpha Left Google, 2011-11-08 16:19:11]

I've set the commit bit, let's see if this works.

You'll also need to submit a change to pull in this change through DEPS.

[tzik, 2011-11-09 13:40:54]

Thanks, but maybe no commit bot is watching us.
Could you commit it manually?

On 2011/11/08 16:19:11, Alpha wrote:
> I've set the commit bit, let's see if this works.
> 
> You'll also need to submit a change to pull in this change through DEPS.

[Alpha Left Google, 2011-11-09 13:45:17]

On 2011/11/09 13:40:54, tzik wrote:
> Thanks, but maybe no commit bot is watching us.
> Could you commit it manually?
> 
> On 2011/11/08 16:19:11, Alpha wrote:
> > I've set the commit bit, let's see if this works.
> > 
> > You'll also need to submit a change to pull in this change through DEPS.

change submitted as 109236.