Skip to content

Improve warning message if multi-release JARs are not supported #1833

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

Merged
merged 1 commit into from
Oct 5, 2023

Conversation

Flowdalic
Copy link
Contributor

This is a follow up from my pervious PR [1]. Unfortunatly, it only occured to me after it was merged that the warning message is incomplete. In my case it was not the runtime environment which did not support multi-release JARs, but the build system. This adjusts the message accordingly.

1: #1760

This is a follow up from my pervious PR [1]. Unfortunatly, it only
occured to me after it was merged that the warning message is
incomplete. In my case it was not the runtime environment which did
not support multi-release JARs, but the build system. This adjusts the
message accordingly.

1: apache#1760
Copy link
Contributor

@ppkarwasz ppkarwasz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Flowdalic,

Thanks, there is nothing more annoying than error messages that point you in the wrong direction.

@ppkarwasz ppkarwasz added this to the 2.21.0 milestone Oct 5, 2023
@vy vy added the enhancement Additions or updates to features label Oct 5, 2023
@vy vy merged commit a1400fd into apache:2.x Oct 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Additions or updates to features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants