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.

Sign up for GitHub

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

Jump to bottom

ipfs get -o file $address fails (minor argument parse bug) #856

Closed
joeyh opened this issue Mar 5, 2015 · 5 comments
Closed

ipfs get -o file $address fails (minor argument parse bug) #856

joeyh opened this issue Mar 5, 2015 · 5 comments
Labels
exp/novice Someone with a little familiarity can pick up kind/bug A bug in existing code (including security flaws) topic/commands Topic commands topic/docs-ipfs Topic docs-ipfs

Comments

@joeyh
Copy link

joeyh commented Mar 5, 2015

ipfs get -o foo Qma8oHaVQtGxsapu1ZCP27CS9A5KhD2C4sqLPSxK7xNLEu

Error: Expected 1 arguments, got 2

It seems that --output=foo works, but no way to use the short option despite it being documented.
@joeyh joeyh changed the title ipfs get -o file $address fails ipfs get -o file $address fails (minor argument parse bug) Mar 5, 2015
@mappum
Copy link
Contributor

mappum commented Mar 5, 2015

That is what is currently expected, -o=foo is supported but -o foo is not. However, if many people expect this argument convention to work then it is a UX bug.

@joeyh
Copy link
Author

joeyh commented Mar 5, 2015

This could be considered a documentation bug; I read the --help and
didn't see that it needed -o=

see shy jo

@whyrusleeping
Copy link
Member

@joeyh agreed on it being documentation bug

@whyrusleeping whyrusleeping added kind/bug A bug in existing code (including security flaws) topic/docs-ipfs Topic docs-ipfs labels Mar 5, 2015
@whyrusleeping
Copy link
Member

i think this is fixed now, right @jbenet ?

@jbenet
Copy link
Member

jbenet commented May 6, 2015

I believe so, yes. thanks again @AtnNn

@jbenet jbenet closed this as completed May 6, 2015
@RichardLitt RichardLitt added the exp/novice Someone with a little familiarity can pick up label Feb 2, 2016
@lidel lidel mentioned this issue Apr 21, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
exp/novice Someone with a little familiarity can pick up kind/bug A bug in existing code (including security flaws) topic/commands Topic commands topic/docs-ipfs Topic docs-ipfs
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@joeyh @jbenet @mappum @RichardLitt @whyrusleeping