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
add makensis #5242
add makensis #5242
Conversation
# System to create Windows installers | ||
# See: https://nsis.sourceforce.io | ||
|
||
complete -c makensis -o CMDHELP -d 'Prints out help for a specific command, or lists all commands' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So this is used as makensis -CMDHELP
, with one "-"?
makensis.fish
Outdated
complete -c makensis -o V2 -d 'Verbosity: no info' | ||
complete -c makensis -o V3 -d 'Verbosity: no script' | ||
complete -c makensis -o V4 -d 'Verbosity: all' | ||
complete -c makensis -o V4 -d 'Verbosity: all' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This line is duplicated.
complete -c makensis -o P2 -d 'Sets the compiler process priority to normal' | ||
complete -c makensis -o P3 -d 'Sets the compiler process priority to above normal' | ||
complete -c makensis -o P4 -d 'Sets the compiler process priority to high' | ||
complete -c makensis -o P5 -d 'Sets the compiler process priority to realtime' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Below you have "Verbosity: all", why not say "compiler process priority: realtime" here?
makensis.fish
Outdated
@@ -0,0 +1,26 @@ | |||
# System to create Windows installers | |||
# See: https://nsis.sourceforce.io |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This link seems broken to me?
Merged, thanks! |
Am I missing something? Why is |
@mqudsi: Terribly sorry, I did not expect that! |
No harm done :) |
* add makensis * remove duplicate line * fix typo
Description
Talk about your changes here.
Fixes issue #
TODOs: