apache / apache/maven-scm

[SCM-998] Streamline file separator used in ScmFile.getPath()

Open
#1,221 0 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
Java
Stars
104
Forks
189
Avg merge
17h 46m
Merged PRs (30d)
6

Description

**[Konrad Windszus](https://issues.apache.org/jira/secure/ViewProfile.jspa?name=kwin)** opened **[SCM-998](https://issues.apache.org/jira/browse/SCM-998?redirect=false)** and commented

Currently the different `ScmManager` methods sometimes return the OS separator and sometimes only the forward slash in the `ScmFile` (used in the complex return object).

As forward slash is currently returned on all OSes for `ScmManager.checkIn(...)` all other methods like `ScmManager.add(...)` and `ScmManager.remove(...)` should behave the same. Also the javadoc at `ScmFile.getPath()` should be clarified in this regard.

---

**Issue Links:**
- [SCM-807](https://issues.apache.org/jira/browse/SCM-807) JGit impl check-in fails unless the Maven project is in the working copy root
(_**"Discovered while testing"**_)

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by reading ScmFile.getPath() and the ScmManager.checkIn(), add(), and remove() entry points mentioned in the issue. Compare the separators returned by these methods across operating systems, then clarify the getPath() Javadoc and verify that add() and remove() use the same forward-slash behavior as checkIn().

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
build-system, tooling
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.