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

Add support for bash-completion #205

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

louisroyer
Copy link
Contributor

This allows autocomplete when calling aegisub from terminal. Before this commit, when having for example file.webm and file.ass in the directory, both were proposed while only the later could be open using CLI argument.

@louisroyer louisroyer force-pushed the bash-comp-typesetting-tools branch from 27ed45f to 3d0d81d Compare December 29, 2024 17:48
This allows autocomplete when calling aegisub from terminal.
Before this commit, when having for example `file.webm` and `file.ass` in
the directory, both were proposed while only the later could be open
using CLI argument.
@louisroyer louisroyer force-pushed the bash-comp-typesetting-tools branch from 3d0d81d to eeb91e4 Compare February 3, 2025 10:31
@@ -26,3 +26,4 @@ option('update_server', type: 'string', value: 'https://aegisub-updates.redvice.
option('update_url', type: 'string', value: '/trunk', description: 'Base path to use for the update checker')

option('build_osx_bundle', type: 'boolean', value: false, description: 'Package Aegisub.app on OSX')
option('bash_completion', type: 'boolean', value: true, description: 'Install bash shell completions')
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Won't this run by default on Windows? I don't think I can merge this as-is.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't have a Windows system to test. I added a new commit to disable this for Windows.

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