Fix #1194

[jcppman]

since 9.0.6, bundle() returns a read-only stream by var output = readonly(this.pipeline)

but if this._bundled is true, a 'reset' will be triggered, which cause the
problem mentioned in #1194, because this.pipeline !== output after this.reset(), they are not the same thing anymore.

[terinjokes]

LGTM. Possible to get a test in for this?
On Apr 3, 2015 9:22 AM, "Chriest Yu" notifications@github.com wrote:

since 9.0.6, bundle() returns a read-only stream by var output =
readonly(this.pipeline), but if this._bundled is true, a 'reset' will be
triggered, which cause the
problem mentioned in #1194
#1194, because this.pipeline

!== output after this.reset()

You can view, comment on, or merge this pull request online at:

#1195
Commit Summary

• Fix 9.0.6 : Cannot catch errors + have to call bundle() twice #1194

File Changes

• M index.js
  https://github.com/substack/node-browserify/pull/1195/files#diff-0
  (2)

Patch Links:

• https://github.com/substack/node-browserify/pull/1195.patch
• https://github.com/substack/node-browserify/pull/1195.diff

—
Reply to this email directly or view it on GitHub
#1195.

[ghost]

Published in 9.0.7 with a test case. Thanks for the patch!