Skip to content

ioutilx: replace cygpath with pure Go path conversion#4994

Open
varsha-0007 wants to merge 3 commits into
lima-vm:masterfrom
varsha-0007:remove-cygpath
Open

ioutilx: replace cygpath with pure Go path conversion#4994
varsha-0007 wants to merge 3 commits into
lima-vm:masterfrom
varsha-0007:remove-cygpath

Conversation

@varsha-0007
Copy link
Copy Markdown
Contributor

Removes the dependency on cygpath.exe by replacing the external
tool call in WindowsSubsystemPath with pure Go path conversion
using filepath and strings packages.

This is part of the cygpath.exe removal work tracked in #4907.

  • Converts Windows paths (e.g. C:\Users\foo) to Unix-style
    paths (e.g. /c/Users/foo) natively in Go
  • Adds tests for WindowsSubsystemPath

Signed-off-by: Varsha varsha.narvi.07@gmail.com

Signed-off-by: varsha <varsha.narvi.07@gmail.com>

fix: fix import grouping for gci linter

Signed-off-by: varsha <varsha.narvi.07@gmail.com>

fix: fix import grouping and trailing whitespace

Signed-off-by: varsha <varsha.narvi.07@gmail.com>

fix: fix gci import grouping

Signed-off-by: varsha <varsha.narvi.07@gmail.com>
Signed-off-by: varsha <varsha.narvi.07@gmail.com>
Signed-off-by: varsha <varsha.narvi.07@gmail.com>

ioutilx: replace cygpath with pure Go path conversion

Signed-off-by: varsha <varsha.narvi.07@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant