[feature/bug fix] Support for ES6 modules transpiled by babel >= 6

[kkwiatkowski]

Motivation

When you use Babel 6 to transpile code, the exports varies a bit from regular CommonJS exports.
And because of that loaders written in ES6, transpiled by Babel 6 are not handled correctly in Webpack.

> webpack

Hash: 203cfe0179c71fd5b113
Version: webpack 1.13.0
Time: 903ms
        Asset     Size  Chunks             Chunk Names
./bin/main.js  1.95 kB       0  [emitted]  main
    + 3 hidden modules

ERROR in Loader c:\www\dll-prototype\dll-linker-example\node_modules\dll\lib\linker?dll-wrapper-example didn't return a function
 @ ./src/main.js 3:25-55

This is also related with: #webpack/issues/2047

Pull request

I don't know if this is proper place for that, but I hope it's fine.
Would be great, if it's treated as patch, if merged.

[kkwiatkowski]

Any feedback about this pull request, @sokra?

[kkwiatkowski]

Is there any chance to not let it die, @sokra?

[sokra]

Sorry, webpack/core is no longer used by webpack.

webpack 2 now uses https://github.com/webpack/loader-runner

and it supports loaders exporting ES6 https://github.com/webpack/loader-runner/blob/master/lib/loadLoader.js#L31

I'm willing to merge your changes, but they need to be compatible with the webpack 2 format:

export default function(source) { return source; /* normal */ }
export function pitch(...) { /* pitch */ }

Your PR doesn't implement the pitch part correctly.

You could move the default read before this line: https://github.com/webpack/core/blob/master/lib/NormalModuleMixin.js#L290

[kkwiatkowski]

Ok, I'll do that. Thanks for answer 👍