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

Pomless build with tycho 2.7.5 does not work with other tools #2401

Closed
jan-pesta opened this issue May 12, 2023 · 5 comments
Closed

Pomless build with tycho 2.7.5 does not work with other tools #2401

jan-pesta opened this issue May 12, 2023 · 5 comments
Milestone

Comments

@jan-pesta
Copy link

I am migrating to newer tycho 2.7.5 from 1.4.0.

I just find out that sonar analysis are not working with new extension.
When version on has been used, it generate following Source paths:
[INFO] 02:42:40.417 Source paths: .polyglot.build.properties, src
With version 2.7.5 it is generating
[INFO] 02:42:21.902 Source paths: .polyglot.META-INF

In case of multiple source folders, the older version is correctly providing all source folders listed.

I am not able to test it with 3.x line as it is forcing to use newer java that 11, but it looks like is generating same Source paths as 2.7.5.

@jan-pesta
Copy link
Author

jan-pesta commented May 12, 2023

I got following source files:
[INFO] 10:16:36.413 Source paths: .polyglot.META-INF, src

When I tried some changes - and this was result of incompatible configuration, tycho 1.4.0 has been used for build, but extras was with a version 2.7.5, so it looks like it is not from pomless build

@jan-pesta
Copy link
Author

I just find out that there is no class path resolution when running sonar analysis.

Is there a way to force classpath resolution to be executed also when direct goal sonar:sonar is executed?

Seems this to be side effect of #460 .

@laeubi
Copy link
Member

laeubi commented May 15, 2023

Is there a way to force classpath resolution to be executed also when direct goal sonar:sonar is executed?

You can open a bug to the sonar-lint plugin that it does not require any classpath, beside that Tycho 4 has an option -Dtycho.target.eager=true to perform target platform resolution before any other plugin running.

@laeubi
Copy link
Member

laeubi commented May 16, 2023

Can you provide an integration-test to demonstrate the issue?

@jan-pesta
Copy link
Author

I think important information was that classpath resolution has been refactored into compile time. I have solved the problem when I put sonar into one build with compilation.

so solution was to start mvn clean install sonar:sonar together, and issue is solved.

previously I have two mvn runs.

@laeubi laeubi added this to the 4.0 milestone Jun 24, 2023
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

No branches or pull requests

2 participants