I don't see any issues with the changes.
Thanks,
Jane
Shalini Muthukrishnan wrote:
> Please review this pom.xml so i can commit the changes.
>
> Thanks,
> Shalini.
>
> Shalini Muthukrishnan wrote:
>> Hi Jerome, Sahoo, Jane,
>>
>> Please review the pom.xml diffs for jdbc/jdbc-ra/jdbc-ra-distribution
>> module.
>>
>> The changes have been made so as to exclude the jdbc-core.jar,
>> jdbc30.jar and jdbc40.jar from the glassfish/modules directory. The
>> jdbc-ra.zip that is built from jdbc-ra-distribution includes files
>> that are to be placed under glassfish/lib/install/applications.
>> jdbc-ra-distribution is a distribution-fragment.
>>
>> Please provide your comments by EOD 04/08.
>>
>> Thanks,
>> Shalini.
>>
>> Index: jdbc/jdbc-ra/jdbc-ra-distribution/pom.xml
>> ===================================================================
>> --- jdbc/jdbc-ra/jdbc-ra-distribution/pom.xml (revision 25964)
>> +++ jdbc/jdbc-ra/jdbc-ra-distribution/pom.xml (working copy)
>> @@ -54,18 +54,21 @@
>> <groupId>org.glassfish.jdbc.jdbc-ra.jdbc-core</groupId>
>> <artifactId>jdbc-core</artifactId>
>> <version>${project.parent.version}</version>
>> + <scope>provided</scope>
>> </dependency>
>>
>> <dependency>
>> <groupId>org.glassfish.jdbc.jdbc-ra.jdbc40</groupId>
>> <artifactId>jdbc40</artifactId>
>> <version>${project.parent.version}</version>
>> + <scope>provided</scope>
>> </dependency>
>>
>> <dependency>
>> <groupId>org.glassfish.jdbc.jdbc-ra.jdbc30</groupId>
>> <artifactId>jdbc30</artifactId>
>> <version>${project.parent.version}</version>
>> + <scope>provided</scope>
>> </dependency>
>>
>> </dependencies>
>>
>>
>>
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: dev-unsubscribe_at_glassfish.dev.java.net
>> For additional commands, e-mail: dev-help_at_glassfish.dev.java.net
>>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe_at_glassfish.dev.java.net
> For additional commands, e-mail: dev-help_at_glassfish.dev.java.net
>