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

Unified Diff: scripts/slave/recipe_modules/isolate/api.py

Issue 620703002: Add isolate.clean_isolated_files and use in all related recipes. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/build
Patch Set: Created 6 years, 3 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
Index: scripts/slave/recipe_modules/isolate/api.py
diff --git a/scripts/slave/recipe_modules/isolate/api.py b/scripts/slave/recipe_modules/isolate/api.py
index 6c24b357e27b0385ee3b60f008d2bfcd113aa973..cfb488fce0c59d30fad576a584c9de5f0e3652cd 100644
--- a/scripts/slave/recipe_modules/isolate/api.py
+++ b/scripts/slave/recipe_modules/isolate/api.py
@@ -38,6 +38,19 @@ class IsolateApi(recipe_api.RecipeApi):
config.gyp_env.GYP_DEFINES['test_isolation_mode'] = 'archive'
config.gyp_env.GYP_DEFINES['test_isolation_outdir'] = self._isolate_server
+ def clean_isolated_files(self, build_dir):
+ """Cleans out all *.isolated files from the build directory in
+ preparation for the compile. Needed in order to ensure isolates
+ are rebuilt properly because their dependencies are currently not
+ completely described to gyp."""
+ self.m.python(
+ 'clean isolated files',
+ self.resource('find_isolated_tests.py'),
+ [
+ '--build-dir', build_dir,
+ '--clean-isolated-files'
+ ])
+
def find_isolated_tests(self, build_dir, targets=None, **kwargs):
"""Returns a step which finds all *.isolated files in a build directory.
« no previous file with comments | « scripts/slave/recipe_modules/gpu/api.py ('k') | scripts/slave/recipe_modules/isolate/resources/find_isolated_tests.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698