-
Notifications
You must be signed in to change notification settings - Fork 55
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
Comments
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 |
To help with this, I proposed a change here |
Some discussion of the |
CC @jtnord |
Regarding |
included in 1313.v036de64e1863 |
(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 ofAbstractMultiParentHook
for the plugin in question. Specifically,MultiParentCompileHook.isSnapshotMultiParentPlugin
returnsfalse
, so the defaultprocess-test-classes
goal is used instead ofMultiParentCompileHook
changing the goal toinstall
, which fails (JENKINS-62658 describes the issue withprocess-test-classes
more generally).The problem seems to be that
AbstractMultiParentHook
intentionally doesn't do anything whenlocalCheckoutDir
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
.The text was updated successfully, but these errors were encountered: