Set default of parser option "relativeUrls" to "true"

This is a breaking change!
In order to get the old behavior back, please set the
"parser.relativeUrls" option to "false".

As ui5 themes usually import other themes (e.g. base) to inherit
variables, mixins and rules it is required to set the "relativeUrls"
option to "true" in order to generate valid URLs (e.g. images/fonts).

The default in LESS 1.6.3 is "false".

README.md

@@ -110,7 +110,8 @@ lib2
 
 Type: `object`
 
-Options for the [less](http://lesscss.org) parser (`less.Parser`).
+Options for the [less](http://lesscss.org) parser (`less.Parser`).  
+**Note:** Default of `relativeUrls` option is changed from `false` to `true`.
 
 ##### compiler
 

lib/index.js

@@ -46,6 +46,11 @@ module.exports.build = function(input, options, callback) {
 		library: {}
 	}, options);
 
+    // Set default of "relativeUrls" parser option to "true" (less default is "false")
+    if (!options.parser.hasOwnProperty("relativeUrls")) {
+        options.parser.relativeUrls = true;
+    }
+
 	var result = {
 		variables: {},
 		css: '',

test/expected/imports/main-no-relativeUrls.css

@@ -0,0 +1,11 @@
+@import "dir3/external.css";
+/* dir3/inline.css */
+
+/* dir1/foo.less */
+.foo {
+  background: url("foo.png");
+}
+/* dir2/bar.less */
+.bar {
+  background: url("bar.png");
+}

test/expected/imports/main.css

@@ -2,4 +2,10 @@
 /* dir3/inline.css */
 
 /* dir1/foo.less */
+.foo {
+  background: url("dir1/foo.png");
+}
 /* dir2/bar.less */
+.bar {
+  background: url("dir2/bar.png");
+}

test/fixtures/imports/dir1/foo.less

@@ -1 +1,5 @@
 /* dir1/foo.less */
+
+.foo {
+	background: url("foo.png");
+}

test/fixtures/imports/dir2/bar.less

@@ -1 +1,5 @@
 /* dir2/bar.less */
+
+.bar {
+	background: url("bar.png");
+}

test/test.js

@@ -250,6 +250,45 @@ describe('imports', function() {
 
   });
 
+  it('should use "relativeUrls" parser option by default', function(done) {
+
+    lessOpenUI5.build(readFile('test/fixtures/imports/main.less'), {
+      parser: {
+        filename: 'main.less',
+        paths: [ 'test/fixtures/imports' ]
+      }
+    }, function(err, result) {
+
+      assert.ifError(err);
+
+      assert.equal(result.css, readFile('test/expected/imports/main.css'), 'css should be correctly generated.');
+
+      done();
+
+    });
+
+  });
+
+  it('should not rewrite urls when "relativeUrls" parser option is set to "false"', function(done) {
+
+    lessOpenUI5.build(readFile('test/fixtures/imports/main.less'), {
+      parser: {
+        filename: 'main.less',
+        paths: [ 'test/fixtures/imports' ],
+        relativeUrls: false
+      }
+    }, function(err, result) {
+
+      assert.ifError(err);
+
+      assert.equal(result.css, readFile('test/expected/imports/main-no-relativeUrls.css'), 'css should be correctly generated.');
+
+      done();
+
+    });
+
+  });
+
 });
 
 describe('inline theming parameters', function() {