module: allow long paths for require on Windows

#1801 introduced internal fs methods to speed up require. The methods do not call path._makeLong like their counterpart from the fs module. This brings back the old behaviour. Fixes: #1990Fixes: #1980Fixes: #1849 PR-URL: https://github.com/nodejs/io.js/pull/1991/files Reviewed-By: Bert Belder <[email protected]>

Author: targos

lib/module.js

@@ -67,7 +67,7 @@ function readPackage(requestPath){
 }
 
 varjsonPath=path.resolve(requestPath,'package.json');
-varjson=internalModuleReadFile(jsonPath);
+varjson=internalModuleReadFile(path._makeLong(jsonPath));
 
 if(json===undefined){
 returnfalse;
@@ -100,7 +100,7 @@ Module._realpathCache ={};
 
 // check if the file exists and is not a directory
 functiontryFile(requestPath){
-constrc=internalModuleStat(requestPath);
+constrc=internalModuleStat(path._makeLong(requestPath));
 returnrc===0&&toRealPath(requestPath);
 }
 
@@ -146,7 +146,7 @@ Module._findPath = function(request, paths){
 varfilename;
 
 if(!trailingSlash){
-constrc=internalModuleStat(basePath);
+constrc=internalModuleStat(path._makeLong(basePath));
 if(rc===0){// File.
 filename=toRealPath(basePath);
 }elseif(rc===1){// Directory.

test/parallel/test-require-long-path.js

@@ -0,0 +1,17 @@
+'use strict';
+varcommon=require('../common');
+varfs=require('fs');
+varpath=require('path');
+varassert=require('assert');
+
+// make a path that is more than 260 chars long.
+varfileNameLen=Math.max(261-common.tmpDir.length-1,1);
+varfileName=path.join(common.tmpDir,newArray(fileNameLen+1).join('x'));
+varfullPath=path.resolve(fileName);
+
+common.refreshTmpDir();
+fs.writeFileSync(fullPath,'module.exports = 42;');
+
+assert.equal(require(fullPath),42);
+
+fs.unlinkSync(fullPath);