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

'gap' is a program #3682

Closed
ChrisJefferson opened this issue Mar 11, 2015 · 18 comments
Closed

'gap' is a program #3682

ChrisJefferson opened this issue Mar 11, 2015 · 18 comments

Comments

@ChrisJefferson
Copy link

Since installing oh-my-zsh, I kept finding that running 'gap' would start some weird git related thing.

I have now discovered this is a git alias added by the git package. Perhaps if there is a program called 'gap', that should be used instead of the alias? Or choose a different one?

@apjanke
Copy link
Contributor

apjanke commented Mar 11, 2015

Sounds like you're referring to gap from GAP (https://github.com/gap-system/gap)?

@stibbons and @ncanceill are the maintainers of the git plugin. Can you folks comment?

@ChrisJefferson, as a workaround, you can unalias gap after sourcing $ZSH/oh-my-zsh.sh in your ~/.zshrc.

@ChrisJefferson
Copy link
Author

Yes, it is that gap. I didn't know about unalias (probably should have done), I can at least use that for now to get my machine working again. Don't know if there will always be an argument over these short names.

@ncanceill
Copy link
Contributor

Hi @ChrisJefferson,

I had never heard of the GAP system, and I did not know about a GNU program called gap. This is obviously a conflict, and there are two choices when it comes to this:

  • changing the gap alias to something else
  • not doing anything

We do not like modifying aliases, because it may break backwards-compatibility for existing users of the plugin. This means we need a really good reason for doing so: proof that this conflict affects many users.

This is where you come in. Since you want this problem to be solved, your task is now to find other people like you: people using both GAP and the Git plugin. When you find them, get them to +1 this issue, and we will consider changing the gap alias to something else.

If you choose this option, please keep this issue open until you find enough of them or give up.


In case you do not complete the above task, the alias will definitely not be modified. This would leave you with a few options:

  • use unalias in your ~/.zshrc as suggested above
  • use another alias for your gap command, for instance:
alias the_real_gap="/path/to/the/real/gap"
  • maintain your own fork of OMZ and edit $ZSH/plugins/git/git.plugin.zsh to your needs

If you choose any of these options, please close this issue.


Thank you @apjanke for the ping. Next time @ChrisJefferson please notify the maintainers of the affected plugins.

@ChrisJefferson
Copy link
Author

I'll just stop using the git plugin.

@ncanceill
Copy link
Contributor

OK, this was not in the options I listed, but suit yourself.

Sorry but for the sake of compatibility we cannot easily change plugins aliases. Any other suggestion is welcome.

@gsemet
Copy link
Contributor

gsemet commented Mar 11, 2015

gap is (fortunately) not defined in git2 plugin (#2494)

Collision is indeed inevitable, and I think it should be better to rename conflicting git aliases.

Maybe a good solution would be to split the plugin into several one, so anyone can enable only the one that he wants (git-gst.plugin.zsh for git status aliases, git-co.plugin.zsh for git commit aliases, ...)

@gsemet gsemet mentioned this issue Mar 11, 2015
5 tasks
@ChrisJefferson
Copy link
Author

@ncanceill : I'm not saying there is an easy fix for you, but I don't really fancy either e-mailing the gap mailing list to see if anyone else uses oh-my-zsh's git plugin (I imagine like me, they won't, as it breaks gap), and I only became a user of oh-my-zsh 2 days ago, beginning to keep a separate version sounds painful!

One thing which might be worth considering (I don't know if it is) would be to produce a more useful error message and fix. Perhaps oh-my-zsh could print out aliases which conflict with programs (although I suppose sometimes this is on purpose, as it is replacing them), and give advice on how to fix the problem. At first I assumed the problem was something like auto-correction.

@ncanceill
Copy link
Contributor

Thank you for your input, it only helps us make OMZ better.

I like your idea of checking for conflicts. This could be done by defining a custom safe_alias function, based on alias. By checking the return value of the which command, we can detect if an alias is overriding an existing command, and print a warning.

Would anyone be willing to work on that?

@gsemet
Copy link
Contributor

gsemet commented Mar 12, 2015

Hard to do, because it depends whether gap or any other software is installed on your system...

@ChrisJefferson
Copy link
Author

How about just see if 'which gap' returns anything?

@gsemet
Copy link
Contributor

gsemet commented Mar 13, 2015

it returns none on my system. But on your it will return something. The "which" test is not good enough

@ChrisJefferson
Copy link
Author

I'm confused now about what we are talking about :)

A safe_alias function would (I hope) find that assigning gap on your computer was fine, and on my machine would produce a warning, hopefully giving me some useful advice on how to either enable the alias without warning, or disable it.

Of course, finding what commands are valid on anyone's computer anywhere is probably an unreasonable goal.

@blueyed
Copy link
Contributor

blueyed commented Jun 10, 2015

As @ChrisJefferson suggested, you could use something like:

if [[ -z $commands[gap] ]]; then
    alias gap="git add -p"
fi

@blueyed
Copy link
Contributor

blueyed commented Jun 10, 2015

@ncanceill
There's no need for a safe_alias wrapper I think.
Do you think this can make it into the fixed git plugin?

@blueyed
Copy link
Contributor

blueyed commented Jun 10, 2015

OTOH a safe_alias wrapper could then be used instead of alias everywhere of course, so it's worth adding to the framework probably.

See #3990 (comment).

@ncanceill
Copy link
Contributor

Yes, I am definitely in favor of a safe aliasing system, to use everywhere throughout OMZ.

Will work on it as soon as the git plugin crisis is solved.

ncanceill added a commit to ncanceill/oh-my-zsh that referenced this issue Jun 12, 2015
gaa was brought back by popular demand — see ohmyzsh#3535
gap was replaced with gapa — see ohmyzsh#3682
gdc was replaced with gdca — see ohmyzsh#3977
steshaw pushed a commit to steshaw/oh-my-zsh that referenced this issue Jun 13, 2015
gaa was brought back by popular demand — see ohmyzsh#3535
gap was replaced with gapa — see ohmyzsh#3682
gdc was replaced with gdca — see ohmyzsh#3977
freecastle added a commit to freecastle/oh-my-zsh that referenced this issue Jun 19, 2015
* 'master' of https://github.com/robbyrussell/oh-my-zsh: (96 commits)
  git plugin: fix ggl to avoid empty string see ohmyzsh#2790 (comment)
  Fix plugin/git ggpull and ggpush backward compatibility
  Add missing space causing parse error.
  Fixed odd chars on mac for agnoster theme
  Add mix-fast plugin (similar to rake-fast)
  Update _docker
  Adding quotes to git completion discovery path
  A paver (Python build and configuration utility) plugin
  Revert "Added migration notification for rails plugin"
  Added github in web_search plugin.
  last-working-dir working with spaces in dirnames
  use local options to set no_equals before sourcing bash completion script
  git plugin: new/changed aliases gaa was brought back by popular demand — see ohmyzsh#3535 gap was replaced with gapa — see ohmyzsh#3682 gdc was replaced with gdca — see ohmyzsh#3977
  Add reference to bundler plugin issue in README
  Add `leaves` command to `brew` completion
  remove trash command from osx plugin
  Added README.md file for ForkLift plugin
  Added README.md file for atom plugin
  Added README.md file
  Fix README.md for sst command
  ...
zoli added a commit to zoli/oh-my-zsh that referenced this issue Jun 22, 2015
* 'master' of github.com:robbyrussell/oh-my-zsh: (99 commits)
  git plugin: fix ggl to avoid empty string see ohmyzsh#2790 (comment)
  Fix plugin/git ggpull and ggpush backward compatibility
  Add missing space causing parse error.
  Fixed odd chars on mac for agnoster theme
  Add mix-fast plugin (similar to rake-fast)
  Update _docker
  Adding quotes to git completion discovery path
  A paver (Python build and configuration utility) plugin
  Revert "Added migration notification for rails plugin"
  Added github in web_search plugin.
  last-working-dir working with spaces in dirnames
  use local options to set no_equals before sourcing bash completion script
  git plugin: new/changed aliases gaa was brought back by popular demand — see ohmyzsh#3535 gap was replaced with gapa — see ohmyzsh#3682 gdc was replaced with gdca — see ohmyzsh#3977
  Add reference to bundler plugin issue in README
  Add `leaves` command to `brew` completion
  remove trash command from osx plugin
  Added README.md file for ForkLift plugin
  Added README.md file for atom plugin
  Added README.md file
  Fix README.md for sst command
  ...
fakemelvynkim added a commit to fakemelvynkim/oh-my-zsh that referenced this issue Jun 26, 2015
- Trim git.plugin

* 'master' of https://github.com/robbyrussell/oh-my-zsh: (107 commits)
  git plugin: fix ggl to avoid empty string see ohmyzsh#2790 (comment)
  Fix plugin/git ggpull and ggpush backward compatibility
  Add missing space causing parse error.
  Fixed odd chars on mac for agnoster theme
  Add mix-fast plugin (similar to rake-fast)
  Update _docker
  Adding quotes to git completion discovery path
  A paver (Python build and configuration utility) plugin
  Revert "Added migration notification for rails plugin"
  Added github in web_search plugin.
  last-working-dir working with spaces in dirnames
  use local options to set no_equals before sourcing bash completion script
  git plugin: new/changed aliases gaa was brought back by popular demand — see ohmyzsh#3535 gap was replaced with gapa — see ohmyzsh#3682 gdc was replaced with gdca — see ohmyzsh#3977
  Add reference to bundler plugin issue in README
  Add `leaves` command to `brew` completion
  remove trash command from osx plugin
  Added README.md file for ForkLift plugin
  Added README.md file for atom plugin
  Added README.md file
  Fix README.md for sst command
  ...

Conflicts:
	plugins/git/git.plugin.zsh
	themes/avit.zsh-theme
Spirotot pushed a commit to Spirotot/oh-my-zsh that referenced this issue Jun 27, 2015
gaa was brought back by popular demand — see ohmyzsh#3535
gap was replaced with gapa — see ohmyzsh#3682
gdc was replaced with gdca — see ohmyzsh#3977
erwinvaneijk pushed a commit to erwinvaneijk/oh-my-zsh that referenced this issue Jul 7, 2015
gaa was brought back by popular demand — see ohmyzsh#3535
gap was replaced with gapa — see ohmyzsh#3682
gdc was replaced with gdca — see ohmyzsh#3977
rsrchboy added a commit to rsrchboy/oh-my-zsh that referenced this issue Jul 14, 2015
* remotes/upstream/master: (89 commits)
  virtualenvwrapper: Replace "realpath" with portable zsh-native `:A` and `:h` modifiers
  Plugin now uses completion script from docker-compose repo.
  git plugin: fix ggl to avoid empty string see ohmyzsh#2790 (comment)
  Fix plugin/git ggpull and ggpush backward compatibility
  Fix merge conflict for jira-prefix and jira add comment
  Add missing space causing parse error.
  Fixed odd chars on mac for agnoster theme
  Add mix-fast plugin (similar to rake-fast)
  Update _docker
  Adding quotes to git completion discovery path
  A paver (Python build and configuration utility) plugin
  Revert "Added migration notification for rails plugin"
  Added github in web_search plugin.
  last-working-dir working with spaces in dirnames
  use local options to set no_equals before sourcing bash completion script
  git plugin: new/changed aliases gaa was brought back by popular demand — see ohmyzsh#3535 gap was replaced with gapa — see ohmyzsh#3682 gdc was replaced with gdca — see ohmyzsh#3977
  Add reference to bundler plugin issue in README
  Add `leaves` command to `brew` completion
  remove trash command from osx plugin
  Added README.md file for ForkLift plugin
  ...
sprig added a commit to sprig/oh-my-zsh that referenced this issue Jul 15, 2015
* 'master' of https://github.com/sprig/oh-my-zsh: (536 commits)
  virtualenvwrapper: Replace "realpath" with portable zsh-native `:A` and `:h` modifiers
  Plugin now uses completion script from docker-compose repo.
  git plugin: fix ggl to avoid empty string see ohmyzsh#2790 (comment)
  Fix plugin/git ggpull and ggpush backward compatibility
  Fix merge conflict for jira-prefix and jira add comment
  Add missing space causing parse error.
  Fixed odd chars on mac for agnoster theme
  Add mix-fast plugin (similar to rake-fast)
  Update _docker
  Adding quotes to git completion discovery path
  A paver (Python build and configuration utility) plugin
  Revert "Added migration notification for rails plugin"
  Added github in web_search plugin.
  last-working-dir working with spaces in dirnames
  use local options to set no_equals before sourcing bash completion script
  git plugin: new/changed aliases gaa was brought back by popular demand — see ohmyzsh#3535 gap was replaced with gapa — see ohmyzsh#3682 gdc was replaced with gdca — see ohmyzsh#3977
  Add reference to bundler plugin issue in README
  Add `leaves` command to `brew` completion
  remove trash command from osx plugin
  Added README.md file for ForkLift plugin
  ...

Conflicts:
	.zshrc
lesterchan added a commit to lesterchan/oh-my-zsh that referenced this issue Jul 19, 2015
* upstream/master: (102 commits)
  virtualenvwrapper: Replace "realpath" with portable zsh-native `:A` and `:h` modifiers
  Plugin now uses completion script from docker-compose repo.
  git plugin: fix ggl to avoid empty string see ohmyzsh#2790 (comment)
  Fix plugin/git ggpull and ggpush backward compatibility
  Fix merge conflict for jira-prefix and jira add comment
  Add missing space causing parse error.
  Fixed odd chars on mac for agnoster theme
  Add mix-fast plugin (similar to rake-fast)
  Update _docker
  Adding quotes to git completion discovery path
  A paver (Python build and configuration utility) plugin
  Revert "Added migration notification for rails plugin"
  Added github in web_search plugin.
  last-working-dir working with spaces in dirnames
  use local options to set no_equals before sourcing bash completion script
  git plugin: new/changed aliases gaa was brought back by popular demand — see ohmyzsh#3535 gap was replaced with gapa — see ohmyzsh#3682 gdc was replaced with gdca — see ohmyzsh#3977
  Add reference to bundler plugin issue in README
  Add `leaves` command to `brew` completion
  remove trash command from osx plugin
  Added README.md file for ForkLift plugin
  ...
foobacca added a commit to foobacca/oh-my-zsh that referenced this issue Jul 21, 2015
* origin/master: (89 commits)
  virtualenvwrapper: Replace "realpath" with portable zsh-native `:A` and `:h` modifiers
  Plugin now uses completion script from docker-compose repo.
  git plugin: fix ggl to avoid empty string see ohmyzsh#2790 (comment)
  Fix plugin/git ggpull and ggpush backward compatibility
  Fix merge conflict for jira-prefix and jira add comment
  Add missing space causing parse error.
  Fixed odd chars on mac for agnoster theme
  Add mix-fast plugin (similar to rake-fast)
  Update _docker
  Adding quotes to git completion discovery path
  A paver (Python build and configuration utility) plugin
  Revert "Added migration notification for rails plugin"
  Added github in web_search plugin.
  last-working-dir working with spaces in dirnames
  use local options to set no_equals before sourcing bash completion script
  git plugin: new/changed aliases gaa was brought back by popular demand — see ohmyzsh#3535 gap was replaced with gapa — see ohmyzsh#3682 gdc was replaced with gdca — see ohmyzsh#3977
  Add reference to bundler plugin issue in README
  Add `leaves` command to `brew` completion
  remove trash command from osx plugin
  Added README.md file for ForkLift plugin
  ...
tedbundyjr pushed a commit to tedbundyjr/oh-my-zsh that referenced this issue Jul 21, 2015
# By ncanceill (8) and others
# Via Robby Russell (47) and Andrew Janke (1)
* 'master' of https://github.com/robbyrussell/oh-my-zsh: (110 commits)
  virtualenvwrapper: Replace "realpath" with portable zsh-native `:A` and `:h` modifiers
  Plugin now uses completion script from docker-compose repo.
  git plugin: fix ggl to avoid empty string see ohmyzsh#2790 (comment)
  Fix plugin/git ggpull and ggpush backward compatibility
  Fix merge conflict for jira-prefix and jira add comment
  Add missing space causing parse error.
  Fixed odd chars on mac for agnoster theme
  Add mix-fast plugin (similar to rake-fast)
  Update _docker
  Adding quotes to git completion discovery path
  A paver (Python build and configuration utility) plugin
  Revert "Added migration notification for rails plugin"
  Added github in web_search plugin.
  last-working-dir working with spaces in dirnames
  use local options to set no_equals before sourcing bash completion script
  git plugin: new/changed aliases gaa was brought back by popular demand — see ohmyzsh#3535 gap was replaced with gapa — see ohmyzsh#3682 gdc was replaced with gdca — see ohmyzsh#3977
  Add reference to bundler plugin issue in README
  Add `leaves` command to `brew` completion
  remove trash command from osx plugin
  Added README.md file for ForkLift plugin
  ...
neiljdo added a commit to neiljdo/oh-my-zsh that referenced this issue Jul 23, 2015
* 'master' of https://github.com/robbyrussell/oh-my-zsh: (106 commits)
  virtualenvwrapper: Replace "realpath" with portable zsh-native `:A` and `:h` modifiers
  Plugin now uses completion script from docker-compose repo.
  git plugin: fix ggl to avoid empty string see ohmyzsh#2790 (comment)
  Fix plugin/git ggpull and ggpush backward compatibility
  Fix merge conflict for jira-prefix and jira add comment
  Add missing space causing parse error.
  Fixed odd chars on mac for agnoster theme
  Add mix-fast plugin (similar to rake-fast)
  Update _docker
  Adding quotes to git completion discovery path
  A paver (Python build and configuration utility) plugin
  Revert "Added migration notification for rails plugin"
  Added github in web_search plugin.
  last-working-dir working with spaces in dirnames
  use local options to set no_equals before sourcing bash completion script
  git plugin: new/changed aliases gaa was brought back by popular demand — see ohmyzsh#3535 gap was replaced with gapa — see ohmyzsh#3682 gdc was replaced with gdca — see ohmyzsh#3977
  Add reference to bundler plugin issue in README
  Add `leaves` command to `brew` completion
  remove trash command from osx plugin
  Added README.md file for ForkLift plugin
  ...
torbjoernk added a commit to torbjoernk/oh-my-zsh that referenced this issue Jul 25, 2015
* master:
  virtualenvwrapper: Replace "realpath" with portable zsh-native `:A` and `:h` modifiers
  Plugin now uses completion script from docker-compose repo.
  git plugin: fix ggl to avoid empty string see ohmyzsh#2790 (comment)
  Fix plugin/git ggpull and ggpush backward compatibility
  Fix merge conflict for jira-prefix and jira add comment
  Add missing space causing parse error.
  Fixed odd chars on mac for agnoster theme
  Add mix-fast plugin (similar to rake-fast)
  Update _docker
  Adding quotes to git completion discovery path
  A paver (Python build and configuration utility) plugin
  Revert "Added migration notification for rails plugin"
  Added github in web_search plugin.
  last-working-dir working with spaces in dirnames
  use local options to set no_equals before sourcing bash completion script
  git plugin: new/changed aliases gaa was brought back by popular demand — see ohmyzsh#3535 gap was replaced with gapa — see ohmyzsh#3682 gdc was replaced with gdca — see ohmyzsh#3977
  Add reference to bundler plugin issue in README
  Add `leaves` command to `brew` completion
  remove trash command from osx plugin
  Added README.md file for ForkLift plugin
  Added README.md file for atom plugin
  Added README.md file
  Fix README.md for sst command
  Plugins: vi-mode: Add first README file
  Plugins: archlinux - add basic README.
  Plugins: archlinux - use command -v in favor of which
  Plugins: apache2-macports - add README
  Added references to the `external themes` wiki page.
  git plugin: fix ggpnp when called without argument also allow multiple arguments for ggp and ggl aliases
  * Added header with help and information * Moved git information to left prompt, with   support for:     * +ZSH_THEME_GIT_PROMPT_UNTRACKED, MODIFIED,     STASHED, ADDED, UPSTREAM STATUS     (if applicable) * Removed right prompt * Refactored following Google's guidelines for   scripts [ http://goo.gl/oJSXH2 ]     * Using local variables where applicable
  Update Mix complition upto 1.0.4
  Update aliases in laravel5.plugin.zsh file
  Update taskwarrior.plugin.zsh
  Renamed 'ss' alias to 'sstat' (to avoid collision with /bin/ss).
  Fixed typo in README
  Update bgnotify.plugin.zsh
  fix typo
  adb: add more commands
  Update README.markdown
  Fixed jump pluging to display the right mark name on 'mark . '
  Added README for mercurial plugin
  Add git to prerequisites
  add help link to install zsh
  sublime: Make subl a function so it can be called from functions in other modules
  Added support for detecting Sublime Text 3 on Arch Linux
  Update history-substring-search README
  Added the tugboat autocompletion plugin
  git plugin: make b local in gg* functions because it leaked, and led to ohmyzsh#3991
  git plugin: fix error msg in gwip alias because git rm will fail when no files were deleted
  git plugin: global var for git command due to issues like ohmyzsh#3962 until a proper plugin-loading system is implemented
  git plugin: bring back olg gg aliases by popular demand, see ohmyzsh#3972
  More run and create options
  Only load url-quote-magic if it is available. Partially fixes ohmyzsh#3614
  Disable globbing when executing `zeus rake` aliases to enable use of square brackets without escaping
  Add git_prompt_behind and git_prompt_exists
  add kitchen completion plugin from @petere
  Added a possibility to display the remote branch and the number of commits you are ahead or behind
  Fix homebrew directory in aws plugin.
  Recognize comments by default.
  Move custon *.zsh file sourcing up so that they can load plugins and other things that come after.
  added comment functionality
  Improved the `_git_time_since_commit` function.
  Remove docker insert command
  Add README for httpie plugin
  Fix typo in $VCS_CLEAN_COLOR in adben theme
  Grep bug in Agnoster
  generate with 'pod --completion-script' /cc thx to @ajmccall
  updated the command list to the latest cocoapod version (0.33.0)
  No input for Update Check defaults to Yes
  Add rdrs method to reset the database(s)
  Set AWS_PROFILE and use existing RPROMPT
  Add new boot2docker options
  Add installation instructions to README
  Round 3 of batch-closing obsolete issues
  Respect $ZSH_CACHE_DIR when creating fasd's cache
  Create new aliases for homebrew plugin
  Add sublime text 3 paths to check list
  Don't leak extendedglob to global environment.
  Detect the .venv in any parent directory of the current directory
  Add CakePHP3 plugin
  Differing upgrade and safe-upgrade
  Many updates to git plugin (see ohmyzsh#2790)
  Use $EDITOR
  Fix 'chsh' by checking '/etc/shells' instead of using 'which'
  Add alias of bundle clean
  Added aliases for docker compose
  add markdown extension to README
  Add readme and add new alias cfpc: paste+compile+copy
  Update template zshrc with HYPHEN_INSENSITIVE
  typofix
  Add coffeescript aliases: cf, cfc, cfp
  Add hyphen insensitivity
  Fixing some issues where show repeated weird values, and fixed commands that need the container autocomplete without being running, like start, now it autocompletes showing all the containers so you can start without knowing the right name
  Fix minimal.zsh-theme's check for in_svn and add support for mercurial
  Add autocomplete for frontend-search
  Fixes gradle autocomplete. Regex did not include subproject tasks before.
  fix stackoverflow url in search
  Use the newest Vundle commands
  Add stackoverflow link
  Fixed Typo.
  Use camelCase for npm aliases
  Fix echo nothing if $SHORT_HOST doesn't exist.
  Add .ipsw to the extract plugin.
  Minor cosmetic fix to minimal theme
  [boot2docker plugin] Initial version
  fix paclist() for non-english locales
  gitfast: back-port prompt fix
  gitfast: update to upstream v1.9.2
sparrc pushed a commit to sparrc/oh-my-zsh that referenced this issue Jul 27, 2015
gaa was brought back by popular demand — see ohmyzsh#3535
gap was replaced with gapa — see ohmyzsh#3682
gdc was replaced with gdca — see ohmyzsh#3977
DerToti pushed a commit to DerToti/oh-my-zsh that referenced this issue Jul 29, 2015
gaa was brought back by popular demand — see ohmyzsh#3535
gap was replaced with gapa — see ohmyzsh#3682
gdc was replaced with gdca — see ohmyzsh#3977
samwize added a commit to samwize/oh-my-zsh that referenced this issue Aug 4, 2015
* commit '3d5b3430fd9da12f6e33f534e487fe9b18928ac7': (111 commits)
  virtualenvwrapper: treat git repos as project roots again (instead of requiring a .venv customization directory) Change error output to more conventional OMZ format, so it's clear the plugin is for oh-my-zsh and not base zsh. Use `local` variables instead of manual unsetting.
  virtualenvwrapper: Replace "realpath" with portable zsh-native `:A` and `:h` modifiers
  Plugin now uses completion script from docker-compose repo.
  git plugin: fix ggl to avoid empty string see ohmyzsh#2790 (comment)
  Fix plugin/git ggpull and ggpush backward compatibility
  Fix merge conflict for jira-prefix and jira add comment
  Add missing space causing parse error.
  Fixed odd chars on mac for agnoster theme
  Add mix-fast plugin (similar to rake-fast)
  Update _docker
  Adding quotes to git completion discovery path
  A paver (Python build and configuration utility) plugin
  Revert "Added migration notification for rails plugin"
  Added github in web_search plugin.
  last-working-dir working with spaces in dirnames
  use local options to set no_equals before sourcing bash completion script
  git plugin: new/changed aliases gaa was brought back by popular demand — see ohmyzsh#3535 gap was replaced with gapa — see ohmyzsh#3682 gdc was replaced with gdca — see ohmyzsh#3977
  Add reference to bundler plugin issue in README
  Add `leaves` command to `brew` completion
  remove trash command from osx plugin
  ...
kirb pushed a commit to kirb/oh-my-zsh that referenced this issue Aug 23, 2015
gaa was brought back by popular demand — see ohmyzsh#3535
gap was replaced with gapa — see ohmyzsh#3682
gdc was replaced with gdca — see ohmyzsh#3977
ChristianLohmann pushed a commit to ChristianLohmann/oh-my-zsh that referenced this issue Aug 31, 2015
gaa was brought back by popular demand — see ohmyzsh#3535
gap was replaced with gapa — see ohmyzsh#3682
gdc was replaced with gdca — see ohmyzsh#3977
inutano pushed a commit to inutano/oh-my-zsh that referenced this issue Oct 19, 2015
gaa was brought back by popular demand — see ohmyzsh#3535
gap was replaced with gapa — see ohmyzsh#3682
gdc was replaced with gdca — see ohmyzsh#3977
@unphased
Copy link

unphased commented Mar 3, 2016

If I'm understanding things correctly:

  1. gap got taken out for gapa because OP complained that gap is some tool
  2. when lib: add safe_alias function #4585 is completed, OP's concern can be dealt with while retaining the gap alias
  3. with the completion of lib: add safe_alias function #4585 gapa will go away again, hopefully...

Meanwhile I manually add gap as a zsh alias in my own zsh config which rides on top of omz.

tmitchell pushed a commit to tmitchell/oh-my-zsh that referenced this issue Mar 31, 2016
gaa was brought back by popular demand — see ohmyzsh#3535
gap was replaced with gapa — see ohmyzsh#3682
gdc was replaced with gdca — see ohmyzsh#3977
@ncanceill
Copy link
Contributor

@unphased

Could not have put it better myself. Although we may go with the Micro$oft approach and keep gapa around too... retro-compatibility yay.

NobbZ pushed a commit to NobbZ/oh-my-zsh that referenced this issue May 15, 2016
gaa was brought back by popular demand — see ohmyzsh#3535
gap was replaced with gapa — see ohmyzsh#3682
gdc was replaced with gdca — see ohmyzsh#3977
DennisDenuto pushed a commit to DennisDenuto/oh-my-zsh that referenced this issue Oct 29, 2016
gaa was brought back by popular demand — see ohmyzsh#3535
gap was replaced with gapa — see ohmyzsh#3682
gdc was replaced with gdca — see ohmyzsh#3977
fforw pushed a commit to fforw/oh-my-zsh that referenced this issue Feb 21, 2017
gaa was brought back by popular demand — see ohmyzsh#3535
gap was replaced with gapa — see ohmyzsh#3682
gdc was replaced with gdca — see ohmyzsh#3977
Luxcium added a commit to LuxciumProject/ahmyzsh that referenced this issue Sep 20, 2020
https://github.com/ohmyzsh/ohmyzsh/blob/93c837fec8e9fe61509b9dff9e909e84f7ebe32d/plugins/git/git.plugin.zsh#L1-L293

Copyright (c) 2009-2020 Robby Russell and
[contributors](https://github.com/ohmyzsh/ohmyzsh/contributors)

TAnks to those who have contributed to fix those 20+ issues:

ohmyzsh/ohmyzsh#1790
ohmyzsh/ohmyzsh#3535
ohmyzsh/ohmyzsh#3682
ohmyzsh/ohmyzsh#3962
ohmyzsh/ohmyzsh#3972
ohmyzsh/ohmyzsh#3977
ohmyzsh/ohmyzsh#3991
ohmyzsh/ohmyzsh#4085
ohmyzsh/ohmyzsh#4727
ohmyzsh/ohmyzsh#5130
ohmyzsh/ohmyzsh#5179
ohmyzsh/ohmyzsh#5362
ohmyzsh/ohmyzsh#5442
ohmyzsh/ohmyzsh#6329
ohmyzsh/ohmyzsh#6793
ohmyzsh/ohmyzsh#7682
ohmyzsh/ohmyzsh#7716
ohmyzsh/ohmyzsh#7754
ohmyzsh/ohmyzsh#867
ohmyzsh/ohmyzsh#9103
ohmyzsh/ohmyzsh#9114

and those who help develop
theses aliases and function
whith theses Pull 40+ Requests:

ohmyzsh/ohmyzsh#2790
ohmyzsh/ohmyzsh#3167
ohmyzsh/ohmyzsh#4304
ohmyzsh/ohmyzsh#4751
ohmyzsh/ohmyzsh#4830
ohmyzsh/ohmyzsh#4993
ohmyzsh/ohmyzsh#5025
ohmyzsh/ohmyzsh#5178
ohmyzsh/ohmyzsh#5271
ohmyzsh/ohmyzsh#5287
ohmyzsh/ohmyzsh#5306
ohmyzsh/ohmyzsh#5307
ohmyzsh/ohmyzsh#5390
ohmyzsh/ohmyzsh#5417
ohmyzsh/ohmyzsh#5433
ohmyzsh/ohmyzsh#5511
ohmyzsh/ohmyzsh#5591
ohmyzsh/ohmyzsh#5643
ohmyzsh/ohmyzsh#5729
ohmyzsh/ohmyzsh#5844
ohmyzsh/ohmyzsh#5849
ohmyzsh/ohmyzsh#5871
ohmyzsh/ohmyzsh#6006
ohmyzsh/ohmyzsh#6031
ohmyzsh/ohmyzsh#6059
ohmyzsh/ohmyzsh#6079
ohmyzsh/ohmyzsh#6297
ohmyzsh/ohmyzsh#6378
ohmyzsh/ohmyzsh#6789
ohmyzsh/ohmyzsh#6791
ohmyzsh/ohmyzsh#6893
ohmyzsh/ohmyzsh#7064
ohmyzsh/ohmyzsh#7472
ohmyzsh/ohmyzsh#7556
ohmyzsh/ohmyzsh#7629
ohmyzsh/ohmyzsh#7630
ohmyzsh/ohmyzsh#7724
ohmyzsh/ohmyzsh#7841
ohmyzsh/ohmyzsh#8089
ohmyzsh/ohmyzsh#8563
ohmyzsh/ohmyzsh#8617
ohmyzsh/ohmyzsh#8622
ohmyzsh/ohmyzsh#8697
ohmyzsh/ohmyzsh#8902
ohmyzsh/ohmyzsh#8935
ohmyzsh/ohmyzsh#8957
ohmyzsh/ohmyzsh#9049
Signed-off-by: Benjamin Vincent (Luxcium) <luxcium@neb401.com>
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

No branches or pull requests

6 participants