[ http://jira.codehaus.org/browse/MRM-155?page=comments#action_73859 ]
Brett Porter commented on MRM-155:
--
I've added an additional test for the reverse case (legacy managed, default
remote) and noticed this fix wasn't sufficient. I've corrected
[ http://jira.codehaus.org/browse/MRM-155?page=comments#action_73858 ]
Brett Porter commented on MRM-155:
--
ok, found and fixed the problem.
> When proxy is used by maven1artifact are stored in managed repo using legacy
> layout.
> ---
[ http://jira.codehaus.org/browse/MRM-155?page=comments#action_73856 ]
Brett Porter commented on MRM-155:
--
this:
if ( path.endsWith( ".md5" ) || path.endsWith( ".sha1" ) )
{
// always read from the managed repository,
[ http://jira.codehaus.org/browse/MRM-155?page=comments#action_73855 ]
Brett Porter commented on MRM-155:
--
I removed:
if ( target.exists() )
{
break;
}
[ http://jira.codehaus.org/browse/MRM-155?page=comments#action_73854 ]
Brett Porter commented on MRM-155:
--
unfortunately this broke existing tests. Please make sure to run them all again
before submitting.
I'm going to rework it to get that fix
[ http://jira.codehaus.org/browse/MRM-155?page=comments#action_73533 ]
Brett Porter commented on MRM-155:
--
Nicolas, we've started a policy of only applying patches with test cases to
attempt to boost and maintain our test coverage. Please resubm
[ http://jira.codehaus.org/browse/MRM-155?page=comments#action_73463 ]
nicolas de loof commented on MRM-155:
-
The attached LegacyArtifactDiscoverer is an error. It is not required to solve
this issue and introduced a test failure..
> When pr
[ http://jira.codehaus.org/browse/MRM-155?page=comments#action_73274 ]
Brett Porter commented on MRM-155:
--
could you include tests that produce the problem that this fixes too, please?
Also, I'd prefer one diff using svn diff that marks out each
