Bug 1357702 - Make eslint-plugin-spidermonkey-js handle paths correct on Windows to avoid Unexpected character issues on ESLint. r?Mossop
MozReview-Commit-ID: LsQCgBOgceY
--- a/tools/lint/eslint/eslint-plugin-spidermonkey-js/lib/processors/self-hosted.js
+++ b/tools/lint/eslint/eslint-plugin-spidermonkey-js/lib/processors/self-hosted.js
@@ -3,21 +3,26 @@
*
* This Source Code Form is subject to the terms of the Mozilla Public
* License, v. 2.0. If a copy of the MPL was not distributed with this
* file, You can obtain one at http://mozilla.org/MPL/2.0/.
*/
"use strict";
+var path = require("path");
+
const selfHostedRegex = /js\/src\/(?:builtin|shell)\/.*?\.js$/;
const macroRegex = /\s*\#(if|ifdef|else|elif|endif|include|define|undef).*/;
module.exports = {
preprocess(text, filename) {
+ if (path.win32) {
+ filename = filename.split(path.sep).join("/");
+ }
if (!selfHostedRegex.test(filename)) {
return [text];
}
let lines = text.split(/\n/);
for (let i = 0; i < lines.length; i++) {
if (!macroRegex.test(lines[i])) {
// No macro here, nothing to do.
--- a/tools/lint/eslint/eslint-plugin-spidermonkey-js/package.json
+++ b/tools/lint/eslint/eslint-plugin-spidermonkey-js/package.json
@@ -1,11 +1,11 @@
{
"name": "eslint-plugin-spidermonkey-js",
- "version": "0.1.0",
+ "version": "0.1.1",
"description": "A collection of rules that help enforce JavaScript coding standard in the Mozilla SpiderMonkey project.",
"keywords": [
"eslint",
"eslintplugin",
"eslint-plugin",
"mozilla",
"spidermonkey"
],