Tarrasch / zsh-autoenv

Autoenv for zsh
700 stars 30 forks source link

Unset global variabels #33

Closed srijanshetty closed 9 years ago

srijanshetty commented 9 years ago

_autoenv_source_dir & _autoenv_chpwd_prev_dir are globally set and they pollute my prompt when I cd into those directories

Tarrasch commented 9 years ago

LGTM.

blueyed commented 9 years ago

You are using zsh's AUTO_NAME_DIRS option, aren't you?

We should fix this by unsetting this option temporarily / locally instead when assigning to these globals probably.

srijanshetty commented 9 years ago

@blueyed : Yes. Now, I'm unsure as to disable AUTO_NAME_DIRS or to unset globals. Is there a way to eliminate the globals?

blueyed commented 9 years ago

Unsetting them is not an option, but it seems like we can just get rid of them. See #34.

I'll merge it once Travis tests pass.

blueyed commented 9 years ago

@srijanshetty Thanks for the report, and bringing it to our attention!

Having less code/globals is always nice! :)