It'd be better to default to /dev/null or NUL: if the project doesn't
provide a license.
--
dIon Gillard, Multitask Consulting
Work: http://www.multitask.com.au
Developers: http://adslgateway.multitask.com.au/developers
James Taylor
<jtaylor@4lane To: Turbine Maven Developers List
.com> <[EMAIL PROTECTED]>
cc:
05/27/02 09:35 Subject: Re: [PATCH] Checkstyle fails if
LICENSE.txt is absent
AM
Please respond
to "Turbine
Maven
Developers
List"
But is it not reasonable to provide an empty file and use that if the
project does not define one?
On Sun, 2002-05-26 at 19:31, [EMAIL PROTECTED] wrote:
>
> Check the FAQ....
> See
>
http://jakarta.apache.org/turbine/maven/ref/faq.html#check-source-disable-header
>
> --
> dIon Gillard, Multitask Consulting
> Work: http://www.multitask.com.au
> Developers: http://adslgateway.multitask.com.au/developers
>
>
>
> St�phane MOR
> <stephane_listes To:
[EMAIL PROTECTED]
> @yahoo.fr> cc:
> Subject: [PATCH]
Checkstyle fails if LICENSE.txt is absent
> 05/27/02 07:03
> AM
> Please respond
> to "Turbine
> Maven Developers
> List"
>
>
>
>
>
>
> Hi,
>
> I noticed on new projects I started with Maven that, when there is no
> ${basedir}/LICENSE.txt, Checkstyle fails, throwing a
> "Unable to create a checker" error, causing the whole "maven:site" build
> to fail with a very limited information of why it
> failed.
>
> I tried several ways to come around this, setting headerFile to "" if
> the file didn't exist, creating another target depending on
> whether the file exists (very very bad solution, IMO), etc, which all
> failed.
>
> What I did is include a dummy LICENSE.txt in ${maven.home}/plugins/core/
> (only 2 empty lines at the top), and tell Checkstyle
> to use that file if the file is missing.
>
> The LICENSE.txt file is to be copied to src/templates/build/plugins/core/
.
>
> Here are the patches to do that.
>
> St�phane
>
> Index: src/templates/build/plugins/core/Control.vm
> ===================================================================
> RCS file:
>
/home/cvspublic/jakarta-turbine-maven/src/templates/build/plugins/core/Control.vm,v
>
> retrieving revision 1.1
> diff -u -r1.1 Control.vm
> --- src/templates/build/plugins/core/Control.vm 15 May 2002
> 19:51:11 -0000 1.1
> +++ src/templates/build/plugins/core/Control.vm 26 May 2002
> 20:29:35 -0000
> @@ -7,6 +7,7 @@
> ## -------------------------------------------------------
> $buildElements.add("plugins/$plugin/build.xml")
> $buildElements.add("plugins/$plugin/default.properties")
> +$buildElements.add("plugins/$plugin/LICENSE.txt")
>
> ## -------------------------------------------------------
> ## Make the list of delegators
>
> Index: src/templates/build/plugins/core/build.xml
> ===================================================================
> RCS file:
>
/home/cvspublic/jakarta-turbine-maven/src/templates/build/plugins/core/build.xml,v
>
> retrieving revision 1.2
> diff -u -r1.2 build.xml
> --- src/templates/build/plugins/core/build.xml 24 May 2002
> 15:41:28 -0000 1.2
> +++ src/templates/build/plugins/core/build.xml 26 May 2002
> 20:29:57 -0000
> @@ -351,6 +351,12 @@
> </classpath>
> </taskdef>
>
> + <condition property="maven.checkstyle.header.file" value="
> ${maven.home}/plugins/core/LICENSE.txt">
> + <not>
> + <available file="${maven.checkstyle.header.file}"/>
> + </not>
> + </condition>
> +
> <checkstyle
> lcurlyType="${maven.checkstyle.lcurly.type}"
> lcurlyMethod="${maven.checkstyle.lcurly.method}"
>
>
>
>
>
> --
> To unsubscribe, e-mail: <
> mailto:[EMAIL PROTECTED]>
> For additional commands, e-mail: <
> mailto:[EMAIL PROTECTED]>
>
>
>
>
> --
> To unsubscribe, e-mail: <
mailto:[EMAIL PROTECTED]>
> For additional commands, e-mail: <
mailto:[EMAIL PROTECTED]>
>
>
--
To unsubscribe, e-mail: <
mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <
mailto:[EMAIL PROTECTED]>
--
To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
