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

zsh/.zsh_functions 'chgext' function conflicts with default unixorn/jpb.zshplugin 'chgext' alias to 'change-extension' function #53

Closed
jmcculloch opened this issue Jan 25, 2019 · 1 comment

Comments

@jmcculloch
Copy link

jmcculloch commented Jan 25, 2019

Last login: Fri Jan 25 12:35:53 on ttys005
Identity added: /Users/jmcculloch/.ssh/id_rsa (...)
/Users/jmcculloch/.zsh_functions:161: defining function based on alias `chgext'
/Users/jmcculloch/.zsh_functions:161: parse error near `()'

This appears to be conflicting with the chgext alias defined in the plugin unixorn/jpb.zshplugin

zsh/.zgen-setup

  # Add my collection of miscellaneous utility functions.
  zgen load unixorn/jpb.zshplugin
$ grep -r 'change-extension' .
Binary file ./unixorn/jpb.zshplugin-master/.git/index matches
./unixorn/jpb.zshplugin-master/bin/change-extension:# Usage: change-extension jpeg jpg
./unixorn/jpb.zshplugin-master/jpb.plugin.zsh:#   change-extension erb haml
./unixorn/jpb.zshplugin-master/jpb.plugin.zsh:function change-extension() {
./unixorn/jpb.zshplugin-master/jpb.plugin.zsh:alias chgext='change-extension'
./unixorn/jpb.zshplugin-master/README.md:* change-extension - ?

This was resolved with the fix in issue #40 and reintroduced by commit commit 5d2ae03 (Don't use function keyword)

@unixorn
Copy link
Owner

unixorn commented Jan 26, 2019

Thanks for the fix

@unixorn unixorn closed this as completed Jan 26, 2019
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

2 participants