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

[JENKINS-66247] Do not override the version of annotation-indexer from core #426

Merged
merged 1 commit into from
Jul 29, 2021

Conversation

jglick
Copy link
Member

@jglick jglick commented Jul 29, 2021

#424 should not have overridden the version of this library. See jenkinsci/pipeline-model-definition-plugin#443.

For now, just run ITs against a version of core containing jenkinsci/jenkins#5208. The better fix will be to make access-modifier-checker capable of reading both the old and new index location.

@jglick jglick added the bug label Jul 29, 2021
@jglick jglick requested review from timja, basil and bitwiseman July 29, 2021 19:45
@MarkEWaite
Copy link
Contributor

MarkEWaite commented Jul 29, 2021

Git plugin 4.8.0 was released with plugin pom 4.20. Git plugin 4.8.1 was released with plugin pom 4.21. Git client plugin 3.9.0 was released with plugin pom 4.21.

When I try to update them to use plugin pom 4.23, the CI job fails with access modifier errors for the Messages class. See https://ci.jenkins.io/job/Plugins/job/git-plugin/job/PR-1122/1/console and https://ci.jenkins.io/job/Plugins/job/git-client-plugin/view/change-requests/job/PR-729/1/console for examples of the failures.

I believe that it is legal to access the Messages class, at least according to https://www.jenkins.io/doc/developer/internationalization/prepare-a-java-source-file/#step-4-reference-messages-strings and https://www.jenkins.io/doc/developer/plugin-development/dependencies-and-class-loading/#views-and-other-resources . If not, can you suggest the correct change to allow those plugins to use plugin pom 4.23?

@bitwiseman
Copy link
Contributor

Same on scm-api-plugin: jenkinsci/scm-api-plugin#89

@jglick
Copy link
Member Author

jglick commented Jul 30, 2021

It is legal to access Messages from inside your own plugin of course. Not from other plugins. I will try to figure out what is going on here.

@jglick
Copy link
Member Author

jglick commented Jul 30, 2021

Will track in #428. Sorry for the regressions piled on top of regressions!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants