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

IVY-1566 - Store External form of URL in cached files #59

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
4 participants
@apupier
Contributor

apupier commented Aug 10, 2017

So hard to configure the project in my IvyIDE, cannot launch test individually so I abandoned the to search how to write a test.

I tested manually in my use case and I don't have anymore MalformedURlException

the ivydata*.properties now contaisn always "file:"

artifact\:ivy\#ivy\#xml\#-455539479.location=file\:C\:/Users/Aurelien Pupier/.m2/repository/org/apache/camel/camel/2.17.2/camel-2.17.2.pom
resolver=localm2
artifact\:ivy\#ivy\#xml\#-455539479.is-local=true
artifact\:ivy\#ivy\#xml\#-455539479.original=artifact\:camel\#pom.original\#pom\#1474703200
artifact\:camel\#pom.original\#pom\#1474703200.exists=true
artifact\:ivy\#ivy\#xml\#-455539479.exists=true
artifact.resolver=localm2
artifact\:camel\#pom.original\#pom\#1474703200.is-local=true
artifact\:camel\#pom.original\#pom\#1474703200.location=file\:C\:/Users/Aurelien Pupier/.m2/repository/org/apache/camel/camel/2.17.2/camel-2.17.2.pom
artifact\:camel\#pom.original\#pom\#1474703200.original=artifact\:camel\#pom.original\#pom\#1474703200
@apupier

This comment has been minimized.

Show comment
Hide comment
@apupier

apupier Aug 11, 2017

Contributor

for the Jenkins error:

ERROR: Step ‘Publish JUnit test result report’ failed: No test report files were found. Configuration error?

but tests were launched and are successful

Contributor

apupier commented Aug 11, 2017

for the Jenkins error:

ERROR: Step ‘Publish JUnit test result report’ failed: No test report files were found. Configuration error?

but tests were launched and are successful

@twogee

This comment has been minimized.

Show comment
Hide comment
@twogee

twogee Aug 11, 2017

Contributor

If I'm not mistaken, one of the builds for #58 was aborted by @nlalevee -- something wrong with Jenkins?

Contributor

twogee commented Aug 11, 2017

If I'm not mistaken, one of the builds for #58 was aborted by @nlalevee -- something wrong with Jenkins?

@nlalevee

This comment has been minimized.

Show comment
Hide comment
@nlalevee

nlalevee Aug 11, 2017

Member

Yep, I fixed the configuration, I tried to launch a rebuild for this PR, but it seems that rebuild on Jenkins doesn't take into account the PR, so I aborted it.

Member

nlalevee commented Aug 11, 2017

Yep, I fixed the configuration, I tried to launch a rebuild for this PR, but it seems that rebuild on Jenkins doesn't take into account the PR, so I aborted it.

@apupier

This comment has been minimized.

Show comment
Hide comment
@apupier

apupier Aug 11, 2017

Contributor

is there a specific comment specified to retrigger the build?

Contributor

apupier commented Aug 11, 2017

is there a specific comment specified to retrigger the build?

@twogee

This comment has been minimized.

Show comment
Hide comment
@twogee

twogee Aug 19, 2017

Contributor

Please update your local repo, rebase and force push -- new commit will start a new build.

Contributor

twogee commented Aug 19, 2017

Please update your local repo, rebase and force push -- new commit will start a new build.

@asfgit asfgit closed this in 0cdcb1f Aug 28, 2017

@jaikiran

This comment has been minimized.

Show comment
Hide comment
@jaikiran

jaikiran Aug 28, 2017

Contributor

Thank you for the PR and sorry it took a while to merge this. I have included the change you suggested in this PR with a minor change and merged it upstream.

Contributor

jaikiran commented Aug 28, 2017

Thank you for the PR and sorry it took a while to merge this. I have included the change you suggested in this PR with a minor change and merged it upstream.

@apupier apupier deleted the apupier:IVY-1566-AvoidStoringPathWhichAreNotUrl branch Aug 28, 2017

@apupier

This comment has been minimized.

Show comment
Hide comment
@apupier

apupier Aug 28, 2017

Contributor

thanks @jaikiran do you have an Estimated Time for next release containing this bugfix?

Contributor

apupier commented Aug 28, 2017

thanks @jaikiran do you have an Estimated Time for next release containing this bugfix?

@twogee

This comment has been minimized.

Show comment
Hide comment
@twogee

twogee Aug 28, 2017

Contributor

This is a bit off topic, but why are Jenkins builds still failing to find JaCoCo?

Contributor

twogee commented Aug 28, 2017

This is a bit off topic, but why are Jenkins builds still failing to find JaCoCo?

@jaikiran

This comment has been minimized.

Show comment
Hide comment
@jaikiran

jaikiran Aug 28, 2017

Contributor

thanks @jaikiran do you have an Estimated Time for next release containing this bugfix?

@apupier - the plan is to have it soon (no specific date yet). We are almost at the last few tasks/bug fixes we have in mind. A more concrete date will be discussed in the ant dev mailing list http://ant.apache.org/ivy/mailing-lists.html

If you want to give it a try with our nightly builds, it's available as noted in this mail https://www.mail-archive.com/ivy-user@ant.apache.org/msg06252.html

Contributor

jaikiran commented Aug 28, 2017

thanks @jaikiran do you have an Estimated Time for next release containing this bugfix?

@apupier - the plan is to have it soon (no specific date yet). We are almost at the last few tasks/bug fixes we have in mind. A more concrete date will be discussed in the ant dev mailing list http://ant.apache.org/ivy/mailing-lists.html

If you want to give it a try with our nightly builds, it's available as noted in this mail https://www.mail-archive.com/ivy-user@ant.apache.org/msg06252.html

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