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

Flatten maven plugin 1.1.x #102

Closed
wants to merge 3 commits into from

Conversation

nruzic
Copy link

@nruzic nruzic commented Jul 10, 2019

resolveCiFriendliesOnly mode is not perfect currently.
It should interpolate version and parent elements only.
I have discovered that if you have some profiles active during build, flatten-maven-plugin in resolveCiFriendliesOnly mode will interpolate some placeholders in active profiles (e.g. if you use mvn clean install -PsomeProfile).
This is not desirable, at least for me :-)
I have introduced alternative solution for resolving original pom model in FlattenMojo so that no placeholders are resolved except version and parent.
Solution is not perfect, but it can be used as guide for more permanent solution to the problem.
Test case is included which illustrates the problem.

@famod
Copy link
Contributor

famod commented Sep 5, 2019

oraclejdk8 problem on Travis should vanish once #106 is merged.

@hohwille
Copy link
Member

#106 is merged. Restarting the travis build for PRs does not help. Seems all need to merge that back into their forks to update PRs properly.

@hohwille
Copy link
Member

PR never worked in travis and has merge conflicts. Will it still be maintained (=merged)?

@hohwille
Copy link
Member

Replaced by PR #114

@hohwille hohwille closed this Dec 17, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants