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

fzf: Fix shell directory for archlinux package #7119

Merged
merged 4 commits into from Sep 8, 2018
Merged

fzf: Fix shell directory for archlinux package #7119

merged 4 commits into from Sep 8, 2018

Conversation

maxbrunet
Copy link
Contributor

A small change to fix #7110, I keep it minimal to stay as close possible to the upstream install script.

I also made a change to not clutter PATH if fzf is already available.

I tested these changes on the following fzf installations:

  • MacOS + brew
  • Archlinux + Pacman
  • Fedora + RPM
  • Linux + Git

* brew has it available via symlink in /usr/local/bin
* Fedora and Arch packages have it place in /usr/bin
@mcornella mcornella merged commit bb90849 into ohmyzsh:master Sep 8, 2018
@mcornella
Copy link
Member

Thanks Maxime!

lesterchan added a commit to lesterchan/oh-my-zsh that referenced this pull request Sep 21, 2018
* upstream/master:
  man: rename file to *.plugin.zsh (ohmyzsh#6016)
  misc: remove execution permission from various files
  lib: fix history wrapper when passing numbers
  react-native: update readme with new aliases (ohmyzsh#7135)
  git-flow: add "gflfp" alias for feature publish (ohmyzsh#6350)
  react-native: add aliases for newer iPhones (ohmyzsh#7134)
  git: use color auto for ref names in git log (ohmyzsh#5729)
  git: add git rm aliases (ohmyzsh#5433)
  git: add push force aliases (ohmyzsh#6297)
  git: add gbD alias to force-delete branch (ohmyzsh#5844)
  dash: update dash bundle identifier (ohmyzsh#7127)
  git: add glols alias for glol --stat (ohmyzsh#5871)
  git: add alias to `git stash --all` (ohmyzsh#5511)
  git: add pull rebase --autostash aliases (ohmyzsh#6791)
  urltools: add readme (ohmyzsh#7126)
  osx: add function to remove .DS_Store files (ohmyzsh#7008)
  lib: quote arguments to env_default
  Fix incorrect error message when running bi without having bundler installed (ohmyzsh#7112)
  lib: small change to git_compare_version
  fzf: Fix shell directory for archlinux package (ohmyzsh#7119)
lellisls pushed a commit to lellisls/oh-my-zsh that referenced this pull request Oct 1, 2018
* fzf: Fix shell directory for archlinux package

* fzf: Don't clutter PATH if fzf already available

* brew has it available via symlink in /usr/local/bin
* Fedora and Arch packages have it place in /usr/bin

* fzf: Fix archlinux guess by using release file

* fzf: unset leftover variables
seth-cohen pushed a commit to seth-cohen/oh-my-zsh that referenced this pull request Oct 29, 2018
* fzf: Fix shell directory for archlinux package

* fzf: Don't clutter PATH if fzf already available

* brew has it available via symlink in /usr/local/bin
* Fedora and Arch packages have it place in /usr/bin

* fzf: Fix archlinux guess by using release file

* fzf: unset leftover variables
jmartindf pushed a commit to jmartindf/oh-my-zsh that referenced this pull request Nov 10, 2018
* fzf: Fix shell directory for archlinux package

* fzf: Don't clutter PATH if fzf already available

* brew has it available via symlink in /usr/local/bin
* Fedora and Arch packages have it place in /usr/bin

* fzf: Fix archlinux guess by using release file

* fzf: unset leftover variables
sagischwarz pushed a commit to sagischwarz/ohmyzsh that referenced this pull request Nov 19, 2018
* fzf: Fix shell directory for archlinux package

* fzf: Don't clutter PATH if fzf already available

* brew has it available via symlink in /usr/local/bin
* Fedora and Arch packages have it place in /usr/bin

* fzf: Fix archlinux guess by using release file

* fzf: unset leftover variables
stonetao pushed a commit to stonetao/oh-my-zsh that referenced this pull request Apr 7, 2019
* fzf: Fix shell directory for archlinux package

* fzf: Don't clutter PATH if fzf already available

* brew has it available via symlink in /usr/local/bin
* Fedora and Arch packages have it place in /usr/bin

* fzf: Fix archlinux guess by using release file

* fzf: unset leftover variables
chihchun pushed a commit to chihchun/oh-my-zsh that referenced this pull request Aug 6, 2019
* fzf: Fix shell directory for archlinux package

* fzf: Don't clutter PATH if fzf already available

* brew has it available via symlink in /usr/local/bin
* Fedora and Arch packages have it place in /usr/bin

* fzf: Fix archlinux guess by using release file

* fzf: unset leftover variables
lewg pushed a commit to lewg/oh-my-zsh that referenced this pull request Nov 25, 2019
* fzf: Fix shell directory for archlinux package

* fzf: Don't clutter PATH if fzf already available

* brew has it available via symlink in /usr/local/bin
* Fedora and Arch packages have it place in /usr/bin

* fzf: Fix archlinux guess by using release file

* fzf: unset leftover variables
kankongmeng pushed a commit to kankongmeng/oh-my-zsh that referenced this pull request Jan 8, 2020
* fzf: Fix shell directory for archlinux package

* fzf: Don't clutter PATH if fzf already available

* brew has it available via symlink in /usr/local/bin
* Fedora and Arch packages have it place in /usr/bin

* fzf: Fix archlinux guess by using release file

* fzf: unset leftover variables
spiliopoulos pushed a commit to spiliopoulos/zsh-config that referenced this pull request Jun 17, 2020
* fzf: Fix shell directory for archlinux package

* fzf: Don't clutter PATH if fzf already available

* brew has it available via symlink in /usr/local/bin
* Fedora and Arch packages have it place in /usr/bin

* fzf: Fix archlinux guess by using release file

* fzf: unset leftover variables
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.

[fzf plugin] Wrong key-bindings.zsh folder
2 participants