Improve utility-function loading

[powerlevel9k]

Improve utility-function loading

[Tritlo]

Wow, thanks. My solution is a bit of a hack, but I want my .zshrc to be as self contained as possible.

[dritter]

@Tritlo I think you got me wrong here. This is not about you, writing your dotfiles wrong. It is about how powerlevel9k loads utility functions. That does not play well, if the theme is loaded as a function. In that case we assume that the theme was loaded by prezto and seek for the utility files in a prezto-like path. This is the - probably - faulty line.

Another example is outcoldman. He loads the theme also by zgen.

Did not see that coming.. 😢

[bhilburn]

@dritter - Agreed. The issue here is the design we currently have in next where we have broken out utility functions. We need to figure out how to make this work to make workflows like @Tritlo easier.

@Tritlo - Sorry for any miscommunication, here! Would you be up for helping us test a fix to powerlevel9k to make this work better?

[Tritlo]

@dritter I see. Should I rather just use

zgen load bhilburn/powerlevel9k powerlevel9k.zsh-theme

how does this affect the functionality of powerlevel9k? Is it only the next branch?

[dritter]

@Tritlo No no. All of the different methods should work. Regardless how you load the theme and regardless if the theme was symlinked or loaded directly.

And yes, it is currently only on next branch. So the question could be answered easily: Does the next branch work for you? Or do you get weird error messages?

[Tritlo]

Yes, the branch seems to work, and no weird errors. However, the battery segment does not seem to work.

[rjorgenson]

@Tritlo What issue are you running into with the battery segment? I've had limited environments to test it on(only ubuntu 14.04 and OS X) and any insight you could provide would be very helpful to making it more portable =] Could you open a separate bug for tracking that problem? Thanks!

[dritter]

@bhilburn I think we should have this in mind even zgen seems to be clear. The hardcoded path bothers me..

[Tritlo]

@rjorgenson Apparently the battery plugin works for me (ubuntu 14.04), I just had the color setting so that the text was black on black, making it invisible :/

[rjorgenson]

Ahh, well that's good that it works =] I'll take a look at it and see if I can figure out a better way to set default colors so that doesn't happen.

[Tritlo]

@rjorgenson you could maybe just swap out "black" for "cyan". I doubt many people have cyan as their default color.

[dritter]

To stay on topic: command -V could be our friend:

$ command -V get_icon_names
get_icon_names is a shell function from /Users/dr/.oh-my-zsh/custom/themes/powerlevel9k-wip/functions/icons.zsh

That works for me in ZSH 5.1.1.

[dritter]

Some news here:
command -V is just an alias to whence -v. And whence -v prints the path to the script since ZSH 5.0.8 (link). The origin of that feature seems to lie in this message.
I haven't found another way of getting the installation path automatically. In the case that this is still not sufficient, I added a variable POWERLEVEL9K_INSTALLATION_PATH, which needs to be set to the exact location (including the file name).

I've made pull request #154 to address this.

[bhilburn]

Merged #154. Please test on next and open a new bug if you find issues.