Guard gcc_version in a safe condition.

Guard gcc_version in a safe condition.

gcc_version is only defined on Linux and thus we need to check for it before
using it on other platforms.

This should fix the following error:

gyp: name 'gcc_version' is not defined while evaluating condition 'gcc_version >= 48'

R=thakis@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=196565

[Nico, 2013-04-26 00:35:56]

https://codereview.chromium.org/14172021/diff/1/hunspell.gyp
File hunspell.gyp (right):

https://codereview.chromium.org/14172021/diff/1/hunspell.gyp#newcode76
hunspell.gyp:76: ['os_posix==1 and OS!="mac" and OS!="ios"',
This now needs another 'conditions' child, right? Or you could just move these
conditions in front of the gcc_version check, I think and short-circuits in
python

[tfarina, 2013-04-26 00:48:07]

https://codereview.chromium.org/14172021/diff/1/hunspell.gyp
File hunspell.gyp (right):

https://codereview.chromium.org/14172021/diff/1/hunspell.gyp#newcode76
hunspell.gyp:76: ['os_posix==1 and OS!="mac" and OS!="ios"',
On 2013/04/26 00:35:56, Nico wrote:
> This now needs another 'conditions' child, right? Or you could just move these
> conditions in front of the gcc_version check, I think and short-circuits in
> python

Done.

[Nico, 2013-04-26 00:58:07]

lgtm

https://codereview.chromium.org/14172021/diff/6001/hunspell.gyp
File hunspell.gyp (right):

https://codereview.chromium.org/14172021/diff/6001/hunspell.gyp#newcode77
hunspell.gyp:77: 'cflags': [
The old indent was right, I think? At least it matches line 70

[tfarina, 2013-04-26 01:02:27]

landing.

Thanks for the review!

https://codereview.chromium.org/14172021/diff/6001/hunspell.gyp
File hunspell.gyp (right):

https://codereview.chromium.org/14172021/diff/6001/hunspell.gyp#newcode77
hunspell.gyp:77: 'cflags': [
On 2013/04/26 00:58:07, Nico wrote:
> The old indent was right, I think? At least it matches line 70
ops, done, also changed the declaration inside of cflags to 2 spaces instead of
3.

[tfarina, 2013-04-26 01:03:20]

Committed patchset #3 manually as r196565 (presubmit successful).