From f9947c3792bf290b8511b4df63e63ff4cac395af Mon Sep 17 00:00:00 2001 From: Andrey Lushnikov Date: Tue, 2 Nov 2021 01:00:54 -0700 Subject: [PATCH] devops: fix Chromium archiving on Windows (#9959) New upstream bundling config still includes `interactive_ui_tests.exe` for windows archive. We do not build it and do not use it - so we manually exclude it from packaging. --- .../chromium/compute_files_to_archive.js | 17 +++++++++++++---- 1 file changed, 13 insertions(+), 4 deletions(-) diff --git a/browser_patches/chromium/compute_files_to_archive.js b/browser_patches/chromium/compute_files_to_archive.js index d660794655..4bdfdc045e 100644 --- a/browser_patches/chromium/compute_files_to_archive.js +++ b/browser_patches/chromium/compute_files_to_archive.js @@ -7,7 +7,16 @@ const fs = require('fs'); const configs = JSON.parse(fs.readFileSync(process.argv[2], 'utf8')).archive_datas; const config = configs.find(config => config.gcs_path.includes('chrome-linux.zip') || config.gcs_path.includes('chrome-win.zip') || config.gcs_path.includes('chrome-mac.zip')); -for (const file of config.files || []) - console.log(file); -for (const dir of config.dirs || []) - console.log(dir); + +const excludeList = new Set([ + // We do not need interactive tests in our archive. + 'interactive_ui_tests.exe', +]); + +const entries = [ + ...(config.files || []), + ...(config.dirs || []), +].filter(entry => !excludeList.has(entry)); + +for (const entry of entries) + console.log(entry);