Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Export only the browserify transform by default, allowing for webpack… #7

Closed
wants to merge 1 commit into from

Conversation

tmcw
Copy link
Contributor

@tmcw tmcw commented Nov 3, 2015

… intercompatibility.

Downstream: mapbox/mapbox-gl-js#1649

@tmcw
Copy link
Contributor Author

tmcw commented Nov 3, 2015

cc @jfirebaugh for the review

@jfirebaugh
Copy link
Contributor

I don't think this'll work for how mapbox-gl-js uses glify. require('glify')(path) has to be callable and do different things if depending on if it's being called as a transform or as a normal method.

I'm not sure why we even need glify though. The only thing it does nowadays is run the source through glsl-compiler, which I guess is for minification but probably doesn't produce that much benefit. We might be better off avoiding the hassle and using brfs instead.

cc @kkaefer @ansis

@jfirebaugh
Copy link
Contributor

Closing, see mapbox/mapbox-gl-js#1701.

@jfirebaugh jfirebaugh closed this Nov 12, 2015
@jfirebaugh jfirebaugh deleted the explicit-environments branch November 12, 2015 00:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants