Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3456)

Unified Diff: build/all.gyp

Issue 159422: Now that testshell depends on imagediff we can start to cleanup the targets a... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 11 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/all.gyp
===================================================================
--- build/all.gyp (revision 21662)
+++ build/all.gyp (working copy)
@@ -90,19 +90,17 @@
['OS=="mac"', {
'targets': [
{
- # Target to build everything needed for layout tests to cut down
- # on what the layout test bots have to build.
+ # This target is legacy and can go away when the bots have been
+ # updated to direclty build the test_shell targets.
'target_name': 'build_for_layout_tests',
'type': 'none',
'dependencies': [
- '../tools/imagediff/image_diff.gyp:image_diff',
- '../webkit/tools/test_shell/test_shell.gyp:test_shell',
- '../webkit/tools/test_shell/test_shell.gyp:test_shell_tests',
+ '../webkit/tools/test_shell/test_shell.gyp:*',
],
},
{
# Target to build everything plus the dmg. We don't put the dmg
- # in the All target because developer really don't need it.
+ # in the All target because developers really don't need it.
'target_name': 'all_and_dmg',
'type': 'none',
'dependencies': [
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698