less 2.6.1 → 2.7.3

package/.travis.yml

@@ -1,7 +1,7 @@
 language: node_js
 node_js:
-  - "4.2.0"
-  - "4.0.0"
+  - "6"
+  - "4"
   - "0.12"
   - "0.10"
 install:
@@ -9,6 +9,7 @@ install:
   - npm install
 env:
   global:
+    - PHANTOMJS_CDNURL=http://cnpmjs.org/downloads
     - secure: TrNVruWYaUK5ALga1y7wRY+MLjWJECUSCsBmKW5EUmIevOUxqHWu7M89FANKxstEeFRRAGH3QJbloRxnzIgh0U0ah5npE9XA1bYXGO5khoXeIyk7pNRfjIo8aEnJH1Vp8vWA6J6ovxdJ7lCFKEGvGKxGde50knVl7KFVVULlX2U=
     - secure: Rzh+CEI7YRvvVkOruPE8Z0dkU0s13V6b6cpqbN72vxbJl/Jm5PUZkjTFJdkWJrW3ErhCKX6EC7XdGvrclqEA9WAqKzrecqCJYqTnw4MwqiAj6F9wqE/BqhoWg4xPxm0MK/7eJMvLCgjNpe+gc1CaeFJZkLSNWn6nOFke+vVlf9Q=
 sudo: false

package/CHANGELOG.md

@@ -1,3 +1,28 @@
+# 2.7.2
+
+2017-01-04
+
+  - Revert changes to contrast() function in 2.7.0
+
+# 2.7.1 HOTFIX
+
+2016-05-09
+
+ - Revert commit 470af20 (source map performance) which broke source maps
+
+# 2.7.0
+
+2016-05-07
+
+ - Fixes for contrast() function
+ - Allow root functions for plugins
+ - Allow semi-colon separators in functions
+ - Performance improvements for source maps
+ - Fix "blank page" issue in browser
+ - Compatibility fixes for Node 6.x
+ - Fix AST to include single-line comments
+ - Fix reversion for "color-like" words (outputting color values)
+
 # 2.6.1
 
 2016-03-04
@@ -141,7 +166,7 @@
  - Improved keyword and anonymous usage with the replace function
  - Added `getCSSAppendage` to sourcemap builder to avoid duplication in plugins
  - Fix problem with plugins when used with the promises version of render
- - If the render callback throws an exception it now propogates instead of calling the callback again with an error
+ - If the render callback throws an exception it now propagates instead of calling the callback again with an error
 
 # 2.1.0
 

package/Gruntfile.js

@@ -14,18 +14,17 @@ module.exports = function (grunt) {
         build: grunt.file.readYAML('build/build.yml'),
         pkg: grunt.file.readJSON('package.json'),
         meta: {
-            license: '<%= _.pluck(pkg.licenses, "type").join(", ") %>',
             copyright: 'Copyright (c) 2009-<%= grunt.template.today("yyyy") %>',
             banner: '/*!\n' +
                 ' * Less - <%= pkg.description %> v<%= pkg.version %>\n' +
                 ' * http://lesscss.org\n' +
                 ' *\n' +
                 ' * <%= meta.copyright %>, <%= pkg.author.name %> <<%= pkg.author.email %>>\n' +
-                ' * Licensed under the <%= meta.license %> License.\n' +
+                ' * Licensed under the <%= pkg.license %> License.\n' +
                 ' *\n' +
                 ' */\n\n' +
                 ' /**' +
-                ' * @license <%= meta.license %>\n' +
+                ' * @license <%= pkg.license %>\n' +
                 ' */\n\n'
         },
 

package/README.md

@@ -51,6 +51,6 @@ Copyright (c) 2009-2016 [Alexis Sellier](http://cloudhead.io) & The Core Less Te
 Licensed under the [Apache License](LICENSE).
 
 
-[so]: http://stackoverflow.com/questions/tagged/twitter-bootstrap+less "StackOverflow.com"
+[so]: http://stackoverflow.com/questions/tagged/less "StackOverflow.com"
 [issues]: https://github.com/less/less.js/issues "GitHub Issues for Less.js"
 [download]: https://github.com/less/less.js/zipball/master "Download Less.js"

package/appveyor.yml

@@ -4,6 +4,7 @@ environment:
     - nodejs_version: "0.10"
     - nodejs_version: "0.12"
     - nodejs_version: "4"
+    - nodejs_version: "6"
 
 # Install scripts. (runs after repo cloning)
 install:

package/bin/lessc

@@ -40,18 +40,32 @@ var silent = false,
     plugins: plugins
 };
 var sourceMapOptions = {};
-var continueProcessing = true,
-    currentErrorcode;
-
-// calling process.exit does not flush stdout always
-// so use this to set the exit code
-process.on('exit', function() { process.reallyExit(currentErrorcode); });
+var continueProcessing = true;
+
+// Calling process.exit does not flush stdout always. Instead of exiting the process, we set the process' exitCode,
+// close all handles and wait for the event loop to exit the process.
+// @see https://github.com/nodejs/node/issues/6409
+// Unfortunately, node 0.10.x does not support setting process.exitCode, so we need to call reallyExit() explicitly.
+// @see https://nodejs.org/api/process.html#process_process_exitcode
+// Additionally we also need to make sure that uncaughtExceptions are never swallowed.
+// @see https://github.com/less/less.js/issues/2881
+// This code can safely be removed if node 0.10.x is not supported anymore.
+process.on("exit", function() { process.reallyExit(process.exitCode); });
+process.on("uncaughtException", function(err) {
+    console.error(err);
+    process.exitCode = 1;
+});
+// This code will still be required because otherwise rejected promises would not be reported to the user
+process.on("unhandledRejection", function(err) {
+    console.error(err);
+    process.exitCode = 1;
+});
 
 var checkArgFunc = function(arg, option) {
     if (!option) {
         console.error(arg + " option requires a parameter");
         continueProcessing = false;
-        currentErrorcode = 1;
+        process.exitCode = 1;
         return false;
     }
     return true;
@@ -62,7 +76,7 @@ var checkBooleanArg = function(arg) {
     if (!onOff) {
         console.error(" unable to parse " + arg + " as a boolean. use one of on/t/true/y/yes/off/f/false/n/no");
         continueProcessing = false;
-        currentErrorcode = 1;
+        process.exitCode = 1;
         return false;
     }
     return Boolean(onOff[2]);
@@ -252,7 +266,7 @@ function printUsage() {
                 } else {
                     console.error("Unable to load plugin " + name +
                         " please make sure that it is installed under or at the same level as less");
-                    currentErrorcode = 1;
+                    process.exitCode = 1;
                 }
                 break;
             default:
@@ -263,7 +277,7 @@ function printUsage() {
                     console.error("Unable to interpret argument " + arg +
                         " - if it is a plugin (less-plugin-" + arg + "), make sure that it is installed under or at" +
                         " the same level as less");
-                    currentErrorcode = 1;
+                    process.exitCode = 1;
                 }
                 break;
         }
@@ -288,15 +302,20 @@ function printUsage() {
         sourceMapOptions.sourceMapInputFilename = input;
         if (!sourceMapOptions.sourceMapFullFilename) {
             if (!output && !sourceMapFileInline) {
-                console.log("the sourcemap option only has an optional filename if the css filename is given");
-                console.log("consider adding --source-map-map-inline which embeds the sourcemap into the css");
+                console.error("the sourcemap option only has an optional filename if the css filename is given");
+                console.error("consider adding --source-map-map-inline which embeds the sourcemap into the css");
+                process.exitCode = 1;
                 return;
             }
             // its in the same directory, so always just the basename
-            sourceMapOptions.sourceMapOutputFilename = path.basename(output);
-            sourceMapOptions.sourceMapFullFilename = output + ".map";
+            if (output) {
+                sourceMapOptions.sourceMapOutputFilename = path.basename(output);
+                sourceMapOptions.sourceMapFullFilename = output + ".map";
+            }
             // its in the same directory, so always just the basename
-            sourceMapOptions.sourceMapFilename = path.basename(sourceMapOptions.sourceMapFullFilename);
+            if ('sourceMapFullFilename' in sourceMapOptions) {
+                sourceMapOptions.sourceMapFilename = path.basename(sourceMapOptions.sourceMapFullFilename);
+            }
         } else if (options.sourceMap && !sourceMapFileInline) {
             var mapFilename = path.resolve(process.cwd(), sourceMapOptions.sourceMapFullFilename),
                 mapDir = path.dirname(mapFilename),
@@ -316,8 +335,8 @@ function printUsage() {
     }
 
     if (sourceMapOptions.sourceMapRootpath === undefined) {
-        var pathToMap = path.dirname(sourceMapFileInline ? output : sourceMapOptions.sourceMapFullFilename),
-            pathToInput = path.dirname(sourceMapOptions.sourceMapInputFilename);
+        var pathToMap = path.dirname((sourceMapFileInline ? output : sourceMapOptions.sourceMapFullFilename) || '.'),
+            pathToInput = path.dirname(sourceMapOptions.sourceMapInputFilename || '.');
         sourceMapOptions.sourceMapRootpath = path.relative(pathToMap, pathToInput);
     }
 
@@ -325,7 +344,7 @@ function printUsage() {
         console.error("lessc: no input files");
         console.error("");
         printUsage();
-        currentErrorcode = 1;
+        process.exitCode = 1;
         return;
     }
 
@@ -345,7 +364,8 @@ function printUsage() {
 
     if (options.depends) {
         if (!outputbase) {
-            console.log("option --depends requires an output path to be specified");
+            console.error("option --depends requires an output path to be specified");
+            process.exitCode = 1;
             return;
         }
         process.stdout.write(outputbase + ": ");
@@ -366,6 +386,7 @@ function printUsage() {
                     }
                     console.error('lessc: failed to create file ' + filename);
                     console.error(description);
+                    process.exitCode = 1;
                 } else {
                     less.logger.info('lessc: wrote ' + filename);
                 }
@@ -383,7 +404,9 @@ function printUsage() {
     };
 
     var writeOutput = function(output, result, onSuccess) {
-        if (output) {
+        if (options.depends) {
+            onSuccess();
+        } else if (output) {
             ensureDirectory(output);
             fs.writeFile(output, result.css, {encoding: 'utf8'}, function (err) {
                 if (err) {
@@ -395,6 +418,7 @@ function printUsage() {
                     }
                     console.error('lessc: failed to create file ' + output);
                     console.error(description);
+                    process.exitCode = 1;
                 } else {
                     less.logger.info('lessc: wrote ' + output);
                     onSuccess();
@@ -419,7 +443,7 @@ function printUsage() {
     var parseLessFile = function (e, data) {
         if (e) {
             console.error("lessc: " + e.message);
-            currentErrorcode = 1;
+            process.exitCode = 1;
             return;
         }
 
@@ -466,7 +490,7 @@ function printUsage() {
             },
             function(err) {
                 less.writeError(err, options);
-                currentErrorcode = 1;
+                process.exitCode = 1;
             });
     };
 
@@ -485,4 +509,4 @@ function printUsage() {
             parseLessFile(false, buffer);
         });
     }
-})();
+})();