Skip to content
This repository has been archived by the owner on Feb 28, 2018. It is now read-only.

Breaks on module 'slate' #17

Open
natew opened this issue Apr 28, 2017 · 8 comments
Open

Breaks on module 'slate' #17

natew opened this issue Apr 28, 2017 · 8 comments

Comments

@natew
Copy link

natew commented Apr 28, 2017

Attempting to use package 'slate' breaks webpackbin unfortunately, works in browser.

@christianalfoni
Copy link
Member

Hi @natew, you have some more info? I just tried it and it works?

@christianalfoni
Copy link
Member

Oh, sorry, it is combining all three... slate, react and react-dom... that is certainly strange. It is probably picking up some invalid entry point. Have to check

@christianalfoni
Copy link
Member

Super strange... only place I find base64-js is inside the "dist/slate.js" file, but that is being completely ignored. Makes no sense any of that code being included, hm

@christianalfoni
Copy link
Member

christianalfoni commented Apr 28, 2017

Okay... this is surprisingly difficult to figure out. Even made a records.json file... no record of looking into that file. But... what is this dist stuff anyways? Why does it depend on base64-js? Seems to be something else than the lib itself? It does not depend on it in package.json either?

@natew
Copy link
Author

natew commented Apr 28, 2017

Sorry to hear its so weird! Not my package, just one I use. Thanks for making webpackbin, it's really impressive!

@christianalfoni
Copy link
Member

I even tried deleting the dist folder before bundling... still asks for base64-js. Insanely weird

@christianalfoni
Copy link
Member

Okay, I think I found issue... cheerio, dep of slate, uses readable-stream, which uses buffer-shim, which uses buffer... a node core module. There is a webpack config to turn off some buffer stuff, but then we end up with: dll.js:26 Uncaught Error: Cannot find module "../maps/decode.json"

Maaan, this is bugging me so much! :-D

@christianalfoni
Copy link
Member

Sorry, have to give this up :( I did find some interesting things related to generally mocking stuff, so picked that up at least. It is rather strange because this service does use json loader to handle that, but for some reason this deep dependency is not hit.

Thanks for feedback :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants