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

Install cli names are the wrong name #6

Closed
jeffhoye opened this issue Sep 1, 2016 · 3 comments
Closed

Install cli names are the wrong name #6

jeffhoye opened this issue Sep 1, 2016 · 3 comments

Comments

@jeffhoye
Copy link

jeffhoye commented Sep 1, 2016

Win 10, cygwin

After an update install, the .js file names in the npm repo are the wrong names.

Workaround is to rename all 4 files in this directory: add '-cli' to the file name:
'C:\Users\me\AppData\Roaming\npm\node_modules\silica\src\cli'

$ silica build
module.js:457
throw err;
^

Error: Cannot find module 'C:\Users\me\AppData\Roaming\npm\node_modules\silica\src\cli\silica-cli-build'
at Function.Module._resolveFilename (module.js:455:15)
at Function.Module._load (module.js:403:25)
at Module.runMain (module.js:590:10)
at run (bootstrap_node.js:394:7)
at startup (bootstrap_node.js:149:9)
at bootstrap_node.js:509:3

@jeffhoye
Copy link
Author

jeffhoye commented Oct 6, 2016

gatsbyjs/gatsby#80

@jeffhoye
Copy link
Author

jeffhoye commented Oct 6, 2016

tj/commander.js#327

@jeffhoye
Copy link
Author

jeffhoye commented Oct 6, 2016

I think the fix is just to rename the following in src/cli/

silica-build.js >> silica-cil-build.js
silica-create.js >> silica-cil-create.js
silica-watch.js >> silica-cil-watch.js

But please try on the mac to make sure it doesn't break it there.

kayleg pushed a commit that referenced this issue Oct 13, 2017
@kayleg kayleg closed this as completed Mar 7, 2019
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

2 participants