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

Recursive add on Windows #637

Closed
richardschneider opened this issue Nov 24, 2017 · 5 comments
Closed

Recursive add on Windows #637

richardschneider opened this issue Nov 24, 2017 · 5 comments

Comments

@richardschneider
Copy link
Contributor

js-ipfs is adding extra folders when running on Windows. Also the parent folder test/fixtures/ should be removed from the output.

This was detected by the refs.spec and discovered in PR 636

node ..\js-ipfs\src\cli\bin.js add -r test\fixtures\test-folder

added QmbkMNB6rwfYAxRvnG9CWJ6cKKHEdq2ZKTozyF5FQ7H8Rs test/fixtures/test-folder/version
added QmctZfSuegbi2TMFY2y3VQjxsH5JbRBu7XmiLfHNvshhio test/fixtures/test-folder/ls
added QmU7wetVaAqc3Meurif9hcYBHGvQmL5QdpPJYBoZizyTNL test/fixtures/test-folder/ipfs-add
added QmafNdmKMjUPoAQLXtsjB5Q9DfRp5cVQY621gmyBGaYCyK test/fixtures/test-folder/hello-link
added QmdFyxZXsFiP4csgfM5uPu99AvFiKH62CSPDw5TP92nr7w test/fixtures/test-folder/files/ipfs.txt
added QmY9cxiHqTFoWamkQVkpmmqzBrY3hCBEL2XNu3NtX74Fuu test/fixtures/test-folder/files/hello.txt
added QmTYFLz5vsdMpq4XXw1a1pSxujJc9Z5V3Aw1Qg64d849Zy test/fixtures/test-folder/files
added QmNeHxDfQfjVFyYj2iruvysLH9zpp78v3cu1s3BZq1j5hY test/fixtures/test-folder/cat
added QmcUYKmQxmTcFom4R4UZP7FWeQzgJkwcFn51XrvsMy7PE9 test/fixtures/test-folder/add
added Qma8UM1iXpRjXJaG9PKxWzz9bnpruASGpLDKEy5cd5Apmz test/fixtures/test-folder
added QmQwGoVHMVMrQbZjDKkQqzDVgLXpe1NJ5FmGdmBB1FZVUM test/fixtures
added QmVZScPwqwp4e1VY5gYUMEc2g1LgcZdv9HoWSqTJuTYfWh test

ipfs add -r test\fixtures\test-folder

 QmcUYKmQxmTcFom4R4UZP7FWeQzgJkwcFn51XrvsMy7PE9 test-folder/add
 QmNeHxDfQfjVFyYj2iruvysLH9zpp78v3cu1s3BZq1j5hY test-folder/cat
 QmY9cxiHqTFoWamkQVkpmmqzBrY3hCBEL2XNu3NtX74Fuu test-folder/files/hello.txt
 QmdFyxZXsFiP4csgfM5uPu99AvFiKH62CSPDw5TP92nr7w test-folder/files/ipfs.txt
 QmafNdmKMjUPoAQLXtsjB5Q9DfRp5cVQY621gmyBGaYCyK test-folder/hello-link
 QmU7wetVaAqc3Meurif9hcYBHGvQmL5QdpPJYBoZizyTNL test-folder/ipfs-add
 QmctZfSuegbi2TMFY2y3VQjxsH5JbRBu7XmiLfHNvshhio test-folder/ls
 QmbkMNB6rwfYAxRvnG9CWJ6cKKHEdq2ZKTozyF5FQ7H8Rs test-folder/version
added QmTYFLz5vsdMpq4XXw1a1pSxujJc9Z5V3Aw1Qg64d849Zy test-folder/files
added Qma8UM1iXpRjXJaG9PKxWzz9bnpruASGpLDKEy5cd5Apmz test-folder
@richardschneider
Copy link
Contributor Author

richardschneider commented Nov 24, 2017

There is a related issue on ipfs ls.

It looks like issue #408 was not completely resolved.

@richardschneider
Copy link
Contributor Author

The above should fix this issue. But I can not install ipfsd-ctl anymore (see ipfs/js-ipfsd-ctl#177) and cannot proceed with testing. :(

@daviddias
Copy link
Contributor

Thank you @richardschneider. Seems that you opened this issue in the wrong repo (js-ipfs-api vs js-ipfs) but I got the memo that the fix is at ipfs/js-ipfs#1108 and that you need ipfs/js-ipfsd-ctl#177

@richardschneider
Copy link
Contributor Author

richardschneider commented Nov 25, 2017

Yes, a bug can crop up in one place, but is squashed in another place.

@daviddias
Copy link
Contributor

@richardschneider I didn't follow your last comment, could you rephrase?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants