Skip to content

Update eclipse to use v4.33 #571

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.

Sign up for GitHub

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

Jump to bottom
Merged
testforstephen merged 1 commit into from
Sep 18, 2024
Merged

Update eclipse to use v4.33 #571

testforstephen merged 1 commit into from
Sep 18, 2024

Conversation

@vincadrn
Copy link
Contributor

@vincadrn vincadrn commented Sep 16, 2024

Using current version of 4.33-I-builds yields this error when trying to ./mvnw clean install:

Failed to load p2 metadata repository from location https://download.eclipse.org/eclipse/updates/4.33-I-builds/I20240613-1800:
No repository found at https://download.eclipse.org/eclipse/updates/4.33-I-builds/I20240613-1800.

Tried to use just 4.33 instead and the error is gone now.

[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary for Java Debug Server for Visual Studio Code :: Parent 0.53.0:
[INFO]
[INFO] Java Debug Server for Visual Studio Code :: Parent . SUCCESS [  0.046 s]
[INFO] Java Debug Server for Visual Studio Code :: Debugger Core SUCCESS [  3.488 s]
[INFO] Java Debug Server for Visual Studio Code :: Debugger Plugin SUCCESS [  4.642 s]
[INFO] Java Debug Server for Visual Studio Code :: P2 Update site SUCCESS [  0.420 s]
[INFO] Java Debug Server for Visual Studio Code :: Target Platform SUCCESS [  0.004 s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------

@vincadrn vincadrn force-pushed the fix/dbg-tgt-repo-loc branch from 89295fd to 76488a5 Compare September 16, 2024 09:58
testforstephen
testforstephen approved these changes Sep 18, 2024
View reviewed changes
Copy link
Contributor

@testforstephen testforstephen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. thanks for contribution.

@testforstephen testforstephen merged commit e54f5c2 into microsoft:main Sep 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@testforstephen testforstephen testforstephen approved these changes

@jdneo jdneo Awaiting requested review from jdneo jdneo is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@vincadrn @testforstephen