See below....do we use a patched version of CheckStyle being used?
--
dIon Gillard, Multitask Consulting
Work: http://www.multitask.com.au
Developers: http://adslgateway.multitask.com.au/developers
----- Forwarded by dIon Gillard/Multitask Consulting/AU on 11/04/2002 11:43 AM -----
"Oliver Burn" <[EMAIL PROTECTED]>
10/04/2002 09:07 AM
To: <[EMAIL PROTECTED]>
cc:
Subject: Re: [Checkstyle-user] Fw: How to turn off
checkstyle.header.file?
> See
> http://jakarta.apache.org/turbine/maven/faq.html#check-source-disable-header
>
> NUL: works fine on Windows,
> /dev/null on Unix-based systems....
Excellent - glad to of been of help without any coding. :-)
I had a look at the Maven site and had a couple of questions/comments
(if I may).
You list a dependency on Checkstyle-2.2 at
http://jakarta.apache.org/turbine/maven/dependencies.html. This is confusing
as we have only released version 2.1 - so I assume this is a build you are
making directly out of CVS?? It is also not clear where to get it from. I
would appreciate it if you used another name like Checkstyle-patched-2.1
if it is a patched version.
Question more for tonight - should I be using Mavern for Checkstyle? Has
it
been done on a SourceForge project?
The link to Checkstyle in the FAQ is broken. Search for
"The CheckStyle defaults" on the FAQ.
I can see a potential maintenance nightmare for you in documenting the
Checkstyle features. I am referring to
http://jakarta.apache.org/turbine/maven/properties.html#Checkstyle%20Settings
Cheers,
Oliver
PS: Just a few errors at
http://jakarta.apache.org/turbine/maven/checkstyle-report.html :-)
> --
> dIon Gillard, Multitask Consulting
> Work: http://www.multitask.com.au
> Developers: http://adslgateway.multitask.com.au/developers
>
> dIon Gillard/Multitask Consulting/AU wrote on 09/04/2002 02:00:43 PM:
>
> > "Oliver Burn" <[EMAIL PROTECTED]> wrote on 09/04/2002 01:54:33 PM:
> >
> > > > "Oliver Burn" <[EMAIL PROTECTED]> wrote on 09/04/2002 10:51:15
> AM:
> > > >
> > > > > I am not sure what is being asked for, because if you do not
> > set thename
> > > > of a
> > > > > header name file using either:
> > > > >
> > > > > - the property 'checkstyle.header.file'; or
> > > > >
> > > > > - the ANT parameter 'headerFile'
> > > > >
> > > > > then Checkstyle will not complain. If you do however set the
> > header file
> > > > name,
> > > > > then Checkstyle will look for the file and complain if it cannot
> find
> > > > it. I
> > > > > would not be in favour of silently handling the case where
> arespecified
> > > > file
> > > > > does not exist as this is a bad design practice.
> > > > I agree. However, in Maven, we always specify the name of the
> > header file.
> > > > Hence there's no way for us to not specify it, short of writing
> multiple
> > > > targets with if's on them.....I was hoping we could use a
> > completely blank
> > > > value to say don't use it....or another
> > parameter.....or.....(insert other
> > > > suggestions here)
> > >
> > > The work around is to specify an empty file - maybe /dev/null will
> work
> > > on Unix?
> >
> > Good call, maybe NUL: will work on Windows....will test it out.
> >
> >
> > > I find it strange you agree that is bad design practice, but you
want
> me to
> > > put it in! You can try and convince me at SJUG tomorrow, but I am
very
> > > unlikely to put the hack in. :-)
> >
> > Hey, I agreed on the silent failing being bad....I don't want you to
> > do this, but what about where there is a property specified, but
> > it's blank... that can't possibly work....Does the ant property get
> > set if I specify ""?
> >
> > I'm trying to find alternatives to a bad hack :-)
> >
> >
> > > Regards,
> > > Oliver
> >
> > --
> > dIon Gillard, Multitask Consulting
> > Work: http://www.multitask.com.au
> > Developers: http://adslgateway.multitask.com.au/developers
>
