You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cocoon.apache.org by Grzegorz Kossakowski <gk...@apache.org> on 2007/08/07 21:30:13 UTC

Re: svn commit: r563610 - /cocoon/trunk/blocks/cocoon-serializers/cocoon-serializers-charsets/pom.xml

cziegeler@apache.org pisze:
> Author: cziegeler
> Date: Tue Aug  7 11:37:04 2007
> New Revision: 563610
> 
> URL: http://svn.apache.org/viewvc?view=rev&rev=563610
> Log:
> Add missing dependency.
> 
> Modified:
>     cocoon/trunk/blocks/cocoon-serializers/cocoon-serializers-charsets/pom.xml
> 
> Modified: cocoon/trunk/blocks/cocoon-serializers/cocoon-serializers-charsets/pom.xml
> URL: http://svn.apache.org/viewvc/cocoon/trunk/blocks/cocoon-serializers/cocoon-serializers-charsets/pom.xml?view=diff&rev=563610&r1=563609&r2=563610
> ==============================================================================
> --- cocoon/trunk/blocks/cocoon-serializers/cocoon-serializers-charsets/pom.xml (original)
> +++ cocoon/trunk/blocks/cocoon-serializers/cocoon-serializers-charsets/pom.xml Tue Aug  7 11:37:04 2007
> @@ -49,6 +49,10 @@
>    
>    <dependencies>
>      <dependency>
> +      <groupId>javax.servlet</groupId>
> +      <artifactId>servlet-api</artifactId>
> +    </dependency>


Shouldn't this dependency had scope "provided"?

-- 
Grzegorz Kossakowski
http://reflectingonthevicissitudes.wordpress.com/

Re: svn commit: r563610 - /cocoon/trunk/blocks/cocoon-serializers/cocoon-serializers-charsets/pom.xml

Posted by Carsten Ziegeler <cz...@apache.org>.
Grzegorz Kossakowski wrote

>>       <dependencies>
>>      <dependency>
>> +      <groupId>javax.servlet</groupId>
>> +      <artifactId>servlet-api</artifactId>
>> +    </dependency>
> 
> 
> Shouldn't this dependency had scope "provided"?
> 
Good catch, yes, I fixed this in our root pom, so the scope is now
"provided" for every use in one of our modules.

Carsten

-- 
Carsten Ziegeler
cziegeler@apache.org