-
-
Notifications
You must be signed in to change notification settings - Fork 227
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
Ghost-CLI 1.5.2 fails in a loop #675
Comments
I can actually reproduce this 100% with the recommended stack as well. Following our install instructions step by step and word by word, but using node 8 instead of 6 results in the loop just as described when running |
Gonna close this because this isn't a CLI issue, and is rather an issue with npm/node-gyp. According to this issue, the fix is to install with There's not much that I can do to remove the gyp dependency short of removing our dependency on |
This definitely needs documentation on https://docs.ghost.org. |
Setting up a new droplet and installing Node 8 and CLI with The first run with
A second try seems to work at first but asking for the installed ghost version (
And trying to install ghost then gives me this:
@kirrg001 I don't think updating the docs is enough here. Please try it yourself as well, maybe there's a workaround that we can add to the docs, but for now, I couldn't get it working. |
So after some deliberation on slack, instead of adding docs to try and work around whatever the issue is in npm/node-gyp, we're gonna remove the dependency on ghost-ignition in favor of extracting the prettylogger class (which is the only file from ghost-ignition that is used). That way, the dtrace-provider dependency (pulled in by ghost-ignition) won't be installed. |
…m class closes TryGhost#675 [ci skip] - drop ghost-ignition dependency - add pretty-stream class from ghost-ignition
closes TryGhost#675 [ci skip] - drop ghost-ignition dependency - add pretty-stream class from ghost-ignition
closes TryGhost#675 [ci skip] - drop ghost-ignition dependency - add pretty-stream class from ghost-ignition
closes TryGhost#675 - drop ghost-ignition dependency - add pretty-stream class from ghost-ignition
closes #675 - drop ghost-ignition dependency - add pretty-stream class from ghost-ignition
The installation of the CLI 1.6.0 was successfully with Node v8. But I am still running into on
Fails on this step
Fresh container, fresh installation.
@acburdine I think we need to reopen this? |
I retry fresh installation of ghost via the CLI. Linux version (it's not an Ubuntu) :
Node and NPM version :
I confirmed the installation of the CLI 1.6.0 with Node v8 is successfully now, Fresh empty ghost folder :
Ghost installation with the CLI :
For me i think all is fine, I've done it twice but I can't reproduce your issue. |
@gh0-0st Thank you so much for letting us know 👍 |
@kirrg001 Listing the
|
@acburdine When do you have time to talk about this issue? |
90% sure this is caused by: https://www.npmjs.com/package/configstore The
There must be a postinstall hook somewhere. |
closes TryGhost#675 - remove update-notifier because we weren't using most of its dependencies - use latest-version and a simple semver check instead
closes #675 - remove update-notifier because we weren't using most of its dependencies - use latest-version and a simple semver check instead
I tested this on Ubuntu and error appeared. This solution worked for me:
|
I've also got this today. Are we still using the |
@kirrg001 we shouldn't be, no |
I was seeing the configstore folder today, that's why i have asked. I installed 1.9 on a fresh container |
I think npm also uses the folder for update notifications |
@vikaspotluri123 you'd be right - npm uses update-notifier, which uses configstore |
And because we execute the installation of the CLI with sudo, the folder is created with root and creates this permission problem. Hmm. |
A quick fix would be to switch This is a bigger underlying permission problem, because yarn tries to write into |
That would work, but also |
The only other quick fix i can think of is to use
|
If I understand correctly, yarn is trying to read the |
Had a chat with @ErisDS and we think it would be cool to add a piece of code into the CLI. So if you call We really don't wanne tell users to use a different command to install the CLI e.g. add |
here is solution: https://forum.ghost.org/t/an-unexpected-error-occurred-eacces-permission-denied/2834
|
@cmlanche yup - gonna add that into the CLI here soon. |
The yarn error for me was fixed by updating permissions for Yarn - detailed in the following post: |
closes TryGhost#675 - convert update check to a listr task set - add ~/.config directory check to list of prechecks
closes TryGhost#675 - convert update check to a listr task set - add ~/.config directory check to list of prechecks
closes #675 - convert update check to a listr task set - add ~/.config directory check to list of prechecks
Still getting this with
So it seems the Ghost-CLI fix is not yet published on npm. Solved with:
|
It was not released cc @acburdine |
I dont really understand this part, before i added another user that will be easier for me to remember adduser newuser Then, when i saw the ghostuser par i decided to also change it, but it throws this: newuser:x:1002:1002:Ghost: command not found Can i get some more help with this. |
A case exists on the problem see #9511.
Since it's closed, I was told to open another one, which I am doing here.
Issue Summary
When proceeding with the standard "ghostuser" account and use "sudo", on a Debian Stretch Linux (9.4), with NodeJS v8.10.0 installed, I try to install Ghost-CLI v1.5.2, it starts and runs in a loop.
Here are the details below:
Steps to Reproduce
Sorry but same issue with 'sudo' and need a break 'Ctrl+C' to exit this loop.
BUT, really curious, if repeat install Ghost-CLI, this upgrade and done :
Check :
I repeated the same test twice and it reproduces the same problem well:
first install failed in loop, break this with Ctrl+C and try second install, this indicate rollback then upgrade and done.
The text was updated successfully, but these errors were encountered: