maven-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Brian E. Fox" <>
Subject RE: ScopeArtifactFilter
Date Tue, 03 Oct 2006 14:38:13 GMT
Well that's odd. In my case I'm using this for the dependency plugin but
I also want to support the other scopes. It means I have to write a
bunch of code around this existing class and I figured I couldn't be the
only one doing this. I'm not sure how adding:
>         else if ( DefaultArtifact.SCOPE_PROVIDED.equals( scope ) )
>         {
>             systemScope = false;
>             providedScope = true;
>             compileScope = false;
>             runtimeScope = false;
>             testScope = false;
>         }
>         else if ( DefaultArtifact.SCOPE_SYSTEM.equals( scope ) )
>         {
>             systemScope = true;
>             providedScope = false;
>             compileScope = false;
>             runtimeScope = false;
>             testScope = false;
>         }

Is a huge issue. What if it was an overloaded constructor with a flag to
say "yes I care about all the scopes"?

-----Original Message-----
From: Kenney Westerhof [] 
Sent: Tuesday, October 03, 2006 9:42 AM
To: Maven Developers List
Subject: Re: ScopeArtifactFilter

Brian E. Fox wrote:
> Is it correct that ScopeArtifactFilter effectively ignores provided 
> and system scopes in the includes check? By the code below, you can 
> see that only compile, runtime and test are supported.

Unfortunately, yes.

I've tried to get this changed a bunch of times in the past but it was

I think it's the responsibility of the recipient of those transitive
dependencies to decide wheter to use them or not, especially in the case
of 'provided' scope.
Provided for what? The target platform! Not eclipse or unit tests or the
java compiler (which usually fails because of a 'class bytes not found'

I was just thinking about this last night, funny it pops up again now :)

-- Kenney

To unsubscribe, e-mail: For additional
commands, e-mail:

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message