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

Temporary directories not cleaned #288

Open
joelberkeley opened this issue May 26, 2024 · 2 comments · May be fixed by #301
Open

Temporary directories not cleaned #288

joelberkeley opened this issue May 26, 2024 · 2 comments · May be fixed by #301
Labels
feature request Request for new functionality user experience Related to subjective feelings of users

Comments

@joelberkeley
Copy link
Contributor

joelberkeley commented May 26, 2024

Looks like pack's not tidying temporary directories. I often Ctrl+C mid-install, might be the cause

$ ls .pack/
bin/     db/      .tmp0/   .tmp10/  .tmp12/  .tmp14/  .tmp16/  .tmp18/  .tmp2/   .tmp21/  .tmp4/   .tmp6/   .tmp8/   user/
.cache/  install/ .tmp1/   .tmp11/  .tmp13/  .tmp15/  .tmp17/  .tmp19/  .tmp20/  .tmp3/   .tmp5/   .tmp7/   .tmp9/
@stefan-hoeck
Copy link
Owner

Thanks for reporting this and sorry for the late response. You can remove such artifacts (together with older pack collections) with pack gc. Ideally, pack would catch interrupting signals and cleanup before terminating. I know this can be done with idris2-uv, but I'm not sure whether the stuff in base would be already enough here.

@buzden buzden added feature request Request for new functionality user experience Related to subjective feelings of users labels Jul 2, 2024
@Matthew-Mosior
Copy link
Contributor

Matthew-Mosior commented Aug 14, 2024

@stefan-hoeck Does pack need only rely upon base? I believe idris2-async could help here as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request Request for new functionality user experience Related to subjective feelings of users
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants