libwebp: silence libtool warning on non-neon builds

libwebp: silence libtool warning on non-neon builds


BUG=141949
TEST=build


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

[jzern, 2012-08-10 22:09:28]

Mac build log doesn't seem to have the warning anymore, any other bots to run?

[Nico, 2012-08-10 23:14:33]

LGTM

One optional nit below. Thank you for taking care of this so quickly!

http://codereview.chromium.org/10855111/diff/1/third_party/libwebp/libwebp.gyp
File third_party/libwebp/libwebp.gyp (right):

http://codereview.chromium.org/10855111/diff/1/third_party/libwebp/libwebp.gy...
third_party/libwebp/libwebp.gyp:74: 'type': 'none',
nit: I believe some generators might choke on type none targets with a 'sources'
section, so I'd move 'sources' into the 'armv7' branch too. (Even if this should
work as is, I feel it's cleaner with this change.)

[jzern, 2012-08-11 02:07:54]

http://codereview.chromium.org/10855111/diff/1/third_party/libwebp/libwebp.gyp
File third_party/libwebp/libwebp.gyp (right):

http://codereview.chromium.org/10855111/diff/1/third_party/libwebp/libwebp.gy...
third_party/libwebp/libwebp.gyp:74: 'type': 'none',
On 2012/08/10 23:14:33, Nico wrote:
> nit: I believe some generators might choke on type none targets with a
'sources'
> section, so I'd move 'sources' into the 'armv7' branch too. (Even if this
should
> work as is, I feel it's cleaner with this change.)
>
Sounds reasonable. I hid the rest under armv7.

[commit-bot: I haz the power, 2012-08-11 03:23:18]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jzern@chromium.org/10855111/2002

[commit-bot: I haz the power, 2012-08-11 05:06:52]

Try job failure for 10855111-2002 (retry) on linux_rel for step "browser_tests".
It's a second try, previously, step "browser_tests" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_rel&...

[commit-bot: I haz the power, 2012-08-11 17:36:44]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jzern@chromium.org/10855111/2002

[commit-bot: I haz the power, 2012-08-12 02:29:20]

Change committed as 151196