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

144 errors after JDT merge in I20220921-0710 #341

Closed
iloveeclipse opened this issue Sep 21, 2022 · 10 comments
Closed

144 errors after JDT merge in I20220921-0710 #341

iloveeclipse opened this issue Sep 21, 2022 · 10 comments
Labels
regression Regression defect

Comments

@iloveeclipse iloveeclipse added the regression Regression defect label Sep 21, 2022
@jukzi
Copy link
Contributor

jukzi commented Sep 22, 2022

most of them are like

Caused by: java.lang.NullPointerException: Cannot invoke "org.eclipse.jdt.launching.environments.IExecutionEnvironment.getDefaultVM()" because "environment" is null
at org.eclipse.pde.internal.build.site.PDEState.bestVmInstallFor(PDEState.java:495)
at org.eclipse.pde.internal.build.site.PDEState.querySystemPackages(PDEState.java:466)
at org.eclipse.pde.internal.build.site.PDEState.resolveState(PDEState.java:443)
at org.eclipse.pde.internal.build.site.BuildTimeSite.getRegistry(BuildTimeSite.java:134)
at org.eclipse.pde.internal.build.builder.BuildDirector.computeElements(BuildDirector.java:126)
at org.eclipse.pde.internal.build.builder.BuildDirector.generateChildrenScripts(BuildDirector.java:284)
at org.eclipse.pde.internal.build.builder.BuildDirector.generate(BuildDirector.java:230)
at org.eclipse.pde.internal.build.builder.BuildDirector.generateIncludedFeatureBuildFile(BuildDirector.java:268)
at org.eclipse.pde.internal.build.builder.BuildDirector.generate(BuildDirector.java:228)
at org.eclipse.pde.internal.build.builder.BuildDirector.generate(BuildDirector.java:223)
at org.eclipse.pde.internal.build.BuildScriptGenerator.generateFeatures(BuildScriptGenerator.java:253)
at org.eclipse.pde.internal.build.BuildScriptGenerator.generate(BuildScriptGenerator.java:130)
at org.eclipse.pde.internal.build.tasks.BuildScriptGeneratorTask.run(BuildScriptGeneratorTask.java:110)
at org.eclipse.pde.internal.build.tasks.BuildScriptGeneratorTask.execute(BuildScriptGeneratorTask.java:96)
at org.apache.tools.ant.UnknownElement.execute(UnknownElement.java:299)

@vik-chand
Copy link
Member

I think it is because jdt.debug Java 19 merge to master is missing. This should get corrected in the I build containing this merge.

@iloveeclipse
Copy link
Member Author

@SarikaSinha : I saw you've just merged jdt debug and Noopur did that for jdt.ui.
Are we ready with outstanding merges now?
If yes, I would like to trigger new I Build to see which regressions are left.

@jukzi
Copy link
Contributor

jukzi commented Sep 22, 2022

@HannesWell regarding
#194
even though an JDK19 environment would probably fix the issue, it is documented that JavaRuntime.getExecutionEnvironmentsManager().getEnvironment(execEnvID) may return null and in that case org.eclipse.pde.internal.build.site.PDEState.bestVmInstallFor(IExecutionEnvironment) could just return null too.

@SarikaSinha
Copy link
Contributor

@SarikaSinha : I saw you've just merged jdt debug and Noopur did that for jdt.ui. Are we ready with outstanding merges now? If yes, I would like to trigger new I Build to see which regressions are left.

We are working on adjusting the bundle versions in UI and debug. Will trigger after that.

@vik-chand
Copy link
Member

Resolved after jdt.debug and jdt.ui merge

@SarikaSinha
Copy link
Contributor

I build has been triggered.

@vik-chand
Copy link
Member

@HannesWell
Copy link
Member

@HannesWell regarding #194 even though an JDK19 environment would probably fix the issue, it is documented that JavaRuntime.getExecutionEnvironmentsManager().getEnvironment(execEnvID) may return null and in that case org.eclipse.pde.internal.build.site.PDEState.bestVmInstallFor(IExecutionEnvironment) could just return null too.

That's right. Do you plan to provide a PR to fix that?

jukzi pushed a commit to jukzi/eclipse.pde that referenced this issue Sep 23, 2022
JavaRuntime.getExecutionEnvironmentsManager().getEnvironment(execEnvID)
may return null
@jukzi
Copy link
Contributor

jukzi commented Sep 23, 2022

That's right. Do you plan to provide a PR to fix that?

#342

HannesWell pushed a commit that referenced this issue Sep 23, 2022
JavaRuntime.getExecutionEnvironmentsManager().getEnvironment(execEnvID)
may return null

Co-authored-by: Jörg Kubitz <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
regression Regression defect
Projects
None yet
Development

No branches or pull requests

5 participants