Fix build Linux ChromeOS GTK bot on the main waterfall.

Fix build Linux ChromeOS GTK bot on the main waterfall.

TBR=oshima@chromium.org
BUG=112962
TEST=Linux ChromeOS GTK bot not broken on the main waterfall

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

[Emmanuel Saint-loubert-Bié, 2012-02-08 21:39:04]

PTAL

[oshima, 2012-02-08 21:59:32]

https://chromiumcodereview.appspot.com/9358044/diff/1/build/all.gyp
File build/all.gyp (right):

https://chromiumcodereview.appspot.com/9358044/diff/1/build/all.gyp#newcode527
build/all.gyp:527: 'target_name': 'chromeos_builder',
who is using this target? (chromeos/aura seems to be using aura_builder +
individual targets).

[Emmanuel Saint-loubert-Bié, 2012-02-08 22:08:21]

Linux ChromeOS GTK  uses that target. This is the point of this CL

[oshima, 2012-02-08 22:11:16]

https://chromiumcodereview.appspot.com/9358044/diff/1/build/all.gyp
File build/all.gyp (right):

https://chromiumcodereview.appspot.com/9358044/diff/1/build/all.gyp#newcode572
build/all.gyp:572: 'target_name': 'chromeos_builder',
isn't this the target that chromeos gtk will use?

[Emmanuel Saint-loubert-Bié, 2012-02-08 22:39:01]

Oshima,
I think we need to land this since we have a broken bot on the waterfall.
If you believe that the first target is unused (I am not so sure). Then let's
file a bug and I promise to clean this up, recheck, all the bots, etc...
OK?
-- Emmanuel

https://chromiumcodereview.appspot.com/9358044/diff/1/build/all.gyp
File build/all.gyp (right):

https://chromiumcodereview.appspot.com/9358044/diff/1/build/all.gyp#newcode527
build/all.gyp:527: 'target_name': 'chromeos_builder',
I am not sure that all the bots have been switched aura_builde.r This is why the
sub-targets are the way they are.

https://chromiumcodereview.appspot.com/9358044/diff/1/build/all.gyp#newcode572
build/all.gyp:572: 'target_name': 'chromeos_builder',
Yes exactly.

[Emmanuel Saint-loubert-Bié, 2012-02-08 22:58:05]

Hi Oshima,

I cannot find any use of the chromeos_builder in any build script except for
GTK. So we are fine to remove the other section.

PTAL. Thanks! 

-- Emmanuel

[commit-bot: I haz the power, 2012-02-08 23:12:21]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/saintlou@chromium.org/9358044/6002

[oshima, 2012-02-08 23:55:40]

LGTM