Can I skip maven main process when running some recipe

[XiaoSK]

Some Java projects use JDK 8 and depend on tools.jar located under the JDK's systemPath. However, many recipes utilize classes that are only available in JDK 9 or later. Consequently, running these recipes on JDK 11 fails. I observed that adding the JVM option -Dmaven.main.skip=true allows these recipes to execute successfully by skipping certain build phases.

Can I safely apply -Dmaven.main.skip=true to all recipes? If so, what impact does this have on the recipes? Under which circumstances would this cause side effects, and when would it be safe without adverse consequences?

[timtebeek]

However, many recipes utilize classes that are only available in JDK 9 or later.

Which recipes? Because we compile ours back down to Java 8.

I've not heard of folks using -Dmaven.main.skip=true before; if that works for you then great; there's a small risk in that you might miss generating or compiling sources that we might need for type attribution, but if you know that's not the case for you then I suppose it's fine. We mostly tag onto the regular lifecycle to avoid those steps being skipped.

[XiaoSK]

@timtebeek I mean the recipes that I created myself. Could you provide an example illustrating how adding maven.main.skip to an official recipe could cause it to fail?

[timtebeek]

So we attach the run phase to run after process-test-classes, precisely since that could trigger say and OpenAPI code generator to generate & compile sources we need for type information.

rewrite-maven-plugin/src/main/java/org/openrewrite/maven/RewriteRunMojo.java

Lines 29 to 32 in 86478fe

	@Execute(phase = LifecyclePhase.PROCESS_TEST_CLASSES)
	@Mojo(name = "run", requiresDependencyResolution = ResolutionScope.TEST, threadSafe = true,
	defaultPhase = LifecyclePhase.PROCESS_TEST_CLASSES)
	public class RewriteRunMojo extends AbstractRewriteRunMojo {

If that's not expected on your end then you could be fine without that step I think; would interested to hear your findings if continued.