Skip to content
This repository has been archived by the owner on Jul 3, 2019. It is now read-only.

fix(deps): update tar-fs with the special characters patch #102

Merged
merged 1 commit into from
Jun 10, 2017

Conversation

mcibique
Copy link
Contributor

@mcibique mcibique commented Jun 9, 2017

fixes npm/npm#16882. tar-fs has already been fixed and published. I added a test only for win32 platform because issues with special characters can be seen only there. Also the test can be used regardless of the package used for extracting tarballs (node-tar or tar-fs).

Copy link
Owner

@zkat zkat left a comment

Choose a reason for hiding this comment

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

Nice! Thanks a lot for following through on this! 🎉 🐑

@zkat zkat merged commit ed43aa3 into zkat:latest Jun 10, 2017
zkat added a commit that referenced this pull request Nov 15, 2017
)"

This reverts commit ed43aa3.

This commit is necessary because @mcibique did not respond to
#107 after more than 3 months and
several pings. This is therefore the only commit that could not
be relicensed from CC0.
zkat added a commit that referenced this pull request Nov 15, 2017
)"

This reverts commit ed43aa3.

This commit is necessary because @mcibique did not respond to
#107 after more than 3 months and
several pings. This is therefore the only commit that could not
be relicensed from CC0.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

npm@5 install error: ENOENT: no such file or directory
2 participants