Fix newline handling in auth file

Tarrasch / zsh-autoenv

Autoenv for zsh

698 stars 30 forks source link

Fix newline handling in auth file #13

Closed blueyed closed 9 years ago

Tarrasch commented 9 years ago

EDIT: I meant this comment for the other PR ^^, brb

Tarrasch commented 9 years ago

What problem is this PR solving?

blueyed commented 9 years ago

I've noticed that my env auth file got messed up over time - the entries not being on separate lines. This might have other reasons, but I've added tests and streamlined it - although the fix then was quite tiny.

I've thought about using NUL bytes to separate entries or something similar, but stopped there. This reminds me however that at least the grep pattern could be tightened a bit more.

Tarrasch commented 9 years ago

Haha, yea, I never noticed my .env_auth* file contains empty lines. I see what this patch solves now, Feel free to press merge.

Yes sorry, I've been super lazy and still haven't had time to upgrade my repo yet. :)