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

Make g also list when run with no arguments #21

Closed
wants to merge 2 commits into from

Conversation

crccheck
Copy link

@crccheck crccheck commented May 3, 2011

since I use l as a shortcut to ls, it's inconvenient that bashmarks
also uses l. but hey, I know a function that isn't doing anything and
is super convenient to pull double duty... when g is run without
arguments, make it list

since I use `l` as a shortcut to `ls`, it's inconvenient that bashmarks
also uses `l`. but hey, I know a function that isn't doing anything and
is super convenient to pull double duty... when `g` is run without
arguments, make it list
@pthrasher
Copy link

I'd be more in favor of changing l to something different, as I like the functionality mentioned in issue #19 better.

@crccheck
Copy link
Author

crccheck commented Sep 2, 2011

I can definitely see the benefits of making 'g' behave like 'cd'. But I also really hate the idea of unnecessary single character aliases. Making g <tab> behave like l would be nice too.

@pthrasher
Copy link

I could see g listing, and then have g by itself working as mentioned in #19. Actually, it appears it already works this way. My vote is to merge in the pull request I submitted. I think it makes multiple parties happy. You still have tab completion on g.

@crccheck
Copy link
Author

crccheck commented Sep 5, 2011

well now that I think about it, the whole idea of just making everything global 1 character functions is awkward unless you're just writing for yourself. There should a canonical bashmarks executable that acts like g, and then bashmarks -s for s and bashmarks -l for l. Then people can just write their own aliases/functions to suit their own needs in their bashrc.

@pthrasher
Copy link

At first, I want to disagree with you. But I do like this idea. Allowing people to write their own one letter (or multi letter) aliases.

@crccheck
Copy link
Author

crccheck commented Sep 5, 2011

Closing this PR. I don't think I'll implement what I wrote since my fork works for me (hooray laziness). Not sure if I should write up my last comment as an Issue since it's a big change.

@crccheck crccheck closed this Sep 5, 2011
@pthrasher
Copy link

I'll probably propose this via a feature branch... I'll make a pull request for it.

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.

None yet

2 participants