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

Suggested Setup in README Doesn't Work #67

Open
codeviking opened this issue Oct 14, 2014 · 3 comments
Open

Suggested Setup in README Doesn't Work #67

codeviking opened this issue Oct 14, 2014 · 3 comments

Comments

@codeviking
Copy link

When I dial this into the latest browserify:

var bundle = browserify({
    require : { http : 'http-browserify' }
});

I get an error saying Error: path must be a string.

Please update with correct configuration instructions!

@codeviking codeviking changed the title Suggested Setup in README Doesn't work Suggested Setup in README Doesn't Work Oct 14, 2014
@skyhaikuteam
Copy link

Did you ever manage to resolve this? I'm also experiencing the same issue. Wonder if the browserify api has changed fairly recently.

@codeviking
Copy link
Author

@skyhaikuteam I did not. I ended up not using the package due to the issue.

@deathcap
Copy link

@codeviking @skyhaikuteam browserify now automatically uses http-browserify for http by default:

https://github.com/substack/node-browserify/blob/master/lib/builtins.js

exports.http = require.resolve('http-browserify');

the config from the readme is no longer needed (and should probably be deleted):

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

No branches or pull requests

3 participants