Skip to content
This repository has been archived by the owner on Mar 10, 2020. It is now read-only.

fix(files.add): handle weird directory names #646

Merged
merged 1 commit into from
Dec 17, 2017
Merged

Conversation

richardschneider
Copy link
Contributor

glob is interpreting the '[' in weird [v0] as a special magic character. According to its documentation, a caller should be able to escape it with \; but it does not work. See the glob issue.

Should fix a directory with an odd name.

Copy link
Contributor

@daviddias daviddias left a comment

Choose a reason for hiding this comment

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

Thanks @richardschneider , mind adding tests as well?

@richardschneider
Copy link
Contributor Author

This is getting an existing test to work. So I don't need to add another one.

@richardschneider
Copy link
Contributor Author

@diasdavid The test already exists.

@daviddias
Copy link
Contributor

Thanks for submitting this patch @richardschneider :)

PS: It's frustrating that we can see Appveyor getting green because there is no windows-386 of go-ipfs.

@daviddias daviddias merged commit 012b86c into master Dec 17, 2017
@daviddias daviddias deleted the weird-dirname branch December 17, 2017 21:39
@ghost ghost removed the in progress label Dec 17, 2017
@daviddias daviddias mentioned this pull request Dec 18, 2017
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.

2 participants