Skip to content
This repository has been archived by the owner on Jul 24, 2019. It is now read-only.

Cleanup ./tmp when done installing #466

Open
avindra opened this issue Feb 8, 2016 · 5 comments
Open

Cleanup ./tmp when done installing #466

avindra opened this issue Feb 8, 2016 · 5 comments

Comments

@avindra
Copy link
Contributor

avindra commented Feb 8, 2016

du -h --max-depth=1 tmp/
23M     tmp/phantomjs

The tmp/ folder remains on the filesystem everything is installed and moved into place. Can we get a cleanup in place to get rid of the 23M of tmp files?

@nicks
Copy link
Contributor

nicks commented Feb 9, 2016

some users take advantage of this so that downloads are cached.

we would accept a pull request to make this an opt-in setting.

I also like tmpwatcher/tmpreaper for cleaning up tmp on machines periodically.

@Arkni
Copy link

Arkni commented Feb 12, 2016

+1 to "make this an opt-in setting".

@nicks
After moving the folder phantomjs-version-platform[1] to lib/phantom, the temporary folder phantomjs-version-platform.extension-extract-new Date()[2] remain in file system. Can we get ride of it after moving the relevant files ?

[1] phantomjs-2.1.1-linux-x86_64 for example
[2] phantomjs-2.1.1-linux-x86_64.tar.bz2-extract-1455233501966 for example

@BernardoSilva
Copy link

👍 to "make this an opt-in setting".

@Qazzian
Copy link

Qazzian commented May 18, 2016

Keeping the download is fine and will help to mitigate network issues.
It's the fact that it produces a new extract folder every time it runs. On a CI server this produces
a silly amount of empty extract directories.

@AAudusseau
Copy link

Any update of this ?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants