Webpack2 Fix

[jhnsnc]

Addresses #5 (error occurring with webpack2)

Problem

I was having the same type of errors when trying to set up this loader with webpack2 (i.e. tr.on is not a function). I was just trying to hook up my shaders to run through glslify.

      {
        test: /\.(glsl|vert|v|frag|f)$/,
        loader: 'ify-loader'
      },

Direct Solution

It seems that webpack 2 goes ahead and takes care of a lot of the loading/sequencing behind the scenes, so we get strings for the transforms/tr on line 64 instead of Buffers(?).

Best Solution = ?

@hughsk Perhaps you can let me know if the fix I came up with is actually a good idea. I'm fairly new to webpack/browserify/glslify and I'm not sure what I might be breaking with this "fix". But perhaps this might help you identify the right solution faster.

This "fix" unblocks my development locally for now, but my usage is very simple with default options.

[jeevan-scienaptic]

Kindly merge this PR.

[glorp47]

^^^ Seconded

[rreusser]

This PR does not seem to work for me. In the sense that it might fix some things but breaks the other tests. I've submitted a PR with a failing test using glslify #8. Seeing if I can figure out the fix…

I might have the wrong config though. Bottom line: I think order to get this fixed, what's required is focus on getting it tested :)

[rreusser]

I don't have a fix yet, but the problem seems to center on the fact that glslify returns a string instead of a transform stream or something. The result is that it expects to be able to chain the transforms but can't insert a string into the mix. The solution seems to be to handle string output separately and work it into the array of transforms, but I haven't yet figured out how to do that.

[etpinard]

I don't have a fix yet, but the problem seems to center on the fact that glslify

Which version of glslify? I remember hearing that glslify was broken before their v6 push.

[rreusser]

@etpinard Most recent, I believe. My strategy: insert console.log(typeof source) right here.

[bpostlethwaite]

superseded by #9