-
Notifications
You must be signed in to change notification settings - Fork 78
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
ci(bazel): Fixes an issue where bazel could not create runfiles on wi…
…ndows. This issue is due to som legacy runfiles linking where a manifest file was expected where it was not on windows.
- Loading branch information
Lukas Holzer
committed
Sep 23, 2020
1 parent
f035263
commit cfcf850
Showing
16 changed files
with
234 additions
and
52 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,16 @@ | ||
diff --git a/node_modules/jest-haste-map/build/crawlers/node.js b/node_modules/jest-haste-map/build/crawlers/node.js | ||
index 6b6e11a..2ff08fa 100644 | ||
--- a/node_modules/jest-haste-map/build/crawlers/node.js | ||
+++ b/node_modules/jest-haste-map/build/crawlers/node.js | ||
@@ -205,7 +205,11 @@ function find(roots, extensions, ignore, callback) { | ||
|
||
function findNative(roots, extensions, ignore, callback) { | ||
const args = Array.from(roots); | ||
+ args.push('('); | ||
args.push('-type', 'f'); | ||
+ args.push('-o'); | ||
+ args.push('-type', 'l'); | ||
+ args.push(')'); | ||
|
||
if (extensions.length) { | ||
args.push('('); |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,16 @@ | ||
diff --git a/node_modules/stylelint/lib/standalone.js b/node_modules/stylelint/lib/standalone.js | ||
index 630af7e..dbf55e7 100644 | ||
--- a/node_modules/stylelint/lib/standalone.js | ||
+++ b/node_modules/stylelint/lib/standalone.js | ||
@@ -198,9 +198,9 @@ module.exports = function(options) { | ||
fileCache = new FileCache(cacheLocation); | ||
// Remove cache file if cache option is disabled | ||
fileCache.destroy(); | ||
- } | ||
+ } | ||
|
||
- return globby(fileList, globbyOptions) | ||
+ return Promise.resolve(fileList) | ||
.then((filePaths) => { | ||
// The ignorer filter needs to check paths relative to cwd | ||
filePaths = filterFilePaths( |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,23 @@ | ||
diff --git internal/node/launcher.sh internal/node/launcher.sh | ||
index e685da73..ba38076e 100644 | ||
--- internal/node/launcher.sh | ||
+++ internal/node/launcher.sh | ||
@@ -148,14 +148,16 @@ fi | ||
|
||
# Export the location of the runfiles helpers script | ||
export BAZEL_NODE_RUNFILES_HELPER=$(rlocation "TEMPLATED_runfiles_helper_script") | ||
-if [[ "${BAZEL_NODE_RUNFILES_HELPER}" != /* ]] && [[ ! "${BAZEL_NODE_RUNFILES_HELPER}" =~ ^[A-Z]:[\\/] ]]; then | ||
+# Paths can be with lower and upper case on windows because of the msys64 package in the powershell | ||
+if [[ "${BAZEL_NODE_RUNFILES_HELPER}" != /* ]] && [[ ! "${BAZEL_NODE_RUNFILES_HELPER}" =~ ^[A-Za-z]:[\/\\] ]]; then | ||
export BAZEL_NODE_RUNFILES_HELPER=$(pwd)/${BAZEL_NODE_RUNFILES_HELPER} | ||
fi | ||
|
||
# Export the location of the require patch script as it can be used to bootstrap | ||
# node require patch if needed | ||
export BAZEL_NODE_PATCH_REQUIRE=$(rlocation "TEMPLATED_require_patch_script") | ||
-if [[ "${BAZEL_NODE_PATCH_REQUIRE}" != /* ]] && [[ ! "${BAZEL_NODE_PATCH_REQUIRE}" =~ ^[A-Z]:[\\/] ]]; then | ||
+# Paths can be with lower and upper case on windows because of the msys64 package in the powershell | ||
+if [[ "${BAZEL_NODE_PATCH_REQUIRE}" != /* ]] && [[ ! "${BAZEL_NODE_PATCH_REQUIRE}" =~ ^[A-Za-z]:[\/\\] ]]; then | ||
export BAZEL_NODE_PATCH_REQUIRE=$(pwd)/${BAZEL_NODE_PATCH_REQUIRE} | ||
fi | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,5 @@ | ||
# What is the purpose of those scripts | ||
|
||
This folder contains scripts to execute commands platform independently. As | ||
node.js is working on Windows, Linux and macOs it is better than having shell | ||
scripts for each language. This should simplify invoking bazel commands. |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,19 @@ | ||
#!/usr/bin/env node | ||
const shelljs = require('shelljs'); | ||
const { join } = require('path'); | ||
const { bazelQuery, bazelTest } = require('./utils/execute-bazel'); | ||
|
||
// The workspace root | ||
const PROJECT_DIR = join(__dirname, '../../'); | ||
|
||
// ShellJS should exit if any command fails. | ||
shelljs.set('-e'); | ||
shelljs.cd(PROJECT_DIR); | ||
|
||
const targets = bazelQuery('attr(generator_function, stylelint, //...)'); | ||
|
||
console.log(` | ||
[ Running Stylelint on ${targets.length} Targets] => | ||
`); | ||
|
||
bazelTest(targets, [], { silent: false }); |
Oops, something went wrong.