Skip to content

don't set cwd_ in ls#20

Open
lyleunderwood wants to merge 1 commit intoebidel:masterfrom
lyleunderwood:preserve-cwd-ls
Open

don't set cwd_ in ls#20
lyleunderwood wants to merge 1 commit intoebidel:masterfrom
lyleunderwood:preserve-cwd-ls

Conversation

@lyleunderwood
Copy link

I don't really understand why cwd_ was being changed by ls, and it took me quite some time to track down the issue. Tests still seem to pass.

@mwise
Copy link

mwise commented Feb 1, 2014

+1 on this. We ran into this issue when using a single filer.js instance to read/write multiple files at once. The fix from this PR resolved our issue.

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

Successfully merging this pull request may close these issues.

2 participants