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

Cannot navigate to a dependency pom.xml by f12 #924

Closed
Tracked by #1135
phommy opened this issue Dec 29, 2022 · 3 comments · Fixed by #925
Closed
Tracked by #1135

Cannot navigate to a dependency pom.xml by f12 #924

phommy opened this issue Dec 29, 2022 · 3 comments · Fixed by #925
Assignees
Labels
bug Something isn't working

Comments

@phommy
Copy link

phommy commented Dec 29, 2022

Describe the bug
When navigate to dependency pom.xml, got wrong folder path.

To Reproduce

  1. Open a pom.xml file of a java project
  2. On a dependencies\dependency node, F12 or ctrl+click
  3. Expected behavior: navigate to the dependency pom.xml. e.g. D:\maven_cache\com\sun\mail\all\1.5.0\all-1.5.0.pom
    (see https://github.com/microsoft/vscode-maven/issues/428).
  4. Actually behavior: naigate to C:\Program Files\Microsoft VS Code\com\sun\mail\all\1.5.0\all-1.5.0.pom , which does not exist.
  5. In settings.xml of maven, localRepository = d:\maven_cache\

Environments (please complete the following information as much as possible):

  • OS: Windows 10
  • VS Code version: 1.74.2
  • Extension version: Maven for Java[0.40.2]

Screenshots
error

@Eskibear Eskibear added the bug Something isn't working label Dec 30, 2022
@Eskibear
Copy link
Member

It should be a regression introduced by:

db028b3#diff-a2b9078fc2f74c772f756ca0ab1fc23d5fe9b2a70c65c1cb3c666c559c7e3bf9R48

@Eskibear
Copy link
Member

@phommy You can switch to pre-release version (0.40.2023011003) to verify the fix, please let me know if it works.

@phommy
Copy link
Author

phommy commented Jan 10, 2023

@phommy You can switch to pre-release version (0.40.2023011003) to verify the fix, please let me know if it works.

@Eskibear it works.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants