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

os-dependent classifiers are flattened #52

Open
nikola-da opened this issue Aug 4, 2017 · 3 comments
Open

os-dependent classifiers are flattened #52

nikola-da opened this issue Aug 4, 2017 · 3 comments

Comments

@nikola-da
Copy link

Hi all,

I came to this plugin through https://maven.apache.org/maven-ci-friendly.html as I wanted to simplify our build process.

However, I noticed that some of the properties are not flattened correctly. Namely, dependencies with <classifier> set to ${os.detected.name} are being flattened to values present on the build system, instead of being left as-is. Is there a way to prevent this behaviour?

If I use <flattenMode>minimum</flattenMode> (or bom), then dependency's classifier in the parent pom is left as-is, but not in the children pom, where it is flattened.

Thanks!

@nikola-da nikola-da reopened this Aug 7, 2017
@khmarbaise
Copy link
Member

Can you please make a full working example project on github...

@nikola-da
Copy link
Author

Kein Problem! 😊

Example is at https://github.com/nikola-da/flatten-maven-plugin-example
There are two tags: working and not-working. Please check out first working, run mvn clean install and inspect ~/.m2/repository/co/knezevic. You'll see that moduleb/....pom contains a reference to ${os.detected.name}. If you run the same command with a checkout of not-working, you'll see that classifier in moduleb for modulea has been set to a particular value of your system.

@hohwille
Copy link
Member

This issue is similar to #57
I would agree and consider this as a sort of bug.
Still my question is which are properties were users expect flattening and which should not be interpolated by flatten?

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

No branches or pull requests

3 participants