dev@glassfish.java.net

Re: [v3] pom review request - appclient/client/acc/pom.xml

From: Jane Young <Jane.Young_at_Sun.COM>
Date: Thu, 16 Jul 2009 15:08:21 -0700

At first I thought you may not need to define src/main/resources since
it's the maven default resources directory then I see that another
resources directory is defined for "target/dependency/glassfish/lib" so
that's the reason you have to define it.

The changes is fine.

Thanks,
Jane


Tim Quinn wrote:
> Hi.
>
> This is a simple one. I will be moving LocalStrings.properties from
> its old, original spot in src/main/java to src/main/resources where it
> belongs. So I can exclude the java source tree from the resources
> gathering and just grab the whole resources dir.
>
> Thanks.
>
> - Tim
>
>
> Index: appclient/client/acc/pom.xml
> ===================================================================
> --- appclient/client/acc/pom.xml (revision 29193)
> +++ appclient/client/acc/pom.xml (working copy)
> @@ -95,14 +95,10 @@
> <build>
> <resources>
> <resource>
> - <directory>src/main/java</directory>
> - <includes>
> - <include>**/*.properties</include>
> - </includes>
> + <directory>src/main/resources</directory>
> </resource>
> -
> <!--
> - The next resource definitions will include the DTDs and
> + The next resource definition will include the DTDs and
> schemas in the module so we can find them at runtime
> in the
> client. The dependency:unpack-dependency goal below will
> by default deposit the unpacked contents into
>