Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

default-directory for process functions are unpredictable #4

Closed
tkf opened this issue Feb 10, 2013 · 2 comments · Fixed by #5
Closed

default-directory for process functions are unpredictable #4

tkf opened this issue Feb 10, 2013 · 2 comments · Fixed by #5

Comments

@tkf
Copy link
Collaborator

tkf commented Feb 10, 2013

(let ((default-directory "/"))
  (deferred:nextc (deferred:process "pwd") 'message)
  (let ((default-directory "~/"))
    (dotimes (_ 3)
      (sleep-for 0.1))))  ; prints $HOME ("/" is expected).

default-directory はサブプロセスの $PWD を決めるのに使われますが、 deferred の process 系関数だと、それを決める方法が無いようです。

@tkf
Copy link
Collaborator Author

tkf commented Feb 10, 2013

意外と簡単に直せたので PR しときました。

@kiwanami
Copy link
Owner

非同期なので仕方ないかなと思っていましたが、はまりやすそうなポイントだと思いますので、良いと思います。
ありがとうございます!

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 a pull request may close this issue.

2 participants