You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ant.apache.org by Jaikiran Pai <ja...@apache.org> on 2018/08/29 06:34:29 UTC
Re: ant-ivy git commit: Update the release notes
More of a FYI - I'm still not convinced that my fix for this handles all
the necessary cases (looks like the ArtifactOrigin#location gets used in
various different ways), so I may either revert back my changes or
decide to change it in a different way. So right now, in its current
form, my changes aren't a fix.
-Jaikiran
On 29/08/18 11:34 AM, gintas@apache.org wrote:
> Repository: ant-ivy
> Updated Branches:
> refs/heads/master d976a4a27 -> fd81f4461
>
>
> Update the release notes
>
> Project: http://git-wip-us.apache.org/repos/asf/ant-ivy/repo
> Commit: http://git-wip-us.apache.org/repos/asf/ant-ivy/commit/fd81f446
> Tree: http://git-wip-us.apache.org/repos/asf/ant-ivy/tree/fd81f446
> Diff: http://git-wip-us.apache.org/repos/asf/ant-ivy/diff/fd81f446
>
> Branch: refs/heads/master
> Commit: fd81f44619b05a176ecbf4ff1499d64b39251520
> Parents: d976a4a
> Author: Gintas Grigelionis <gi...@apache.org>
> Authored: Wed Aug 29 08:03:13 2018 +0200
> Committer: Gintas Grigelionis <gi...@apache.org>
> Committed: Wed Aug 29 08:05:12 2018 +0200
>
> ----------------------------------------------------------------------
> asciidoc/release-notes.adoc | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
> ----------------------------------------------------------------------
>
>
> http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/fd81f446/asciidoc/release-notes.adoc
> ----------------------------------------------------------------------
> diff --git a/asciidoc/release-notes.adoc b/asciidoc/release-notes.adoc
> index cc53bb3..efa7057 100644
> --- a/asciidoc/release-notes.adoc
> +++ b/asciidoc/release-notes.adoc
> @@ -85,6 +85,7 @@ For details about the following changes, check our JIRA install at link:https://
> - FIX: Don't throw a CircularDependencyException when parsing an import scoped dependency in dependencyManagement section of a pom (jira:IVY-1588[])
> - FIX: Respect exclude regardless of resolution order (jira:IVY-1486[]) (thanks to David Turner)
> - FIX: ModuleDescriptorMemoryCache didn't detect outdated entries when Ivy file was updated in the cache by another process
> +- FIX: Store ArtifactOrigin's location as a URL
>
> - IMPROVEMENT: Throw an IllegalStateException when retrieving the resolutionCacheRoot on the DefaultResolutionCacheManager if the basedir (or IvySettings) is not set (jira:IVY-1482[])
> - IMPROVEMENT: Optimization: limit the revision numbers scanned if revision prefix is specified (Thanks to Ernestas Vaiciukevičius)
> @@ -181,7 +182,6 @@ Here is the list of people who have contributed source code and documentation up
> * Tobias Himstedt
> * Achim Huegen
> * Pierre Hägnestrand
> -* Ilya
> * Matt Inger
> * Anders Jacobsson
> * Anders Janmyr
> @@ -196,6 +196,7 @@ Here is the list of people who have contributed source code and documentation up
> * Sebastian Krueger
> * Thomas Kurpick
> * Costin Leau
> +* Ilya Leoshkevich
> * Tat Leung
> * Antoine Levy-Lambert
> * Tony Likhite
>
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org
For additional commands, e-mail: dev-help@ant.apache.org
Re: ant-ivy git commit: Update the release notes
Posted by Jaikiran Pai <ja...@apache.org>.
Hi Maarten,
Agreed - the change(s) should be done taking into account any existing
caches (from previous version of Ivy). As of now, I have reverted my
changes which had introduced the test failures. To fix properly, it will
require a couple of hours of dedicated work to make sure all cases are
covered. However, I haven't been able to focus on it at a stretch and
would like to finish off some pending junitlauncher task work in Ant
project, before I can get to this. Hence I decided to undo my changes
for now and leave it in a state as of our latest released version.
-Jaikiran
On 29/08/18 8:27 PM, Maarten Coene wrote:
> Without knowing the details of the change, one thing we should take care about is that older Ivy versions should still be able to read that property properly.
> Maarten
>
> Op woensdag 29 augustus 2018 16:48:01 CEST schreef Gintautas Grigelionis <g....@gmail.com>:
>
> I like the idea, though. One thing that should be investigated further is
> places where location is obtained by getResource().getName()
> AFAICS that happens in CacheResolver (deprecated), BasicResolver (where
> also a Resource is constructed from location), and
> DefaultRepositoryCacheManager. There's no uniformity in
> Resource-implementing classes either, sometimes getName() works on an URI,
> sometimes a path string.
>
> Gintas
>
> On Wed, 29 Aug 2018 at 08:34, Jaikiran Pai <ja...@apache.org> wrote:
>
>> More of a FYI - I'm still not convinced that my fix for this handles all
>> the necessary cases (looks like the ArtifactOrigin#location gets used in
>> various different ways), so I may either revert back my changes or
>> decide to change it in a different way. So right now, in its current
>> form, my changes aren't a fix.
>>
>> -Jaikiran
>>
>>
>> On 29/08/18 11:34 AM, gintas@apache.org wrote:
>>> Repository: ant-ivy
>>> Updated Branches:
>>> refs/heads/master d976a4a27 -> fd81f4461
>>>
>>>
>>> Update the release notes
>>>
>>> Project: http://git-wip-us.apache.org/repos/asf/ant-ivy/repo
>>> Commit: http://git-wip-us.apache.org/repos/asf/ant-ivy/commit/fd81f446
>>> Tree: http://git-wip-us.apache.org/repos/asf/ant-ivy/tree/fd81f446
>>> Diff: http://git-wip-us.apache.org/repos/asf/ant-ivy/diff/fd81f446
>>>
>>> Branch: refs/heads/master
>>> Commit: fd81f44619b05a176ecbf4ff1499d64b39251520
>>> Parents: d976a4a
>>> Author: Gintas Grigelionis <gi...@apache.org>
>>> Authored: Wed Aug 29 08:03:13 2018 +0200
>>> Committer: Gintas Grigelionis <gi...@apache.org>
>>> Committed: Wed Aug 29 08:05:12 2018 +0200
>>>
>>> ----------------------------------------------------------------------
>>> asciidoc/release-notes.adoc | 3 ++-
>>> 1 file changed, 2 insertions(+), 1 deletion(-)
>>> ----------------------------------------------------------------------
>>>
>>>
>>>
>> http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/fd81f446/asciidoc/release-notes.adoc
>>> ----------------------------------------------------------------------
>>> diff --git a/asciidoc/release-notes.adoc b/asciidoc/release-notes.adoc
>>> index cc53bb3..efa7057 100644
>>> --- a/asciidoc/release-notes.adoc
>>> +++ b/asciidoc/release-notes.adoc
>>> @@ -85,6 +85,7 @@ For details about the following changes, check our
>> JIRA install at link:https://
>>> - FIX: Don't throw a CircularDependencyException when parsing an import
>> scoped dependency in dependencyManagement section of a pom (jira:IVY-1588[])
>>> - FIX: Respect exclude regardless of resolution order (jira:IVY-1486[])
>> (thanks to David Turner)
>>> - FIX: ModuleDescriptorMemoryCache didn't detect outdated entries when
>> Ivy file was updated in the cache by another process
>>> +- FIX: Store ArtifactOrigin's location as a URL
>>>
>>> - IMPROVEMENT: Throw an IllegalStateException when retrieving the
>> resolutionCacheRoot on the DefaultResolutionCacheManager if the basedir (or
>> IvySettings) is not set (jira:IVY-1482[])
>>> - IMPROVEMENT: Optimization: limit the revision numbers scanned if
>> revision prefix is specified (Thanks to Ernestas Vaiciukevičius)
>>> @@ -181,7 +182,6 @@ Here is the list of people who have contributed
>> source code and documentation up
>>> * Tobias Himstedt
>>> * Achim Huegen
>>> * Pierre Hägnestrand
>>> -* Ilya
>>> * Matt Inger
>>> * Anders Jacobsson
>>> * Anders Janmyr
>>> @@ -196,6 +196,7 @@ Here is the list of people who have contributed
>> source code and documentation up
>>> * Sebastian Krueger
>>> * Thomas Kurpick
>>> * Costin Leau
>>> +* Ilya Leoshkevich
>>> * Tat Leung
>>> * Antoine Levy-Lambert
>>> * Tony Likhite
>>>
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org
>> For additional commands, e-mail: dev-help@ant.apache.org
>>
>>
>
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org
For additional commands, e-mail: dev-help@ant.apache.org
Re: ant-ivy git commit: Update the release notes
Posted by Maarten Coene <ma...@yahoo.com.INVALID>.
Without knowing the details of the change, one thing we should take care about is that older Ivy versions should still be able to read that property properly.
Maarten
Op woensdag 29 augustus 2018 16:48:01 CEST schreef Gintautas Grigelionis <g....@gmail.com>:
I like the idea, though. One thing that should be investigated further is
places where location is obtained by getResource().getName()
AFAICS that happens in CacheResolver (deprecated), BasicResolver (where
also a Resource is constructed from location), and
DefaultRepositoryCacheManager. There's no uniformity in
Resource-implementing classes either, sometimes getName() works on an URI,
sometimes a path string.
Gintas
On Wed, 29 Aug 2018 at 08:34, Jaikiran Pai <ja...@apache.org> wrote:
> More of a FYI - I'm still not convinced that my fix for this handles all
> the necessary cases (looks like the ArtifactOrigin#location gets used in
> various different ways), so I may either revert back my changes or
> decide to change it in a different way. So right now, in its current
> form, my changes aren't a fix.
>
> -Jaikiran
>
>
> On 29/08/18 11:34 AM, gintas@apache.org wrote:
> > Repository: ant-ivy
> > Updated Branches:
> > refs/heads/master d976a4a27 -> fd81f4461
> >
> >
> > Update the release notes
> >
> > Project: http://git-wip-us.apache.org/repos/asf/ant-ivy/repo
> > Commit: http://git-wip-us.apache.org/repos/asf/ant-ivy/commit/fd81f446
> > Tree: http://git-wip-us.apache.org/repos/asf/ant-ivy/tree/fd81f446
> > Diff: http://git-wip-us.apache.org/repos/asf/ant-ivy/diff/fd81f446
> >
> > Branch: refs/heads/master
> > Commit: fd81f44619b05a176ecbf4ff1499d64b39251520
> > Parents: d976a4a
> > Author: Gintas Grigelionis <gi...@apache.org>
> > Authored: Wed Aug 29 08:03:13 2018 +0200
> > Committer: Gintas Grigelionis <gi...@apache.org>
> > Committed: Wed Aug 29 08:05:12 2018 +0200
> >
> > ----------------------------------------------------------------------
> > asciidoc/release-notes.adoc | 3 ++-
> > 1 file changed, 2 insertions(+), 1 deletion(-)
> > ----------------------------------------------------------------------
> >
> >
> >
> http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/fd81f446/asciidoc/release-notes.adoc
> > ----------------------------------------------------------------------
> > diff --git a/asciidoc/release-notes.adoc b/asciidoc/release-notes.adoc
> > index cc53bb3..efa7057 100644
> > --- a/asciidoc/release-notes.adoc
> > +++ b/asciidoc/release-notes.adoc
> > @@ -85,6 +85,7 @@ For details about the following changes, check our
> JIRA install at link:https://
> > - FIX: Don't throw a CircularDependencyException when parsing an import
> scoped dependency in dependencyManagement section of a pom (jira:IVY-1588[])
> > - FIX: Respect exclude regardless of resolution order (jira:IVY-1486[])
> (thanks to David Turner)
> > - FIX: ModuleDescriptorMemoryCache didn't detect outdated entries when
> Ivy file was updated in the cache by another process
> > +- FIX: Store ArtifactOrigin's location as a URL
> >
> > - IMPROVEMENT: Throw an IllegalStateException when retrieving the
> resolutionCacheRoot on the DefaultResolutionCacheManager if the basedir (or
> IvySettings) is not set (jira:IVY-1482[])
> > - IMPROVEMENT: Optimization: limit the revision numbers scanned if
> revision prefix is specified (Thanks to Ernestas Vaiciukevičius)
> > @@ -181,7 +182,6 @@ Here is the list of people who have contributed
> source code and documentation up
> > * Tobias Himstedt
> > * Achim Huegen
> > * Pierre Hägnestrand
> > -* Ilya
> > * Matt Inger
> > * Anders Jacobsson
> > * Anders Janmyr
> > @@ -196,6 +196,7 @@ Here is the list of people who have contributed
> source code and documentation up
> > * Sebastian Krueger
> > * Thomas Kurpick
> > * Costin Leau
> > +* Ilya Leoshkevich
> > * Tat Leung
> > * Antoine Levy-Lambert
> > * Tony Likhite
> >
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org
> For additional commands, e-mail: dev-help@ant.apache.org
>
>
Re: ant-ivy git commit: Update the release notes
Posted by Gintautas Grigelionis <g....@gmail.com>.
I like the idea, though. One thing that should be investigated further is
places where location is obtained by getResource().getName()
AFAICS that happens in CacheResolver (deprecated), BasicResolver (where
also a Resource is constructed from location), and
DefaultRepositoryCacheManager. There's no uniformity in
Resource-implementing classes either, sometimes getName() works on an URI,
sometimes a path string.
Gintas
On Wed, 29 Aug 2018 at 08:34, Jaikiran Pai <ja...@apache.org> wrote:
> More of a FYI - I'm still not convinced that my fix for this handles all
> the necessary cases (looks like the ArtifactOrigin#location gets used in
> various different ways), so I may either revert back my changes or
> decide to change it in a different way. So right now, in its current
> form, my changes aren't a fix.
>
> -Jaikiran
>
>
> On 29/08/18 11:34 AM, gintas@apache.org wrote:
> > Repository: ant-ivy
> > Updated Branches:
> > refs/heads/master d976a4a27 -> fd81f4461
> >
> >
> > Update the release notes
> >
> > Project: http://git-wip-us.apache.org/repos/asf/ant-ivy/repo
> > Commit: http://git-wip-us.apache.org/repos/asf/ant-ivy/commit/fd81f446
> > Tree: http://git-wip-us.apache.org/repos/asf/ant-ivy/tree/fd81f446
> > Diff: http://git-wip-us.apache.org/repos/asf/ant-ivy/diff/fd81f446
> >
> > Branch: refs/heads/master
> > Commit: fd81f44619b05a176ecbf4ff1499d64b39251520
> > Parents: d976a4a
> > Author: Gintas Grigelionis <gi...@apache.org>
> > Authored: Wed Aug 29 08:03:13 2018 +0200
> > Committer: Gintas Grigelionis <gi...@apache.org>
> > Committed: Wed Aug 29 08:05:12 2018 +0200
> >
> > ----------------------------------------------------------------------
> > asciidoc/release-notes.adoc | 3 ++-
> > 1 file changed, 2 insertions(+), 1 deletion(-)
> > ----------------------------------------------------------------------
> >
> >
> >
> http://git-wip-us.apache.org/repos/asf/ant-ivy/blob/fd81f446/asciidoc/release-notes.adoc
> > ----------------------------------------------------------------------
> > diff --git a/asciidoc/release-notes.adoc b/asciidoc/release-notes.adoc
> > index cc53bb3..efa7057 100644
> > --- a/asciidoc/release-notes.adoc
> > +++ b/asciidoc/release-notes.adoc
> > @@ -85,6 +85,7 @@ For details about the following changes, check our
> JIRA install at link:https://
> > - FIX: Don't throw a CircularDependencyException when parsing an import
> scoped dependency in dependencyManagement section of a pom (jira:IVY-1588[])
> > - FIX: Respect exclude regardless of resolution order (jira:IVY-1486[])
> (thanks to David Turner)
> > - FIX: ModuleDescriptorMemoryCache didn't detect outdated entries when
> Ivy file was updated in the cache by another process
> > +- FIX: Store ArtifactOrigin's location as a URL
> >
> > - IMPROVEMENT: Throw an IllegalStateException when retrieving the
> resolutionCacheRoot on the DefaultResolutionCacheManager if the basedir (or
> IvySettings) is not set (jira:IVY-1482[])
> > - IMPROVEMENT: Optimization: limit the revision numbers scanned if
> revision prefix is specified (Thanks to Ernestas Vaiciukevičius)
> > @@ -181,7 +182,6 @@ Here is the list of people who have contributed
> source code and documentation up
> > * Tobias Himstedt
> > * Achim Huegen
> > * Pierre Hägnestrand
> > -* Ilya
> > * Matt Inger
> > * Anders Jacobsson
> > * Anders Janmyr
> > @@ -196,6 +196,7 @@ Here is the list of people who have contributed
> source code and documentation up
> > * Sebastian Krueger
> > * Thomas Kurpick
> > * Costin Leau
> > +* Ilya Leoshkevich
> > * Tat Leung
> > * Antoine Levy-Lambert
> > * Tony Likhite
> >
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org
> For additional commands, e-mail: dev-help@ant.apache.org
>
>