Merge pull request #41 from Mopolo/windows-support

Windows support

Author: igorshubovych

lib/config.js

@@ -1,7 +1,8 @@
-var _      = require('lodash');
-var fs     = require('fs');
-var path   = require('path');
-var extend = require('deep-extend');
+var _         = require('lodash');
+var fs        = require('fs');
+var path      = require('path');
+var extend    = require('deep-extend');
+var osHomedir = require('os-homedir');
 
 var config = null;
 
@@ -16,9 +17,9 @@ exports.get = function() {
 
 function load() {
   var DEFAULT  = path.join(__dirname, '..', 'config.json');
-  var CUSTOM = path.join(process.env['HOME'], '.tldrrc');
+  var CUSTOM = path.join(osHomedir(), '.tldrrc');
   var defaultConfig = JSON.parse(fs.readFileSync(DEFAULT));
-  defaultConfig.cache = path.join(process.env['HOME'], '.tldr');
+  defaultConfig.cache = path.join(osHomedir(), '.tldr');
   defaultConfig.proxy = process.env.HTTP_PROXY || process.env.http_proxy;
 
   var customConfig = {};

lib/platform.js

@@ -5,7 +5,8 @@ var folders = {
   'osx': 'osx',
   'darwin': 'osx',
   'linux': 'linux',
-  'sunos': 'sunos'
+  'sunos': 'sunos',
+  'win32': 'win32'
 };
 
 function isSupported(platform) {

package.json

@@ -38,9 +38,6 @@
     "lint": "eslint lib test bin/tldr",
     "watch": "npm run test:quiet -- --reporter=min --watch --growl"
   },
-  "os": [
-    "!win32"
-  ],
   "dependencies": {
     "async": "~1.5.0",
     "colors": "~1.1.2",
@@ -51,6 +48,7 @@
     "marked": "~0.3.5",
     "mkdirp": "~0.5.1",
     "ms": "~0.7.1",
+    "os-homedir": "^1.0.1",
     "request": "~2.67.0",
     "rimraf": "~2.5.0",
     "string": "~3.3.1",

test/platform.spec.js

@@ -40,11 +40,11 @@ describe('Platform', function() {
   });
 
   describe('isSupported', function() {
-    it('should tell that Linux, OSX and SunOS are supported', function() {
+    it('should tell that Linux, OSX, SunOS and Win32 are supported', function() {
       platform.isSupported('osx').should.eql(true);
       platform.isSupported('linux').should.eql(true);
       platform.isSupported('sunos').should.eql(true);
-      platform.isSupported('windows').should.eql(false); // Yet
+      platform.isSupported('win32').should.eql(true);
       platform.isSupported('ios').should.eql(false);
     });
   });