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

This checkin provides support to cleam up the /tmp direcotry #26

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

vbannai
Copy link

@vbannai vbannai commented Aug 26, 2014

on both the remote and local directory of all the temporary files
are created. Please refer to the github issue for more details
#25

on both the remote and local directory of all the temporary files
are created. Please refer to the github issue for more details
JasperVanDenBosch#25
@vbannai
Copy link
Author

vbannai commented Aug 26, 2014

I realize the pexpect.pyc file is still hanging out in the /tmp directory.
I can fix it and send you another pull request. Let me know if there are any changes you want me to make and I will send you a consolidated pull request.

@JasperVanDenBosch
Copy link
Owner

Hi Vinay,
Sorry for the very late response. I like this functionality, but it should probably be optional. Could you make it dependent on some sort of flag passed? I suspect some users won't want us to do sudo stuff implicitly.
Thanks,
Jasper

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