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

Fix start command invocation #372

Merged
merged 1 commit into from
Feb 1, 2022
Merged

Fix start command invocation #372

merged 1 commit into from
Feb 1, 2022

Conversation

pwoolcoc
Copy link
Contributor

@pwoolcoc pwoolcoc commented Feb 1, 2022

When I tried to start the server using the command ./gotosocial server start --config-file ./config.yaml, I got the following error:

root@gotosocial-experiment:/gotosocial# ./gotosocial server start --config-file ./config.yaml
Incorrect Usage: flag provided but not defined: -config-file

NAME:
   gotosocial server start - start the gotosocial server

USAGE:
   gotosocial server start [command options] [arguments...]

OPTIONS:
   --help, -h  show help (default: false)

FATA[0000] flag provided but not defined: -config-file

Putting the flag after the ./gotosocial but before the server start seems to work properly

When I tried to start the server using the command `./gotosocial server start --config-file ./config.yaml`, I got the following error:

```
root@gotosocial-experiment:/gotosocial# ./gotosocial server start --config-file ./config.yaml
Incorrect Usage: flag provided but not defined: -config-file

NAME:
   gotosocial server start - start the gotosocial server

USAGE:
   gotosocial server start [command options] [arguments...]

OPTIONS:
   --help, -h  show help (default: false)

FATA[0000] flag provided but not defined: -config-file
```

Putting the flag after the `./gotosocial` but before the `server start` seems to work properly
@tsmethurst
Copy link
Contributor

The docs are ahead of the release unfortunately: on main, we already redid a bunch of the CLI stuff to make it so you can put the flags in practically any order, so in the next release the docs will be correct. But i'll pull this in anyway, because for the current release it's indeed bum that the flags are incorrectly documented.

Thanks for the PR!

@tsmethurst tsmethurst merged commit aa34ff1 into superseriousbusiness:main Feb 1, 2022
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.

2 participants