brunch / deppack

Extract node modules to browser.
MIT License
4 stars 12 forks source link

ReferenceError: msg is not defined #39

Closed nblackburn closed 8 years ago

nblackburn commented 8 years ago

msg is undefined as it's dependant on data existing to be defined.

brunch watch --server

29 Jul 16:07:45 - info: application started on http://localhost:3333/
29 Jul 16:07:49 - info: compiling
29 Jul 16:07:55 - info: compiling.
/Users/[USERNAME]/Workspace/[PROJECT_NAME]/node_modules/deppack/lib/resolve.js:68
  err = new Error(msg);
                  ^

ReferenceError: msg is not defined
  at glob.then.mods.forEach.friendlyRequireError (/Users/[USERNAME]/Workspace/[PROJECT_NAME]/node_modules/deppack/lib/resolve.js:68:19)
  at /Users/[USERNAME]/Workspace/[PROJECT_NAME]/node_modules/deppack/lib/resolve.js:90:17
  at /Users/[USERNAME]/Workspace/[PROJECT_NAME]/node_modules/browser-resolve/index.js:265:24
  at /Users/[USERNAME]/Workspace/[PROJECT_NAME]/node_modules/resolve/lib/async.js:55:18
  at load (/Users/[USERNAME]/Workspace/[PROJECT_NAME]/node_modules/resolve/lib/async.js:69:43)
  at onex (/Users/[USERNAME]/Workspace/[PROJECT_NAME]/node_modules/resolve/lib/async.js:92:31)
  at /Users/[USERNAME]/Workspace/[PROJECT_NAME]/node_modules/resolve/lib/async.js:22:47
  at FSReqWrap.oncomplete (fs.js:82:15)
lukerollans commented 8 years ago

@paulmillr I'm getting a similar issue while working on a Phoenix app.

Do you have a timeline for when this will be out on npm?

[info] Running Sightseer.Endpoint with Cowboy using http://localhost:4000
/Users/lukerollans/code/sightseer/node_modules/deppack/lib/resolve.js:68
  err = new Error(msg);
                  ^

ReferenceError: msg is not defined
  at friendlyRequireError (/Users/lukerollans/code/sightseer/node_modules/deppack/lib/resolve.js:68:19)
  at browserResolve (/Users/lukerollans/code/sightseer/node_modules/deppack/lib/resolve.js:90:17)
  at /Users/lukerollans/code/sightseer/node_modules/browser-resolve/index.js:265:24
  at /Users/lukerollans/code/sightseer/node_modules/resolve/lib/async.js:46:14
  at process (/Users/lukerollans/code/sightseer/node_modules/resolve/lib/async.js:173:43)
  at ondir (/Users/lukerollans/code/sightseer/node_modules/resolve/lib/async.js:188:17)
  at load (/Users/lukerollans/code/sightseer/node_modules/resolve/lib/async.js:69:43)
  at onex (/Users/lukerollans/code/sightseer/node_modules/resolve/lib/async.js:92:31)
  at /Users/lukerollans/code/sightseer/node_modules/resolve/lib/async.js:22:47
  at FSReqWrap.oncomplete (fs.js:117:15)
nblackburn commented 8 years ago

@lukerollans It's fixed, just waiting for it to be released which i imagine will fall in line with the next brunch release.

paulmillr commented 8 years ago

pushed