The LASTCHANGE file is created and updated by gclient runhooks under

normal circumstances, so it makes more sense to treat it as if it were a
checked-in source file, not a generated file. The rule for its
generation was still getting run once on a clean build, because the
other targets that depended on it expected a gyp timestamp file to exist
for the target.

There's no need for any other targets to actually depend on lastchange
(and in fact several of the targets that make use of the file already
don't depend on the target); it should already exist, and if it doesn't
then the source checkout is probably broken (as the other steps in
runhooks are also mandatory). So, remove the rule entirely.

Review URL: https://chromiumcodereview.appspot.com/10388213

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@139765 0039d316-1c4b-4281-b951-d872f2087c98

3 files changed, 0 insertions, 38 deletions

diff --git a/build/all.gyp b/build/all.gyp
index 753edd3..0f8ef6a 100644
--- a/build/all.gyp
+++ b/build/all.gyp
@@ -56,7 +56,6 @@
         '../v8/tools/gyp/v8.gyp:*',
         '../webkit/support/webkit_support.gyp:*',
         '../webkit/webkit.gyp:*',
-        'util/build_util.gyp:*',
         'temp_gyp/googleurl.gyp:*',
         '<(libjpeg_gyp_path):*',
       ],
diff --git a/build/all_android.gyp b/build/all_android.gyp
index 12498e1..dfd4479 100644
--- a/build/all_android.gyp
+++ b/build/all_android.gyp
@@ -13,7 +13,6 @@
       'type': 'none',
       'dependencies': [
         '../content/content.gyp:content_shell_apk',
-        'util/build_util.gyp:*',
         'android_builder_tests',
       ],
     }, # target_name: All
diff --git a/build/util/build_util.gyp b/build/util/build_util.gyp
deleted file mode 100644
index a7a5a4b..0000000
--- a/build/util/build_util.gyp
+++ /dev/null
@@ -1,36 +0,0 @@
-# Copyright (c) 2011 The Chromium Authors. All rights reserved.
-# Use of this source code is governed by a BSD-style license that can be
-# found in the LICENSE file.
-
-{
-  'targets': [
-    {
-      'target_name': 'lastchange',
-      'type': 'none',
-      'variables': {
-        'lastchange_out_path': '<(DEPTH)/build/util/LASTCHANGE',
-        'default_lastchange_path': '../LASTCHANGE.in',
-      },
-      'actions': [
-        {
-          'action_name': 'lastchange',
-          'inputs': [
-            # Note:  <(default_lastchange_path) is optional,
-            # so it doesn't show up in inputs.
-            './lastchange.py',
-          ],
-          'outputs': [
-            '<(lastchange_out_path)',
-          ],
-          'action': [
-            'python', '<@(_inputs)',
-            '-o', '<(lastchange_out_path)',
-            '-d', '<(default_lastchange_path)',
-          ],
-          'message': 'Extracting last change to <(lastchange_out_path)',
-          'process_outputs_as_sources': '1',
-        },
-      ],
-    },
-  ]
-}