From 2bec35e69369639a8ebdd4ce4936678111b43c82 Mon Sep 17 00:00:00 2001 From: Nikolai Vavilov Date: Sat, 6 Dec 2014 16:42:24 +0200 Subject: [PATCH] path: don't lower-cases drive letters Changing drive letters to lowercase violates the principle of least surprise. PR-URL: https://github.com/iojs/io.js/pull/100 --- lib/path.js | 11 ----------- test/simple/test-module-nodemodulepaths.js | 5 ++--- test/simple/test-path.js | 2 +- 3 files changed, 3 insertions(+), 15 deletions(-) diff --git a/lib/path.js b/lib/path.js index 1765ef075d3e..27c148ab535f 100644 --- a/lib/path.js +++ b/lib/path.js @@ -153,12 +153,6 @@ win32.resolve = function() { resolvedTail = normalizeArray(resolvedTail.split(/[\\\/]+/), !resolvedAbsolute).join('\\'); - // If device is a drive letter, we'll normalize to lower case. - if (resolvedDevice && resolvedDevice.charAt(1) === ':') { - resolvedDevice = resolvedDevice[0].toLowerCase() + - resolvedDevice.substr(1); - } - return (resolvedDevice + (resolvedAbsolute ? '\\' : '') + resolvedTail) || '.'; }; @@ -172,11 +166,6 @@ win32.normalize = function(path) { tail = result[3], trailingSlash = /[\\\/]$/.test(tail); - // If device is a drive letter, we'll normalize to lower case. - if (device && device.charAt(1) === ':') { - device = device[0].toLowerCase() + device.substr(1); - } - // Normalize the tail path tail = normalizeArray(tail.split(/[\\\/]+/), !isAbsolute).join('\\'); diff --git a/test/simple/test-module-nodemodulepaths.js b/test/simple/test-module-nodemodulepaths.js index af44840b4b49..3d48d99ab94b 100644 --- a/test/simple/test-module-nodemodulepaths.js +++ b/test/simple/test-module-nodemodulepaths.js @@ -21,7 +21,6 @@ var common = require('../common'); var assert = require('assert'); -var path = require('path'); var module = require('module'); @@ -30,7 +29,7 @@ var isWindows = process.platform === 'win32'; var file, delimiter, paths; if (isWindows) { - file = path.normalize('C:\\Users\\Rocko Artischocko\\node_stuff\\foo'); + file = 'C:\\Users\\Rocko Artischocko\\node_stuff\\foo'; delimiter = '\\' } else { file = '/usr/test/lib/node_modules/npm/foo'; @@ -40,4 +39,4 @@ if (isWindows) { paths = module._nodeModulePaths(file); assert.ok(paths.indexOf(file + delimiter + 'node_modules') !== -1); -assert.ok(Array.isArray(paths)); +assert.ok(Array.isArray(paths)); \ No newline at end of file diff --git a/test/simple/test-path.js b/test/simple/test-path.js index cdd59bcd5d93..7a8a1b14a904 100644 --- a/test/simple/test-path.js +++ b/test/simple/test-path.js @@ -311,7 +311,7 @@ if (isWindows) { [['c:/ignore', 'd:\\a/b\\c/d', '\\e.exe'], 'd:\\e.exe'], [['c:/ignore', 'c:/some/file'], 'c:\\some\\file'], [['d:/ignore', 'd:some/dir//'], 'd:\\ignore\\some\\dir'], - [['.'], path.normalize(process.cwd())], + [['.'], process.cwd()], [['//server/share', '..', 'relative\\'], '\\\\server\\share\\relative'], [['c:/', '//'], 'c:\\'], [['c:/', '//dir'], 'c:\\dir'],