Mailing-List: contact dev-help@maven.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Maven Developers List" <dev@maven.apache.org>
Subject: Re: maven-wagon git commit: [MRESOLVER-9] DefaultDependencyCollector
 does not correctly handle dependency management.
Reply-To: Maven Developers List <dev@maven.apache.org>
References: <087d915b797a40fcb7b41c728520f06b@git.apache.org>
 <op.ysgz2lnikdkhrr@desktop-2khsk44> <op.yshdytq5kdkhrr@desktop-2khsk44>
 <144d382f-f71f-61d8-073a-dbb358730dbf@apache.org>
 <c6d089c1-080d-304d-10a0-0b175daa46d4@schulte.it>
To: Maven Developers List <dev@maven.apache.org>
From: Christian Schulte <cs@schulte.it>
Message-ID: <f0dd2d86-a338-3742-6fab-d439232dee5e@schulte.it>
Date: Thu, 15 Dec 2016 01:29:43 +0100
User-Agent: Mozilla/5.0 (X11; OpenBSD amd64; rv:45.0) Gecko/20100101
 Thunderbird/45.2.0
MIME-Version: 1.0
In-Reply-To: <c6d089c1-080d-304d-10a0-0b175daa46d4@schulte.it>
Content-Type: text/plain; charset=iso-8859-15
Content-Transfer-Encoding: 7bit
archived-at: Thu, 15 Dec 2016 00:32:12 -0000

Am 12/15/16 um 01:15 schrieb Christian Schulte:
> Am 12/14/16 um 23:29 schrieb Michael Osipov:
>> I just hit the same issue with the Maven Clean Plugin: 
>> plexus-container-default is missing on the classpath.
>>
>> I think that Christian does the right thing, either adjusting code to 
>> documentation or the other way around. We just need to agree what is the 
>> right thing and change the problematic spot accordingly.
>>
>> Don't having compile, runtime and test scoped transitive deps of a 
>> direct test dependency doesn't look right to me. There must be some 
>> reason why this decision was made eons ago.
> 
> That could be another bug.
> 
> A -> B (test) -> C (compile)
> 
> When we say "Test scope is not transitive", C should not be resolved as
> well. That's just another inconsistency. We have B in test scope and so
> all transitive dependencies of B implicitly will become test scope. That
> scope is not transitive, so C and all other transitive dependencies of B
> should not be resolved. It's really still inconsistent.

By this I mean: If the scope of C is not managed to 'test', it currently
will be resolved. It really shouldn't, following the 'Test scope is not
transitive'. I read that as: The test scope does not support transitive
dependencies.


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org
For additional commands, e-mail: dev-help@maven.apache.org