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

New Errors showing up in Xtext builds with Tycho 5 #4226

Closed
cdietrich opened this issue Sep 9, 2024 · 4 comments · Fixed by #4227
Closed

New Errors showing up in Xtext builds with Tycho 5 #4226

cdietrich opened this issue Sep 9, 2024 · 4 comments · Fixed by #4227

Comments

@cdietrich
Copy link
Contributor

cdietrich commented Sep 9, 2024

1) No implementation for TargetPlatformArtifactResolver was bound.
  while locating TargetPlatformWorkspaceReader
  at ClassRealm[extension>org.eclipse.tycho:tycho-maven-plugin:5.0.0-SNAPSHOT, parent: ClassLoaders$AppClassLoader@2cdf8d8a]
      \_ installed by: WireModule -> PlexusBindingModule
  while locating WorkspaceReader annotated with @Named("TargetPlatformWorkspaceReader")

1 error

https://github.com/xtext/xtext-reference-projects/actions/runs/10773540359/job/29873555998

does this ring a bell?

@laeubi
Copy link
Member

laeubi commented Sep 9, 2024

@cdietrich do you use a recent maven version? Maybe you can run with mvn -V ?

This might be due to

but I'm a bit confused it does not came up in the regular integration builds maybe @cstamas has a hint whats going wrong here?

@cdietrich
Copy link
Contributor Author

we use 3.9.1, will check with a newer

@cdietrich
Copy link
Contributor Author

with 3.9.9 it looks better. maybe we need a newer version as
requireMavenVersion

@laeubi
Copy link
Member

laeubi commented Sep 9, 2024

Yes we need at a lest 3.9.6 as it seems. One problem is that this is only checked once the build runs (and we are using extensions here that are not checked at all).

laeubi added a commit to laeubi/tycho that referenced this issue Sep 9, 2024
@laeubi laeubi closed this as completed in 9b3eb9f Sep 9, 2024
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 a pull request may close this issue.

2 participants