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

Cosmetic change: Omit .config file extension from -config argument #910

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

weiss
Copy link
Contributor

@weiss weiss commented Apr 4, 2022

Specify -config sys rather than -config sys.config. This is what the erl(1) manual says, it's what other release generators do, and it saves a few characters on the process command line 😄

However, as both sys and sys.config will work, this fixes a non-issue, so feel free to just close the PR if you don't like the change.

As per the erl(1) manual, the argument to the -config flag is meant to
be "sys" rather than "sys.config" (though both will work). Adjust the
start scripts accordingly.
@tsloughter
Copy link
Member

Probably not going to accept this because, ironically, it actually adds .config all over the files :). I think it will just be confusing to see that when reading the scripts, esp if it isn't immediately recognizable as removing instead of adding the prefix.

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