Runtime dependencies not resolved from target file

[akurtakov]

Using https://git.eclipse.org/c/platform/eclipse.platform.releng.aggregator.git/tree/eclipse.platform.releng.prereqs.sdk/eclipse-sdk-prereqs.target - objenesis is removed and and includeDependencyScope="runtime" fails resolution with:
Cannot resolve project dependencies:
[ERROR] Software being installed: org.eclipse.ui.tests 3.15.500.qualifier
[ERROR] Missing requirement: org.mockito.mockito-core 4.1.0 requires 'java.package; org.objenesis [3.1.0,4.0.0)' but it could not be found
[ERROR] Cannot satisfy dependency: org.eclipse.ui.tests 3.15.500.qualifier depends on: osgi.bundle; org.mockito.mockito-core 2.13.0

@laeubi objenesis is supposed to be resolved as it's transitive runtime dependency of mockito as per https://repo1.maven.org/maven2/org/mockito/mockito-core/4.1.0/mockito-core-4.1.0.pom, right?

[akurtakov]

For the record same failures with includeDependencyScope="compile" if objenesis is not explicitly added.

[laeubi]

objenesis is supposed to be resolved as it's transitive runtime dependency of mockito

At least that's what I would expect as per the dependency tree:

According to https://maven.apache.org/guides/introduction/introduction-to-dependency-mechanism.html

runtime
This scope indicates that the dependency is not required for compilation, but is for execution. Maven includes a dependency with this scope in the runtime and test classpaths, but not the compile classpath.

[laeubi]

Sorry for confusion, m2e-pde with current snapshot version also filter this out:

[laeubi]

Okay, It seems for the current implementation it is required to choose test dependency scope.

As per definition:
This scope indicates that the dependency is not required for compilation, but is for execution. Maven includes a dependency with this scope in the runtime and test classpaths, but not the compile classpath.

[laeubi]

@akurtakov see eclipse-m2e/m2e-core#461

[laeubi]

@mickaelistria I think this is something to consider for 2.7 as well and for the m2e release also.

[laeubi]

m2e change is done, will prepare now for tycho as well. The new syntax then would be includeDependencyScopes="compile,runtime" to include both.