Support on-fly browserification with node-browserify

[ghost]

Would be great if this npm package could be compatible with browserify so we don't have to use the minified version.

https://github.com/substack/node-browserify

At the moment I get an error when using the load() function.

[ixti]

In fact we use node-browserify to build a browserified version internally. Here's what we are doing (from Makefile):

browserify:
    if test ! `which browserify` ; then npm install browserify ; fi
    cp -r support/browserify/ ${TMP_DIR}
    browserify index.js -o ${TMP_DIR}/50_js-yaml.js
    cat ${TMP_DIR}/* > js-yaml.js
    rm -rf ${TMP_DIR}
    cp js-yaml.js demo/js/

In other words, we browserify js-yaml with node-browserify and then prepend and append to the result some portions of JavaScript that make us possible to add some browser-specific hacks and enclose everything to provide only one variable into global scope - jsyaml.

Please, explain which error you get exactly?

[ghost]

I installed js-yaml locally and I'm trying to require() it with browserify:

jsyaml = require "js-yaml"
console.log jsyaml.load "name:Johnny Luu"

It says:

Uncaught ReferenceError: jsyaml is not defined

So at the moment I'm better of to just include the "js-yaml.min" file.

Would be simpler to use the npm package in the browser since you can upgrade it (through package.json) once both for server and browser.

[ixti]

Hm. That's strange. If somebody will point why that might happen - we'll be glad to fix it. And will appreciate patches. But due to limited sources and time we can't handle this issue in the closest future. But the idea is really awesome!

[ghost]

Just tried it out again it actually worked!

I just forgot to require() it in another place of my app.

However, it takes browserify 10-15 seconds to concatenate all js-yaml files. But perhaps that is something browserify should get better at.

I'm glad that it worked. No need to keep the minified browser file up to date anymore! :)

Other people have this problem as well: browserify/browserify#90