Fixes #10 - IE/Edge source maps.

Enforces line break before source map comment.

Why:

* Edge / IE doesn't like source map comment in the same line as other
  content wheras other browsers don't complain.

Author: jakubpawlowicz

Diff for: History.md

@@ -1,3 +1,8 @@
+[4.1.4 / 2017-xx-xx](https://github.com/jakubpawlowicz/clean-css-cli/compare/v4.1.3...4.1)
+==================
+
+* Fixed issue [#10](https://github.com/jakubpawlowicz/clean-css-cli/issues/10) - IE/Edge source maps.
+
 [4.1.3 / 2017-05-18](https://github.com/jakubpawlowicz/clean-css-cli/compare/v4.1.2...v4.1.3)
 ==================
 

Diff for: index.js

@@ -6,6 +6,7 @@ var commands = require('commander');
 var glob = require('glob');
 
 var COMPATIBILITY_PATTERN = /([\w\.]+)=(\w+)/g;
+var lineBreak = require('os').EOL;
 
 function cli(process, beforeMinifyCallback) {
   var packageConfig = fs.readFileSync(path.join(__dirname, 'package.json'));
@@ -245,7 +246,7 @@ function minify(process, beforeMinifyCallback, options, debugMode, removeInlined
 
     if (minified.sourceMap) {
       mapFilename = path.basename(options.output) + '.map';
-      output(process, options, minified.styles + '/*# sourceMappingURL=' + mapFilename + ' */');
+      output(process, options, minified.styles + lineBreak + '/*# sourceMappingURL=' + mapFilename + ' */');
       outputMap(options, minified.sourceMap, mapFilename);
     } else {
       output(process, options, minified.styles);

Diff for: test/binary-test.js

@@ -6,6 +6,7 @@ var path = require('path');
 var url = require('url');
 
 var httpProxy = require('http-proxy');
+var lineBreak = require('os').EOL;
 var SourceMapConsumer = require('source-map').SourceMapConsumer;
 var vows = require('vows');
 
@@ -478,7 +479,7 @@ vows.describe('cleancss')
   .addBatch({
     'source maps - output file': binaryContext('--source-map -o ./reset.min.css ./test/fixtures/reset.css', {
       'includes map in minified file': function () {
-        assert.include(fs.readFileSync('./reset.min.css', 'utf-8'), '/*# sourceMappingURL=reset.min.css.map */');
+        assert.include(fs.readFileSync('./reset.min.css', 'utf-8'), lineBreak + '/*# sourceMappingURL=reset.min.css.map */');
       },
       'creates a map file': function () {
         assert.isTrue(fs.existsSync('./reset.min.css.map'));
@@ -572,7 +573,7 @@ vows.describe('cleancss')
   .addBatch({
     'source maps - with input source map': binaryContext('--source-map -o ./import.min.css ./test/fixtures/source-maps/import.css', {
       'includes map in minified file': function () {
-        assert.include(fs.readFileSync('./import.min.css', 'utf-8'), '/*# sourceMappingURL=import.min.css.map */');
+        assert.include(fs.readFileSync('./import.min.css', 'utf-8'), lineBreak + '/*# sourceMappingURL=import.min.css.map */');
       },
       'includes right content in map file': function () {
         var sourceMap = new SourceMapConsumer(fs.readFileSync('./import.min.css.map', 'utf-8'));
@@ -590,7 +591,7 @@ vows.describe('cleancss')
   .addBatch({
     'source maps - with input source map and source inlining': binaryContext('--source-map --source-map-inline-sources -o ./import-inline.min.css ./test/fixtures/source-maps/import.css', {
       'includes map in minified file': function () {
-        assert.include(fs.readFileSync('./import-inline.min.css', 'utf-8'), '/*# sourceMappingURL=import-inline.min.css.map */');
+        assert.include(fs.readFileSync('./import-inline.min.css', 'utf-8'), lineBreak + '/*# sourceMappingURL=import-inline.min.css.map */');
       },
       'includes embedded sources': function () {
         var sourceMap = new SourceMapConsumer(fs.readFileSync('./import-inline.min.css.map', 'utf-8'));