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.

Sign up for GitHub

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

Jump to bottom

The hash command should be a builtin not an alias #1393

Open
krader1961 opened this issue Sep 2, 2019 · 1 comment
Open

The hash command should be a builtin not an alias #1393

krader1961 opened this issue Sep 2, 2019 · 1 comment
Assignees
@krader1961
Labels
cleanup

Comments

@krader1961
Copy link
Contributor

While working on the b_alias() function to make it the first builtin to use getopt_long() rather than optget() I noticed it has a gross hack to support the bash hash -r flag needed because hash is implemented as an alias of alias. Separate the hash logic from the alias logic so those hacks aren't needed.
@krader1961 krader1961 self-assigned this Sep 2, 2019
@krader1961
Copy link
Contributor Author

See also issue #238 where we are discussing removing the code that attempts, incorrectly and which can never be successful, to emulate bash.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@krader1961 krader1961

Labels
cleanup
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@krader1961