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

Capitalization of rest.js still throws error #14

Closed
jmrr opened this issue Aug 6, 2015 · 4 comments · Fixed by #20
Closed

Capitalization of rest.js still throws error #14

jmrr opened this issue Aug 6, 2015 · 4 comments · Fixed by #20

Comments

@jmrr
Copy link

jmrr commented Aug 6, 2015

I also found a similar error to #5

Error: module "./rest.js" not found from "./explorer/client/js/app/modules/persistence/persistence.js"
As per in the reported bug, I lowercased REST.js to rest.js and problem solved, but I'm just reporting that the bug still exists in a post-fix HEAD.

I'm running Ubuntu 14.10 node v0.12.7

@cooper-22
Copy link

I think I am also having this issue with a clean install on 18/8/2015

[10:27:52] Using gulpfile ~/Downloads/explorer-master/gulpfile.js
[10:27:52] Starting 'build-scripts'...
[10:27:52] Starting 'images'...
[10:27:52] Starting 'fonts'...
[10:27:52] Starting 'build-styles'...
[10:27:52] Starting 'connect'...
[10:27:52] Server started http://localhost:8082
[10:27:52] Finished 'connect' after 30 ms
[10:27:52] Starting 'watch'...
[10:27:52] Finished 'watch' after 84 ms
Autoprefixer's process() method is deprecated and will removed in next major release. Use postcss([autoprefixer]).process() instead
[10:27:53] Finished 'build-styles' after 430 ms
[10:27:53] Finished 'images' after 437 ms
[10:27:53] Finished 'fonts' after 439 ms

events.js:72
        throw er; // Unhandled 'error' event
              ^
Error: module "./rest.js" not found from "/home/alex/Downloads/explorer-master/client/js/app/modules/persistence/persistence.js"
    at notFound (/home/alex/Downloads/explorer-master/node_modules/browserify/index.js:812:15)
    at /home/alex/Downloads/explorer-master/node_modules/browserify/index.js:762:23
    at /home/alex/Downloads/explorer-master/node_modules/browserify/node_modules/browser-resolve/index.js:250:24
    at /home/alex/Downloads/explorer-master/node_modules/browserify/node_modules/browser-resolve/node_modules/resolve/lib/async.js:55:18
    at load (/home/alex/Downloads/explorer-master/node_modules/browserify/node_modules/browser-resolve/node_modules/resolve/lib/async.js:69:43)
    at onex (/home/alex/Downloads/explorer-master/node_modules/browserify/node_modules/browser-resolve/node_modules/resolve/lib/async.js:92:31)
    at /home/alex/Downloads/explorer-master/node_modules/browserify/node_modules/browser-resolve/node_modules/resolve/lib/async.js:22:47
    at Object.oncomplete (fs.js:107:15)

@josephwegner
Copy link

Well, this is super weird. That file should definitely be there!

Can you do a git pull and try again, just to make sure you're updated?

If you're still having the issue, knowing what OS you're using, as well as your node version, would be very helpful.

Thanks!

@josephwegner josephwegner reopened this Aug 18, 2015
@cooper-22
Copy link

I renamed the file manually and that worked fine.

This is the first time I've used node so you may have to bear with me I'm afraid. I think it is: v0.10.25
I'm using Ubuntu 14.04. I downloaded the dashboards-gh-pages.zip file, uncompressed it and inside that dir ran npm install, which failed towards the end.
I had to install a legacy node version I believe.
Finally I ran gulp, which then could not find the rest.js file. (renaming REST.js then worked)

If there is any other info you need I'm happy to get it.

Thanks

@josephwegner
Copy link

👊 DOH!

I thought the name of that file was already lowercase. Stupid me. I probably broke this more with that PR.

I'll PR really quick to lowercase the actual file name, to match all of the refernces.

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

Successfully merging a pull request may close this issue.

3 participants