Skip to content

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

Merged
Manvel merged 1 commit intocmints:masterfrom
ZloeSabo:master
Oct 12, 2018
Merged

Changes the temporary dir location to work on every system#114
Manvel merged 1 commit intocmints:masterfrom
ZloeSabo:master

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