Skip to content

feat(cmds): add command output option #8683

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
Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

feat(cmds): add command output option #8683

wants to merge 2 commits into from

Conversation

@schomatis
Copy link
Contributor

@schomatis schomatis commented Jan 18, 2022

Placeholder applying ipfs/go-ipfs-cmds#223 to add a command option to save output to a file.

Example test (should work with or without the daemon running):

BLOCK_PATH=/ipfs/QmQPeNsJPyVWPFDVHb77w8G42Fvo15z4bG2X8D2GhfbSXc/readme

ipfs block get $BLOCK_PATH > shell_redirected_out

ipfs block get $BLOCK_PATH --command-output option_redirected_out

cmp shell_redirected_out option_redirected_out

Fixes #8335.

@schomatis schomatis requested a review from lidel January 18, 2022 16:18
@schomatis schomatis mentioned this pull request Jan 18, 2022
Open
@BigLep BigLep added this to the Best Effort Track milestone Mar 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lidel lidel Awaiting requested review from lidel

At least 1 approving review is required to merge this pull request.

Assignees

@schomatis schomatis

Labels

None yet

Projects

No open projects
IPFS Shipyard Team
Status: 🔎 In Review

Milestone

Best Effort Track

Development

Successfully merging this pull request may close these issues.

Support "--output=<file>" in cmds that return data to stdout

3 participants

@schomatis @BigLep