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

Fix missing add-zsh-hook in pygmalion.zsh-theme #3072

Merged
merged 1 commit into from
Sep 3, 2014

Conversation

clippit
Copy link
Contributor

@clippit clippit commented Sep 1, 2014

Fix "prompt_setup_pygmalion:12: command not found: add-zsh-hook" error since #3053 removes autoload -U add-zsh-hook line.

For people love the pygmalion theme:)

Fix "command not found: add-zsh-hook" since ohmyzsh#3053 removes `autoload -U add-zsh-hook`
@merwan
Copy link

merwan commented Sep 1, 2014

+1 for this pull request.

@alorence
Copy link

alorence commented Sep 1, 2014

👍 I also would like to see this pull request merged

@mcornella
Copy link
Member

Hi people, while you wait for this to get merged, you can add the autoload -U add-zsh-hook line at the top of your zshrc file to make it work.

Maybe that should be the approach here: adding the line back to
lib/termsupport.zsh. This will work with any other themes and plugins
that were using the add-zsh-hook method.

Marc
El 01/09/2014 10:43, "Antoine Lorence" notifications@github.com escribió:

[image: 👍] I also would like to see this pull request merged


Reply to this email directly or view it on GitHub
#3072 (comment)
.

@mcornella
Copy link
Member

/cc @robbyrussell

Marc
El 01/09/2014 10:52, "Marc Cornellà" marc.cornella@live.com escribió:

Hi people, while you wait for this to get merged, you can add the
autoload -U add-zsh-hook line at the top of your zshrc file to make it
work.

Maybe that should be the approach here: adding the line back to
lib/termsupport.zsh. This will work with any other themes and plugins
that were using the add-zsh-hook method.

Marc
El 01/09/2014 10:43, "Antoine Lorence" notifications@github.com
escribió:

[image: 👍] I also would like to see this pull request merged


Reply to this email directly or view it on GitHub
#3072 (comment)
.

@JeffMelton
Copy link

Thanks, @mcornella, for posting a temporary fix.

@naartjie
Copy link

naartjie commented Sep 3, 2014

Also had this problem.@mcornella thanks for the workaround.

@lslah
Copy link

lslah commented Sep 3, 2014

👍

robbyrussell added a commit that referenced this pull request Sep 3, 2014
Fix missing add-zsh-hook in pygmalion.zsh-theme
@robbyrussell robbyrussell merged commit a8ef111 into ohmyzsh:master Sep 3, 2014
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

8 participants