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

Avoid resources filtering warning #543

Merged
merged 1 commit into from
Oct 6, 2024
Merged

Conversation

Bananeweizen
Copy link
Contributor

Since maven-resources 3.2.0 the encoding shall be specified explicitly. Avoids this console output:

[INFO] The encoding used to copy filtered properties files have not been set. This means that the same encoding will be used to copy filtered properties files as when copying other filtered resources. This might not be what you want! Run your build with --debug to see which files might be affected. Read more at
https://maven.apache.org/plugins/maven-resources-plugin/examples/filtering-properties-files.html

Since maven-resources 3.2.0 the encoding shall be specified explicitly.
Avoids this console output:

[INFO] The encoding used to copy filtered properties files have not been
set. This means that the same encoding will be used to copy filtered
properties files as when copying other filtered resources. This might
not be what you want! Run your build with --debug to see which files
might be affected. Read more at
https://maven.apache.org/plugins/maven-resources-plugin/examples/filtering-properties-files.html
@hboutemy hboutemy merged commit 072dcd7 into CycloneDX:master Oct 6, 2024
2 of 3 checks passed
@hboutemy hboutemy added the bug label Oct 6, 2024
@hboutemy
Copy link
Contributor

hboutemy commented Oct 6, 2024

good improvement, thanks

@Bananeweizen Bananeweizen deleted the warning branch October 7, 2024 07:00
@hboutemy hboutemy added build Build improvement/fix and removed bug labels Oct 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Build improvement/fix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants