Skip to content

Conversation

@kratochviljan
Copy link

No description provided.

@tomasz1986
Copy link
Member

There is a conflict with #955, which also updates the same screenshot (although that one is missing --no-restart).

@kratochviljan
Copy link
Author

--no-restart is an important parameter, whereas --no-console, which is shown in the screenshot, should not be there, because then no log is generated. I tried to fix this, since it’s mentioned all over the internet, but no one has actually done it yet.

I experimented with it for quite a while myself, going through all the parameters one by one to find out which ones should be used and which should not. Another misleading aspect is the use of the notation -no-console / -no-restart (as shown in the current screenshot) instead of --no-restart / --no-console, i.e. the double dashes --. Naturally, single dashes - do not work in the current version of SC, and the console (if not disabled) simply logs an error: unknown flag.

@rafaeloledo
Copy link

--no-restart is an important parameter, whereas --no-console, which is shown in the screenshot, should not be there, because then no log is generated. I tried to fix this, since it’s mentioned all over the internet, but no one has actually done it yet.

I experimented with it for quite a while myself, going through all the parameters one by one to find out which ones should be used and which should not. Another misleading aspect is the use of the notation -no-console / -no-restart (as shown in the current screenshot) instead of --no-restart / --no-console, i.e. the double dashes --. Naturally, single dashes - do not work in the current version of SC, and the console (if not disabled) simply logs an error: unknown flag.

The single dashes are fixed on the #955 which i've done and got merged. I've done this on Sep 3.

image

If you'd made a search on the current active PRs with the file name that you're changing, you'll get the place that it's mentioned.

image

rafaeloledo@97e8b97#diff-71b9ce7b38ce392f891fcacbd3ff3a69ba0425661af5f4d085f0109805e956d5

@rafaeloledo
Copy link

If you want, @tomasz1986, it's possible to merge this commit in the #955. The difference is that the #975 is adding --no-restart. It's changing the original text. Since i've changed other images, my image can be replaced with the one including this another flag if desired.

One thing to note is: --no-restart is being mentioned before the image, indeed.

image

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 this pull request may close these issues.

3 participants