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

ENH: Create user-specific temporary directory on unix systems #834

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
3 participants
@lassoan
Contributor

lassoan commented Nov 4, 2017

Slicer temporary directory is owned by the user who started Slicer first and is not writeable. Issue identified by Artem Mamonov.

Fixes https://issues.slicer.org/view.php?id=4340

ENH: Create user-specific temporary directory on unix systems
Slicer temporary directory is owned by the user who started Slicer first and is not writeable. Issue identified by Artem Mamonov.

Fixes https://issues.slicer.org/view.php?id=4340
@ihnorton

This comment has been minimized.

Member

ihnorton commented Nov 4, 2017

Is there a reason to use a persistent name here instead of process-unique? (e.g. mkdtemp / QTemporaryDir). Is this where Slicer puts the sample data, and we want that to persist?

(edit: oh I see QTemporaryDir is new in Qt5)

@lassoan

This comment has been minimized.

Contributor

lassoan commented Nov 4, 2017

Yes, exactly. Slicer keeps cached data in temp dir, so we need to preserve it.

@jcfr

jcfr approved these changes Nov 4, 2017

@lassoan

This comment has been minimized.

Contributor

lassoan commented Nov 4, 2017

Integrated into r26549.

@lassoan lassoan closed this Nov 4, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment