less 3.0.0 → 3.0.1

package/.project

@@ -0,0 +1,17 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<projectDescription>
+	<name>less.js</name>
+	<comment>Project less.js created by Buildship.</comment>
+	<projects>
+	</projects>
+	<buildSpec>
+		<buildCommand>
+			<name>org.eclipse.buildship.core.gradleprojectbuilder</name>
+			<arguments>
+			</arguments>
+		</buildCommand>
+	</buildSpec>
+	<natures>
+		<nature>org.eclipse.buildship.core.gradleprojectnature</nature>
+	</natures>
+</projectDescription>

package/.settings/org.eclipse.buildship.core.prefs

@@ -0,0 +1,2 @@
+#Wed Feb 14 13:51:01 PST 2018
+connection.project.dir=

package/.vscode/launch.json

@@ -0,0 +1,14 @@
+{
+    // Use IntelliSense to learn about possible attributes.
+    // Hover to view descriptions of existing attributes.
+    // For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
+    "version": "0.2.0",
+    "configurations": [
+        {
+            "type": "node",
+            "request": "launch",
+            "name": "Launch Program",
+            "program": "${workspaceFolder}/index"
+        }
+    ]
+}

package/Gruntfile.js

@@ -4,7 +4,7 @@ module.exports = function (grunt) {
 
 
     grunt.option('stack', true)
-    
+
     // Report the elapsed execution time of tasks.
     require('time-grunt')(grunt);
 
@@ -67,19 +67,19 @@ module.exports = function (grunt) {
             platform: 'Linux'
         }
     ];
-       
+
     var sauceJobs = {};
 
     var browserTests = [   "filemanager-plugin",
         "visitor-plugin",
-        "global-vars", 
-        "modify-vars", 
-        "production", 
+        "global-vars",
+        "modify-vars",
+        "production",
         "rootpath-relative",
-        "rootpath", 
-        "relative-urls", 
-        "browser", 
-        "no-js-errors", 
+        "rootpath",
+        "relative-urls",
+        "browser",
+        "no-js-errors",
         "legacy"
     ];
 
@@ -116,7 +116,7 @@ module.exports = function (grunt) {
                     // test. Passing undefined does not alter the test result. Please note that this
                     // only affects the grunt task's result. You have to explicitly update the Sauce
                     // Labs job's status via its REST API, if you want so.
-            
+
                     // This should be the encrypted value in Travis
                     var user = process.env.SAUCE_USERNAME;
                     var pass = process.env.SAUCE_ACCESS_KEY;
@@ -142,7 +142,7 @@ module.exports = function (grunt) {
                             }
                         });
                     });
-                        
+
                 }
             }
         };
@@ -174,7 +174,7 @@ module.exports = function (grunt) {
 
         shell: {
             options: {
-                stdout: true, 
+                stdout: true,
                 failOnError: true,
                 execOptions: {
                     maxBuffer: Infinity
@@ -186,6 +186,9 @@ module.exports = function (grunt) {
             benchmark: {
                 command: 'node benchmark/index.js'
             },
+            plugin: {
+                command: 'node bin/lessc --clean-css="--s1 --advanced" test/less/lazy-eval.less tmp/lazy-eval.css'
+            },
             "sourcemap-test": {
                 command: [
                     'node bin/lessc --source-map=test/sourcemaps/maps/import-map.map test/less/import.less test/sourcemaps/import.css',
@@ -258,10 +261,10 @@ module.exports = function (grunt) {
         },
 
         eslint: {
-            target: ["Gruntfile.js", 
-                "test/**/*.js", 
-                "lib/less*/**/*.js", 
-                "bin/lessc", 
+            target: ["Gruntfile.js",
+                "test/**/*.js",
+                "lib/less*/**/*.js",
+                "bin/lessc",
                 "!test/browser/jasmine-jsreporter.js",
                 "!test/less/errors/plugin/plugin-error.js"
             ],
@@ -289,11 +292,11 @@ module.exports = function (grunt) {
                 // src is used to build list of less files to compile
                 src: [
                     'test/less/*.less',
-                    // Don't test NPM import, obviously 
+                    // Don't test NPM import, obviously
                     '!test/less/plugin-module.less',
                     '!test/less/import-module.less',
-                    '!test/less/javascript.less', 
-                    '!test/less/urls.less', 
+                    '!test/less/javascript.less',
+                    '!test/less/urls.less',
                     '!test/less/empty.less'
                 ],
                 options: {
@@ -422,7 +425,7 @@ module.exports = function (grunt) {
                     specs: 'test/browser/runner-filemanagerPlugin.js',
                     outfile: 'tmp/browser/test-runner-filemanager-plugin.html'
                 }
-            }            
+            }
         },
 
         'saucelabs-jasmine': sauceJobs,
@@ -519,6 +522,7 @@ module.exports = function (grunt) {
         'clean',
         'eslint',
         'shell:test',
+        'shell:plugin',
         'browsertest'
     ];
 

package/README.md

@@ -1,4 +1,4 @@
-[![npm version](https://badge.fury.io/js/less.svg)](http://badge.fury.io/js/less) [![Build Status](https://travis-ci.org/less/less.js.svg?branch=master)](https://travis-ci.org/less/less.js) [![Build status](https://ci.appveyor.com/api/projects/status/bx2qspy3qbuxpl9q/branch/3.x?svg=true)](https://ci.appveyor.com/project/lukeapage/less-js/branch/3.x) [![Dependencies](https://david-dm.org/less/less.js.svg)](https://david-dm.org/less/less.js) [![devDependency Status](https://david-dm.org/less/less.js/dev-status.svg)](https://david-dm.org/less/less.js#info=devDependencies) [![optionalDependency Status](https://david-dm.org/less/less.js/optional-status.svg)](https://david-dm.org/less/less.js#info=optionalDependencies) [![Twitter Follow](https://img.shields.io/twitter/follow/lesstocss.svg?style=flat-square)](https://twitter.com/lesstocss) [![Join the chat at https://gitter.im/less/less.js](https://badges.gitter.im/Join%20Chat.svg)](https://gitter.im/less/less.js?utm_source=badge&utm_medium=badge&utm_campaign=pr-badge&utm_content=badge) <sup>Chat with Less.js users</sup>
+[![npm version](https://badge.fury.io/js/less.svg)](http://badge.fury.io/js/less) [![Build Status](https://travis-ci.org/less/less.js.svg?branch=master)](https://travis-ci.org/less/less.js) [![Build status](https://ci.appveyor.com/api/projects/status/bx2qspy3qbuxpl9q/branch/master?svg=true)](https://ci.appveyor.com/project/lukeapage/less-js/branch/master) [![Dependencies](https://david-dm.org/less/less.js.svg)](https://david-dm.org/less/less.js) [![devDependency Status](https://david-dm.org/less/less.js/dev-status.svg)](https://david-dm.org/less/less.js#info=devDependencies) [![optionalDependency Status](https://david-dm.org/less/less.js/optional-status.svg)](https://david-dm.org/less/less.js#info=optionalDependencies) [![Twitter Follow](https://img.shields.io/twitter/follow/lesstocss.svg?style=flat-square)](https://twitter.com/lesstocss) [![Join the chat at https://gitter.im/less/less.js](https://badges.gitter.im/Join%20Chat.svg)](https://gitter.im/less/less.js?utm_source=badge&utm_medium=badge&utm_campaign=pr-badge&utm_content=badge) <sup>Chat with Less.js users</sup>
 
 [![Sauce Test Status](https://saucelabs.com/browser-matrix/less.svg)](https://saucelabs.com/u/less)
 

package/bin/lessc

@@ -3,6 +3,7 @@
 var path = require('path'),
     fs = require('../lib/less-node/fs'),
     os = require("os"),
+    utils = require('../lib/less/utils'),
     errno,
     mkdirp;
 
@@ -13,7 +14,8 @@ try {
 }
 
 var less = require('../lib/less-node'),
-    pluginLoader = new less.PluginLoader(less),
+    pluginManager = new less.PluginManager(less),
+    fileManager = new less.FileManager(),
     plugins = [],
     queuePlugins = [],
     args = process.argv.slice(1),
@@ -22,6 +24,7 @@ var less = require('../lib/less-node'),
     options = less.options;
 
 options.plugins = plugins;
+options.reUsePluginManager = true;
 
 var sourceMapOptions = {};
 var continueProcessing = true;
@@ -75,7 +78,7 @@ var sourceMapFileInline = false;
 
 function printUsage() {
     less.lesscHelper.printUsage();
-    pluginLoader.printUsage(plugins);
+    pluginManager.Loader.printUsage(plugins);
     continueProcessing = false;
 }
 function render() {
@@ -137,6 +140,7 @@ function render() {
         sourceMapOptions.sourceMapRootpath = path.relative(pathToMap, pathToInput);
     }
 
+
     if (!input) {
         console.error("lessc: no input files");
         console.error("");
@@ -328,18 +332,18 @@ function processPluginQueue() {
         }
     }
     queuePlugins.forEach(function(queue) {
-        pluginLoader.tryLoadPlugin(queue.name, function(err, data) {
-            if (err) {
-                pluginError(queue.name);
-            }
-            else {
+        var context = utils.clone(options);
+        pluginManager.Loader.loadPlugin(queue.name, process.cwd(), context, less.environment, fileManager)
+            .then(function(data) {
                 pluginFinished({
                     fileContent: data.contents,
                     filename: data.filename,
                     options: queue.options
                 });
-            }
-        });
+            })
+            .catch(function() {
+                pluginError(queue.name);
+            });
     });
 }
 
@@ -414,7 +418,7 @@ function processPluginQueue() {
                 options.javascriptEnabled = true;
                 break;
             case 'no-js':
-                console.error('The "--no-js" argument is deprecated, as inline JavaScript ' + 
+                console.error('The "--no-js" argument is deprecated, as inline JavaScript ' +
                     'is disabled by default. Use "--js" to enable inline JavaScript (not recommended).');
                 break;
             case 'include-path':

package/dist/less.js

@@ -1,5 +1,5 @@
 /*!
- * Less - Leaner CSS v3.0.0
+ * Less - Leaner CSS v3.0.1
  * http://lesscss.org
  *
  * Copyright (c) 2009-2018, Alexis Sellier <self@cloudhead.net>
@@ -895,7 +895,7 @@ PluginLoader.prototype.loadPlugin = function(filename, basePath, context, enviro
     return new Promise(function(fulfill, reject) {
         fileManager.loadFile(filename, basePath, context, environment)
             .then(fulfill).catch(reject);
-    });        
+    });
 };
 
 module.exports = PluginLoader;
@@ -1473,7 +1473,7 @@ AbstractPluginLoader.prototype.evalPlugin = function(contents, context, imports,
         fileInfo: fileInfo
     };
     registry = functionRegistry.create();
-    
+
     var registerPlugin = function(obj) {
         pluginObj = obj;
     };
@@ -1484,7 +1484,7 @@ AbstractPluginLoader.prototype.evalPlugin = function(contents, context, imports,
     } catch (e) {
         return new this.less.LessError(e, imports, filename);
     }
-      
+
     if (!pluginObj) {
         pluginObj = localModule.exports;
     }
@@ -1509,7 +1509,7 @@ AbstractPluginLoader.prototype.evalPlugin = function(contents, context, imports,
             e.message = 'Error during @plugin call';
             return new this.less.LessError(e, imports, filename);
         }
-        
+
     }
     else {
         return new this.less.LessError({ message: "Not a valid plugin" });
@@ -1586,6 +1586,11 @@ module.exports = AbstractPluginLoader;
 
 
 },{"../functions/function-registry":26,"../less-error":36}],19:[function(require,module,exports){
+/**
+ * @todo Document why this abstraction exists, and the relationship between
+ *       environment, file managers, and plugin manager
+ */
+
 var logger = require("../logger");
 var environment = function(externalEnvironment, fileManagers) {
     this.fileManagers = fileManagers || [];
@@ -2838,7 +2843,7 @@ module.exports = function(environment, fileManagers) {
     var SourceMapOutput, SourceMapBuilder, ParseTree, ImportManager, Environment;
 
     var initial = {
-        version: [3, 0, 0],
+        version: [3, 0, 1],
         data: require('./data'),
         tree: require('./tree'),
         Environment: (Environment = require("./environment/environment")),
@@ -2911,10 +2916,10 @@ var utils = require('./utils');
  * @prop {string[]} extract
  *
  * @param {Object} e              - An error object to wrap around or just a descriptive object
- * @param {Object} importManager  - An instance of ImportManager (see import-manager.js)
+ * @param {Object} fileContentMap - An object with file contents in 'contents' property (like importManager) @todo - move to fileManager?
  * @param {string} [currentFilename]
  */
-var LessError = module.exports = function LessError(e, importManager, currentFilename) {
+var LessError = module.exports = function LessError(e, fileContentMap, currentFilename) {
     Error.call(this);
 
     var filename = e.filename || currentFilename;
@@ -2922,13 +2927,13 @@ var LessError = module.exports = function LessError(e, importManager, currentFil
     this.message = e.message;
     this.stack = e.stack;
 
-    if (importManager && filename) {
-        var input = importManager.contents[filename],
+    if (fileContentMap && filename) {
+        var input = fileContentMap.contents[filename],
             loc = utils.getLocation(e.index, input),
             line = loc.line,
             col  = loc.column,
             callLine = e.call && utils.getLocation(e.call, input).line,
-            lines = input.split('\n');
+            lines = input ? input.split('\n') : '';
 
         this.type = e.type || 'Syntax';
         this.filename = filename;
@@ -2951,7 +2956,7 @@ var LessError = module.exports = function LessError(e, importManager, currentFil
 
         this.callLine = callLine + 1;
         this.callExtract = lines[callLine];
-        
+
         this.extract = [
             lines[this.line - 2],
             lines[this.line - 1],
@@ -3167,7 +3172,7 @@ module.exports = function(environment, ParseTree, ImportManager) {
         } else {
             var context,
                 rootFileInfo,
-                pluginManager = new PluginManager(this, true);
+                pluginManager = new PluginManager(this, !options.reUsePluginManager);
 
             options.pluginManager = pluginManager;
 
@@ -3213,7 +3218,7 @@ module.exports = function(environment, ParseTree, ImportManager) {
                     }
                 });
             }
-            
+
             new Parser(context, imports, rootFileInfo)
                 .parse(input, function (e, root) {
                     if (e) { return callback(e); }
@@ -5755,7 +5760,7 @@ PluginManager.prototype.getFileManagers = function() {
     return this.fileManagers;
 };
 
-// 
+//
 module.exports = PluginManagerFactory;
 
 },{}],44:[function(require,module,exports){