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.

Sign up for GitHub

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

Jump to bottom

Use /tmp as $TMPDIR if it is not set #41

Merged
merged 1 commit into from
Feb 21, 2017
Merged

Use /tmp as $TMPDIR if it is not set #41

merged 1 commit into from
Feb 21, 2017

Conversation

mecab
Copy link
Collaborator

@mecab mecab commented Feb 21, 2017

I found $TMPDIR is not set in some environments. In this case, ttcopy tries to make temp file at / (root) then probably go to die due to permission error.

This PR gives /tmp for the default $TMPDIR.

@mecab mecab requested a review from greymd February 21, 2017 11:17
@mecab mecab mentioned this pull request Feb 21, 2017
Merged
greymd
greymd approved these changes Feb 21, 2017
View reviewed changes
Copy link
Owner

@greymd greymd left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approve

@greymd greymd merged commit abce51f into master Feb 21, 2017
@greymd greymd deleted the bug/default-tmpdir branch February 21, 2017 12:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@greymd greymd greymd approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mecab @greymd