dev@glassfish.java.net

Re: Request to review change to appclient/client/acc-config pom.xml

From: Jane Young <jane.young_at_oracle.com>
Date: Mon, 26 Jul 2010 13:47:50 -0700

Looks fine. Thanks.



Tim Quinn wrote:
> My earlier change to the pom to allow offline builds to success worked
> when run from the appclient/client/acc-config directory but not from
> any higher-level directory.
>
> This change works from any of those directories. (It specifies the
> full path instead of the relative path "temp" to the directory where
> the extracted DTDs reside.)
>
> Thanks.
>
> - Tim
>
> Index: appclient/client/acc-config/pom.xml
> ===================================================================
> --- appclient/client/acc-config/pom.xml (revision 39071)
> +++ appclient/client/acc-config/pom.xml (working copy)
> @@ -56,7 +56,7 @@
>
> <extracted-dtd-top-level-directory>${extracted-dtd-root}/glassfish</extracted-dtd-top-level-directory>
>
>
> <extracted-dtd-directory>${extracted-dtd-top-level-directory}/lib/dtds</extracted-dtd-directory>
>
>
> <config-dtd-file>sun-application-client-container_1_2.dtd</config-dtd-file>
>
> -
> <added-test-classpath>${extracted-dtd-root}</added-test-classpath>
> +
> <added-test-classpath>${project.build.directory}/../${extracted-dtd-root}</added-test-classpath>
>
> <!--
> <config-xsd-dir>src/main/resources</config-xsd-dir>
>
> <config-xsd-file>sun-application-client-container_1_2.xsd</config-xsd-file>
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe_at_glassfish.dev.java.net
> For additional commands, e-mail: dev-help_at_glassfish.dev.java.net
>