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

IT showing inability to resolve versions coming from a profile #118

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jtnord
Copy link

@jtnord jtnord commented Nov 22, 2019

Integration test showing #117

@hohwille
Copy link
Member

hohwille commented Dec 17, 2019

@jtnord thanks for providing this PR with an IT to reveal that something is wrong here.

[INFO] artifact org.codehaus.mojo.flatten-maven-plugin:1.2.0-SNAPSHOT: checking for updates from sonatype-apache
[INFO] ..FAILED (4.1 s)
[INFO]   The build exited with code 1. See /home/travis/build/mojohaus/flatten-maven-plugin/target/it/dep-with-version-from-profile-in-parent/build.log for details.

Log/travis-ci is not very helpful here. So you assume that we analyze this by cloning your fork and then fixing the bug? Just want to get sure as I am hoplessly overloaded so hopefully @lasselindqvist might be able to have a look...
https://groups.google.com/forum/#!topic/mojohaus-dev/3r0m6im5M5A

UPDATE: Got the intension from your issue #117 so thanks also for that. Please see my comment also there.

@jtnord
Copy link
Author

jtnord commented Dec 18, 2019

Log/travis-ci is not very helpful here

invoker.streamLogs may help with that (most likely in a profile to make local dev easier).

full build.log

but relevant part is

Caused by: org.apache.maven.plugin.MojoExecutionException: 1 problem was encountered while building the effective model for org.codehaus.mojo.flatten.its:dep-with-version-from-profile-in-parent:0.0.1-SNAPSHOT
[ERROR] 'dependencies.dependency.version' for org.codehaus.mojo.flatten.its:dep:jar is missing. @ 

This PR is here so that if someone wants to look at the issue they have a starting point, I am not actively working on this.

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

Successfully merging this pull request may close these issues.

None yet

2 participants