Skip to content
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.

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

cmd/go: 'go test -coverprofile' emits slash-separated paths on Windows #27486

Closed
bcmills opened this issue Sep 4, 2018 · 3 comments
Closed
Labels
FrozenDueToAge NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. OS-Windows
Milestone

Comments

@bcmills
Copy link
Contributor

bcmills commented Sep 4, 2018

In microsoft/vscode-go#1847 (comment), @ramya-rao-a notes that in Go 1.11, go test -coverprofile on Windows changed to emit slash-separated paths instead of using the OS-local separator.

She has a workaround in vscode (where the regression was detected), but there may be other tools currently broken on Windows.

@bcmills bcmills added OS-Windows NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. labels Sep 4, 2018
@bcmills bcmills added this to the Go1.12 milestone Sep 4, 2018
@bcmills
Copy link
Contributor Author

bcmills commented Sep 4, 2018

@gopherbot, please backport to 1.11: this appears to be a regression from 1.9 or 1.10.

@gopherbot
Copy link

Backport issue(s) opened: #27487 (for 1.11), #27488 (for 1.10).

Remember to create the cherry-pick CL(s) as soon as the patch is submitted to master, according to https://golang.org/wiki/MinorReleases.

@bcmills
Copy link
Contributor Author

bcmills commented Oct 24, 2018

@rsc says that the paths are intentionally (now) supposed to be the import path + filename, since the file may be in the module cache rather than the local filesystem.

Since @ramya-rao-a has updated the VSCode support to accept the new format and nobody else seems to have noticed, let's leave this as-is.

@bcmills bcmills closed this as completed Oct 24, 2018
@golang golang locked and limited conversation to collaborators Oct 24, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
FrozenDueToAge NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. OS-Windows
Projects
None yet
Development

No branches or pull requests

2 participants