ipfs/kubo

`ipfs files cp/write` from local fs? confusing errors?

Open

#3,126 opened on Aug 26, 2016

View on GitHub
 (4 comments) (0 reactions) (0 assignees)Go (13,906 stars) (2,725 forks)batch import
help wantedkind/enhancement

Description

Version/Platform/Processor information (from ipfs version --all):

0.4.3-rc3

Type (bug, feature, meta, test failure, question): feature Area (api, commands, daemon, fuse, etc): commands Priority (from P0: functioning, to P4: operations on fire): P0 Description:

Would be nice if this worked:

> ipfs files cp /Users/earth/go/src/github.com/ipfs/go-ipfs/Makefile /Makefile
Error: file does not exist

The error is super confusing anyway.

Contributor guide

Tech stack
go
Domain
cli
Issue type
feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
half day
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
stale
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
Go programmingIPFS commands knowledgeFamiliarity with ipfs files API
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
60
Research direction
The issue requests adding support for local file paths to `ipfs files cp/write` and improving the confusing error message. Currently, the command likely only accepts IPFS paths. To implement, examine the command handler for `files cp` in the `commands/files/` directory of the go ipfs repository, specifically how paths are parsed and resolved. The error 'file does not exist' from the issue suggests it expects an IPFS path; modify the handler to detect local filesystem paths (e.g., starting with '/') and handle them accordingly. Also update error messages to clearly indicate whether a local path or IPFS path is expected. Review existing tests for `files cp` to understand current behavior and add new tests for local path support.
`ipfs files cp/write` from local fs? confusing errors? · ipfs/kubo#3126 | Good First Issue