Match only make in gyp_chromium excludes.

Match only make in gyp_chromium excludes.

The current test also matches the CMake generator.

R=thakis@chromium.org

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

[bungeman-chromium, 2014-04-15 20:23:30]

I've been using the CMake gyp generator (mostly for editing with KDevelop), and
I've had this patch sitting around for a while. The current code gives an error
if 'make' is found anywhere in the generators list, this change makes it
actually look for a 'make' entry.

[Nico, 2014-04-15 20:25:47]

lgtm

(I wonder if we can just delete this altogether, everyone should've seen this by
now.)

https://codereview.chromium.org/235943009/diff/1/build/gyp_chromium
File build/gyp_chromium (right):

https://codereview.chromium.org/235943009/diff/1/build/gyp_chromium#newcode252
build/gyp_chromium:252: if re.search('(^|,|\\s)make($|,|\\s)',
os.environ.get('GYP_GENERATORS', '')):
Use r'..\s..' instead of '..\\s..'

[bungeman-chromium, 2014-04-15 20:34:06]

I have no real opinion on removing it, though it does seem nice to tell people
that what they're trying to do isn't going to work. I just wanted my use case to
work :-)

https://codereview.chromium.org/235943009/diff/1/build/gyp_chromium
File build/gyp_chromium (right):

https://codereview.chromium.org/235943009/diff/1/build/gyp_chromium#newcode252
build/gyp_chromium:252: if re.search('(^|,|\\s)make($|,|\\s)',
os.environ.get('GYP_GENERATORS', '')):
On 2014/04/15 20:25:47, Nico wrote:
> Use r'..\s..' instead of '..\\s..'

Done.

[bungeman-chromium, 2014-04-15 20:34:52]

Committed patchset #2 manually as r263984 (presubmit successful).