gulpjs / glob-stream

Readable streamx interface over anymatch.
MIT License
178 stars 51 forks source link

5.3 breaks build in webpack #58

Closed fazo96 closed 8 years ago

fazo96 commented 8 years ago

Webpack can't build 5.3 because it has requires that are not statically evaluable in the micromatch dependency.

This commit introduced micromatch and this problem: 09f1b60e50a90806522a4072af0bd1222c0b6deb

I'm also filing an issue with micromatch :+1:

jonschlinkert commented 8 years ago

This is a webpack problem that is caused by how webpack renames and reassigns requires (browserify does not have an issue, not does any native node.js module), but it can be resolved with a custom loader (which is totally expected in the webpack ecosystem, right?)

I think @doowb already created a custom loader for it, but if he doesn't release it and/or this continues to be a problem, we'll fix it in micromatch directly asap

phated commented 8 years ago

@jonschlinkert I think this just needs to be fixed in micromatch. I was told there would be no breaking changes so I minor released and this is breaking us

jonschlinkert commented 8 years ago

Fair enough, I'll push up changes to make it work with webpack.

(fwiw, this isn't a bug in micromatch, it's a bug in webpack - but I do understand that, in effect, it looks like a bug here. I'll start doing that now)

jonschlinkert commented 8 years ago

should be fixed. Pushed up a patch a few minutes ago

fazo96 commented 8 years ago

Thanks a lot for your efforts! :)

doowb commented 8 years ago

I also published unlazy loader that can be used in a webpack build so webpack is able to do the static analysis. Since micromatch was patched, this isn't necessary for your case, but I'm still posting it in case anyone else comes across this problem.

jonschlinkert commented 8 years ago

Thanks a lot for you efforts! :)

No problem, thanks for bringing this to our attention

phated commented 8 years ago

Thanks a ton @jonschlinkert - I'm going to cut a release with that version as minimum

jonschlinkert commented 8 years ago

anytime

dignifiedquire commented 8 years ago

Thanks everyone for the quick turn around, really appreciated!