Fixing manifest paths (by temporarily embedding webpack-manifest-plugin fix)

[weaverryan]

This temporarily embeds shellscape/webpack-manifest-plugin#249, which fixes #907 and also a comment on #914 (comment)

To test this:

$ yarn remove @symfony/webpack-encore
$ yarn add git://github.com/weaverryan/webpack-encore.git#try-manifest-fix

Please let me know if this does or doesn't fix your manifest.json file. I'm also interested in hearing from Windows users, as the fix requires some "path" manipulation.

Thanks!

[minimit]

Works great on mac, can't test on windows

[weaverryan]

Thanks @shellscape for keeping an eye on that ;). I did include it in the new webpack-manifest-plugin directory - I grabbed the LICENSE file from master and put it there - https://github.com/symfony/webpack-encore/pull/921/files#diff-1422c8fd28cafc7a2c9bb9bfba27c0a1598d225ced0644653c1226bbd48604e7 - but let me know if it needs to be somewhere else.

Btw, this embedding I hope is temporary, at best :). I want to get the fix out to our users ASAP because I don't want users to start relying on the new incorrect paths... only for us to change them back later.

[weaverryan]

This is now ready to go. We may never merge it - it depends on shellscape/webpack-manifest-plugin#249 - I'd like to merge and tag a fix ASAP for people. We could always remove this "embedded" plugin easily and re-add the real one.

Cheers!

[weaverryan]

I've fully embedded the fix from shellscape/webpack-manifest-plugin#249

I'm confident that I now have the problem solved correctly there. I'd like to get this fix to our users, so I'm planning to merge this PR and temporarily embed the plugin so we can get it released :).

Cheers!