Skip to content
This repository has been archived by the owner on Aug 4, 2021. It is now read-only.

Commit

Permalink
Removed toLowerCase() on windows, as rollup creates dups
Browse files Browse the repository at this point in the history
  • Loading branch information
danielgindi authored and Daniel Cohen Gindi committed Sep 3, 2018
1 parent 652d554 commit 8067ec4
Show file tree
Hide file tree
Showing 4 changed files with 0 additions and 46 deletions.
10 changes: 0 additions & 10 deletions src/transform.js
Expand Up @@ -37,12 +37,7 @@ function tryParse ( parse, code, id ) {
}
}

const shouldLowercaseDynamicPaths = process.platform === 'win32';

export function normalizeDynamicModulePath (path) {
if ( shouldLowercaseDynamicPaths )
path = path.toLowerCase();

return path.replace(/\\/g, '/');
}

Expand Down Expand Up @@ -242,11 +237,6 @@ export function transformCommonjs ( parse, code, id, isEntry, ignoreGlobal, igno
if ( node.name in uses ) {
if ( node.name === 'require' ) {
if ( isRequireStatement( parent ) ) {
if ( shouldLowercaseDynamicPaths ) {
magicString.appendLeft( parent.arguments[0].start, '(' );
magicString.appendLeft( parent.arguments[0].end, ').toLowerCase()' );
}

magicString.appendLeft( parent.end - 1, ',' + JSON.stringify( dirname(id) === '.' ? null : normalizeDynamicModulePath( dirname(id) ) ) );
}

Expand Down
12 changes: 0 additions & 12 deletions test/form/dynamic-template-literal/output.win32.js

This file was deleted.

12 changes: 0 additions & 12 deletions test/form/ignore-ids-function/output.win32.js

This file was deleted.

12 changes: 0 additions & 12 deletions test/form/ignore-ids/output.win32.js

This file was deleted.

0 comments on commit 8067ec4

Please sign in to comment.