Load symlink custom files #1126

Merged
merged 1 commit into from Jun 22, 2012

Conversation

Projects
None yet
10 participants
Contributor

natsumesou commented May 23, 2012

(.) search only real files.
The deletion of it enables searching symlink *.zsh files

@natsumesou natsumesou Load symlink custom files
    `(.)` search only real files.
    The deletion of it enables searching symlink *.zsh files
61c39b5
Contributor

natsumesou commented May 23, 2012

relational commit
51c55ad

kreeger commented May 23, 2012

Gotta add my +1, here, as the latest update of Oh My Zsh has borked the loading of my custom, symlinked-in .zsh scripts.

forest commented May 23, 2012

+1

Contributor

simeonwillbanks commented May 23, 2012

+1 My symlinked dotfiles are no longer sourced!

$ l /Users/simeon/.oh-my-zsh/custom/lessonplanet.zsh
lrwxr-xr-x 1 simeon staff 69 Mar 16 10:33 /Users/simeon/.oh-my-zsh/custom/lessonplanet.zsh -> /Users/simeon/Repositories/dotfiles/oh-my-zsh/custom/lessonplanet.zsh

$ l /Users/simeon/.oh-my-zsh/custom/simeon.zsh
lrwxr-xr-x 1 simeon staff 63 Mar 16 10:33 /Users/simeon/.oh-my-zsh/custom/simeon.zsh -> /Users/simeon/Repositories/dotfiles/oh-my-zsh/custom/simeon.zsh

BEFORE 51c55ad
$ for config_file ($ZSH_CUSTOM/*.zsh) echo $config_file
/Users/simeon/.oh-my-zsh/custom/example.zsh
/Users/simeon/.oh-my-zsh/custom/lessonplanet.zsh
/Users/simeon/.oh-my-zsh/custom/simeon.zsh

AFTER 51c55ad 👎
$ for config_file ($ZSH_CUSTOM/*.zsh(.N)) echo $config_file
/Users/simeon/.oh-my-zsh/custom/example.zsh

PULL REQUEST 61c39b5 👏
$ $ for config_file ($ZSH_CUSTOM/*.zsh(N)) echo $config_file
/Users/simeon/.oh-my-zsh/custom/example.zsh
/Users/simeon/.oh-my-zsh/custom/lessonplanet.zsh
/Users/simeon/.oh-my-zsh/custom/simeon.zsh

Here is a temporary fix for everyone: simeonwillbanks/dotfiles@96831b4

+1

I'm having the same issue with symlinked custom files and this fixed it.

This affects me as well.

Btw, epic @simeonwillbanks

Contributor

simeonwillbanks commented Jun 12, 2012

jgoulah commented Jun 12, 2012

+1

ayn commented Jun 13, 2012

+1, thanks!

Owner

robbyrussell commented Jun 22, 2012

Great idea, merging this in. :-)

robbyrussell merged commit 493238e into robbyrussell:master Jun 22, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment