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

Alarm server argument ordering for configuration import #1729

Closed
jacquelinegarrahan opened this issue Jan 12, 2021 · 1 comment · Fixed by #2268
Closed

Alarm server argument ordering for configuration import #1729

jacquelinegarrahan opened this issue Jan 12, 2021 · 1 comment · Fixed by #2268

Comments

@jacquelinegarrahan
Copy link
Contributor

Importing a new configuration fails if the -import argument precedes the -config argument.

Screen Shot 2021-01-12 at 1 38 30 PM

@kasemir
Copy link
Collaborator

kasemir commented Jan 12, 2021

Yes, because -import actually performs that action with the information known at the time, i.e. -config needs to set the name, and maybe -server needs to set a remote URL before then using those settings with -import.

This does match the order in which the -help info lists the options, where '-import' is mentioned quite late and so far that seems to have guided every user to success.

If you want to make the command line option handling more robust by for example having -import just set some import_requested flag and then handle that after all the options have been processed, go ahead.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants