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

Changes the temporary dir location to work on every system #114

Merged
merged 1 commit into from
Oct 12, 2018
Merged

Changes the temporary dir location to work on every system #114

merged 1 commit into from
Oct 12, 2018

Conversation

ZloeSabo
Copy link
Contributor

  • uses the system temp dir + pid combination to avoid renaming child
    folder to a parent folder, which fails on linux;
  • cleans up the temporary folder during emergency shutdown to avoid
    leaving orphan files;
  • still fails on linux when providing existing folder to the example
    command;
  • still does not initialize modules for the cloned example.

* uses the system temp dir + pid combination to avoid renaming child
folder to a parent folder, which fails on linux;
* cleans up the temporary folder during emergency shutdown to avoid
leaving orphan files;
* still fails on linux when providing existing folder to the example
command;
* still does not initialize modules for the cloned example.
@Manvel
Copy link
Member

Manvel commented Oct 12, 2018

Thanks @ZloeSabo this is very helpful. I'll have final look this weekend.

Can you please let me know which Linux distribution + version you are using?

@ZloeSabo
Copy link
Contributor Author

@Manvel i've checked with Alpine 3.8 and Mint 19.

@Manvel Manvel mentioned this pull request Oct 12, 2018
4 tasks
@Manvel Manvel merged commit 67198fc into cmints:master Oct 12, 2018
@Manvel
Copy link
Member

Manvel commented Oct 12, 2018

Thanks @ZloeSabo I've created a followup here #115 hope to tackle them this week.

@Manvel
Copy link
Member

Manvel commented Oct 14, 2018

@ZloeSabo Fixes are published in the npm, should be available in the versions greater than 0.7.0.

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