[WAGON-448] Use Java String replace to avoid commons-lang dependency

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Duplicate
• Component/s: wagon-file
• Assignee: Unassigned
• Reporter: Fabian Lange
• Created: 02/Jan/16 15:23
• Updated: 28/May/16 23:16
• Resolved: 03/Jan/16 11:52

Description

I am trying to reduce the amount of dependencies of the Apache Karaf project.
One of the subcomponents uses wagon-http and wagon-file. By doing so, it receives a dependency to commons-lang.

The reason is that wagon-file has a single use of StringUtils.replaceAll().

https://github.com/apache/maven-wagon/blob/master/wagon-providers/wagon-file/src/main/java/org/apache/maven/wagon/providers/file/FileWagon.java#L206

This dependency can be easily avoided using the JDK util

destinationPath.replace('\\', '/');

while this is also a bit faster, the main benefit is that we now can completely remove the lang dependency.

Thank you!

Issue Links

• duplicates: WAGON-447 Remove Commons Lang 2 for a few cases
• links to: GitHub Pull Request #20