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

-localCheckoutDir does not work with multi-module projects #384

Closed
dwnusbaum opened this issue Nov 3, 2022 · 6 comments
Closed

-localCheckoutDir does not work with multi-module projects #384

dwnusbaum opened this issue Nov 3, 2022 · 6 comments
Assignees

Comments

@dwnusbaum
Copy link
Member

dwnusbaum commented Nov 3, 2022

(The plugin-compat-tester component cannot be used in Jira anymore, so I guess GitHub issues are preferred?)

The -localCheckoutDir option does not work with multi-module projects even when there is an implementation of AbstractMultiParentHook for the plugin in question. Specifically, MultiParentCompileHook.isSnapshotMultiParentPlugin returns false, so the default process-test-classes goal is used instead of MultiParentCompileHook changing the goal to install, which fails (JENKINS-62658 describes the issue with process-test-classes more generally).

The problem seems to be that AbstractMultiParentHook intentionally doesn't do anything when localCheckoutDir is used, which means that these properties never get specified, so the properties are null here and the multi-module-specific logic gets skipped.

See #383 (comment) for some additional discussion specific to workflow-cps.

@jglick
Copy link
Member

jglick commented Nov 3, 2022

More generally I would like for the multimodule support to be rewritten. Should collect all the repos that need testing, then run Maven from the root of each of those. Do not want to just mvn test or mvn install in one pass, despite jenkinsci/maven-hpi-plugin#336, because -Djenkins.version=… would compile against the wrong core, but could do two passes.

@olamy
Copy link
Member

olamy commented Nov 4, 2022

To help with this, I proposed a change here
But I close the PR because it looks this will not being accepted.

@jglick
Copy link
Member

jglick commented Nov 11, 2022

Some discussion of the jenkins.version issue: jenkinsci/bom#1479 (comment)

@jglick
Copy link
Member

jglick commented Dec 1, 2022

CC @jtnord

@jglick
Copy link
Member

jglick commented Feb 6, 2023

Regarding mvn install on snapshots, https://maven.apache.org/resolver/local-repository.html#use-cases (3.9.0+) merits investigation.

@jtnord jtnord self-assigned this Mar 17, 2023
@jtnord
Copy link
Member

jtnord commented Apr 19, 2023

included in 1313.v036de64e1863

@jtnord jtnord closed this as completed Apr 19, 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

4 participants