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 best practice guide for using bash aliasing to make shorter commands #139

Closed
djgroen opened this issue Aug 1, 2019 · 5 comments

Comments

@djgroen
Copy link
Owner

commented Aug 1, 2019

I personally use bash aliases to shorten fabsim eagle_hidalgo to feh, which really saves on typing individual commands, and using the toolkit.

In fact I think it's so useful that it may be worth inserting that in the documentation somewhere.

I don't know what place is appropriate for that currently though, perhaps we could make a best practices in automation guide in the docs section?

@djgroen djgroen added the usability label Aug 1, 2019
@djgroen djgroen added this to the VECMA M15 release milestone Aug 1, 2019
@djgroen djgroen self-assigned this Aug 1, 2019
@arabnejad arabnejad self-assigned this Aug 5, 2019
@arabnejad

This comment has been minimized.

Copy link
Collaborator

commented Aug 5, 2019

Hi @djgroen
I made the first prototype for this functionality, github branch : bash_aliasing
usage :
fabsim <machine_name> bash_alias:<alias_name>

since that the user added /FaSim3/bin folder to the .bashrc file, so by reloading that file source ~/.bashrc or restarting the shell, the new command should be available to be used

@djgroen

This comment has been minimized.

Copy link
Owner Author

commented Aug 6, 2019

@arabnejad Very interesting. Could you add a simple example to the docs on how to use this aliasing?

@djgroen

This comment has been minimized.

Copy link
Owner Author

commented Aug 29, 2019

Renamed bash_alias to bash_machine_alias to avoid confusion. Also added a check to avoid overwriting the fabsim command.

@djgroen

This comment has been minimized.

Copy link
Owner Author

commented Aug 29, 2019

See commit 97e0a55

djgroen added a commit that referenced this issue Aug 29, 2019
@djgroen

This comment has been minimized.

Copy link
Owner Author

commented Aug 29, 2019

And I added documentation, so this is all done and dusted now.

@djgroen djgroen closed this Aug 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.