From e95ab5c36ed1746277515bc3514ae8f23dbdac0d Mon Sep 17 00:00:00 2001 From: Chris Rebert Date: Tue, 1 Dec 2015 20:25:09 -0800 Subject: [PATCH] s/grunt-sed/npm script/ ; fixes #18338 [skip sauce] [skip validator] --- Gruntfile.js | 16 ------ grunt/change-version.js | 105 ++++++++++++++++++++++++++++++++++++++++ package.json | 5 +- 3 files changed, 109 insertions(+), 17 deletions(-) create mode 100755 grunt/change-version.js diff --git a/Gruntfile.js b/Gruntfile.js index 27d5fd63db..947716395a 100644 --- a/Gruntfile.js +++ b/Gruntfile.js @@ -388,17 +388,6 @@ module.exports = function (grunt) { } }, - sed: { - versionNumber: { - pattern: (function () { - var old = grunt.option('oldver'); - return old ? RegExp.quote(old) : old; - })(), - replacement: grunt.option('newver'), - recursive: true - } - }, - 'saucelabs-qunit': { all: { options: { @@ -520,11 +509,6 @@ module.exports = function (grunt) { // Default task. grunt.registerTask('default', ['clean:dist', 'test']); - // Version numbering task. - // grunt change-version-number --oldver=A.B.C --newver=X.Y.Z - // This can be overzealous, so its changes should always be manually reviewed! - grunt.registerTask('change-version-number', 'sed'); - grunt.registerTask('commonjs', ['babel:umd', 'npm-js']); grunt.registerTask('npm-js', 'Generate npm-js entrypoint module in dist dir.', function () { diff --git a/grunt/change-version.js b/grunt/change-version.js new file mode 100755 index 0000000000..02807e176e --- /dev/null +++ b/grunt/change-version.js @@ -0,0 +1,105 @@ +#!/usr/bin/env node +'use strict'; + +/*! + * Script to update version number references in the project. + * Copyright 2015 Twitter, Inc. + * Licensed under MIT (https://github.com/twbs/bootstrap/blob/master/LICENSE) + */ +var fs = require('fs'); +var path = require('path'); +var sh = require('shelljs'); +sh.config.fatal = true; +var sed = sh.sed; + +// Blame TC39... https://github.com/benjamingr/RegExp.escape/issues/37 +RegExp.quote = function (string) { + return string.replace(/[-\\^$*+?.()|[\]{}]/g, '\\$&'); +}; +RegExp.quoteReplacement = function (string) { + return string.replace(/[$]/g, '$$'); +}; + +var DRY_RUN = false; + +function walkAsync(directory, excludedDirectories, fileCallback, errback) { + if (excludedDirectories.has(path.parse(directory).base)) { + return; + } + fs.readdir(directory, function (err, names) { + if (err) { + errback(err); + return; + } + names.forEach(function (name) { + var filepath = path.join(directory, name); + fs.lstat(filepath, function (err, stats) { + if (err) { + process.nextTick(errback, err); + return; + } + if (stats.isSymbolicLink()) { + return; + } + else if (stats.isDirectory()) { + process.nextTick(walkAsync, filepath, excludedDirectories, fileCallback, errback); + } + else if (stats.isFile()) { + process.nextTick(fileCallback, filepath); + } + }); + }); + }); +} + +function replaceRecursively(directory, excludedDirectories, allowedExtensions, original, replacement) { + original = new RegExp(RegExp.quote(original), 'g'); + replacement = RegExp.quoteReplacement(replacement); + var updateFile = !DRY_RUN ? (function (filepath) { + if (allowedExtensions.has(path.parse(filepath).ext)) { + sed('-i', original, replacement, filepath); + } + }) : (function (filepath) { + if (allowedExtensions.has(path.parse(filepath).ext)) { + console.log('FILE: ' + filepath); + } + else { + console.log('EXCLUDED:' + filepath); + } + }); + walkAsync('.', excludedDirectories, updateFile, function (err) { + console.error('ERROR while traversing directory!:') + console.error(err); + process.exit(1); + }); +} + +function main(args) { + if (args.length !== 2) { + console.error('USAGE: change-version old_version new_version'); + console.error('Got arguments:', args); + process.exit(1); + } + var oldVersion = args[0]; + var newVersion = args[1]; + var EXCLUDED_DIRS = new Set([ + '.git', + 'node_modules', + 'vendor' + ]); + var INCLUDED_EXTENSIONS = new Set([ + // This extension whitelist is how we avoid modifying binary files + '', + '.css', + '.html', + '.js', + '.json', + '.md', + '.scss', + '.txt', + '.yml' + ]); + replaceRecursively('.', EXCLUDED_DIRS, INCLUDED_EXTENSIONS, oldVersion, newVersion); +}; + +main(process.argv.slice(2)); diff --git a/package.json b/package.json index c7aafd7433..617d84564c 100644 --- a/package.json +++ b/package.json @@ -56,13 +56,13 @@ "grunt-sass": "^1.0.0", "grunt-saucelabs": "~8.6.1", "grunt-scss-lint": "^0.3.8", - "grunt-sed": "~0.1.1", "grunt-stamp": "^0.1.0", "is-travis": "^1.0.0", "load-grunt-tasks": "~3.3.0", "markdown-it": "^5.0.0", "mq4-hover-shim": "^0.3.0", "npm-shrinkwrap": "^200.1.0", + "shelljs": "^0.5.3", "time-grunt": "^1.2.1" }, "engines": { @@ -96,5 +96,8 @@ "dependencies": { "jquery": "*" } + }, + "scripts": { + "change-version": "node grunt/change-version.js" } }