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 Oh my Zsh correct fpath #865

Merged
merged 1 commit into from
Jul 31, 2019
Merged

Fix Oh my Zsh correct fpath #865

merged 1 commit into from
Jul 31, 2019

Conversation

grisotto
Copy link
Contributor

Every time I open the terminal show this:
/.oh-my-zsh/custom/exercism_completion.zsh:local:6: options: can't change type of autoloaded parameter
You need to put the file in any $fpath, but ~/.oh-my-zsh/custom is not one of them.

Look this: beetbox/beets#1731

Every time I open the terminal show this:
```/.oh-my-zsh/custom/exercism_completion.zsh:local:6: options: can't change type of autoloaded parameter```
You need to put the file in any `$fpath`, but `~/.oh-my-zsh/custom` is not one of them.

Look this: beetbox/beets#1731
Copy link
Member

@kytrinyx kytrinyx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! Thank you so much for taking the time to fix this ✨

@kytrinyx kytrinyx merged commit 9d1041e into exercism:master Jul 31, 2019
@NobbZ
Copy link
Member

NobbZ commented Aug 3, 2019

I'm wondering why or how this works…

I'm not using omz anymore, but when I used it, it was discouraged to put any user provided additions outside of custom, as this could interfere with the way how omz updates itself via git.

It really should be in ~/.oh-my-zsh/custom.

@kytrinyx
Copy link
Member

kytrinyx commented Aug 3, 2019

@NobbZ So you're saying we need to revert this?

Would someone please reference some official docs to get a canonical answer?

@NobbZ
Copy link
Member

NobbZ commented Aug 3, 2019

Sadly there is not much documenation about "custom"…

https://github.com/robbyrussell/oh-my-zsh#custom-plugins-and-themes

But as far as I can tell, the exercism completion is not a "plugin" in the way as omz defines its plugins. While I'm not sure what would make a plugin a plugin rather than a bare script.

But, from the sources, the fpath is set to contain the folder mentioned by the OP in the PR when looking at the code:

https://github.com/robbyrussell/oh-my-zsh/blob/master/oh-my-zsh.sh#L20

Therefore, I conclude that omz lacks documentation about those folders and should also properly advertise them as they do with custom while ignoring their contents in gitignore.

@grisotto would you mind opening a ticket there and linking back?

@kytrinyx
Copy link
Member

kytrinyx commented Aug 3, 2019

Therefore, I conclude that omz lacks documentation about those folders and should also properly advertise them as they do with custom while ignoring their contents in gitignore.

That would be really helpful!

@grisotto would you mind opening a ticket there and linking back?

❤️ This would be a great help :-)

@grisotto
Copy link
Contributor Author

grisotto commented Aug 9, 2019

Hi, I think you can revert this PR.
I don't want open an issue in oh-my-zsh. There are a lot of issues and I think this was asked before.

@kytrinyx
Copy link
Member

Ok, I'll do that. Thanks.

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.

3 participants