From: Petter W. <pet...@ya...> - 2004-06-23 05:48:56
|
Hi Carlos, > PS: By the way, the new Ant file is not working > correctly doing the > CheckStyle task. I have never run the CheckStyle task, so it's quite possible that I've broken it while restructuring the Ant file... I do not have CheckStyle installed on my computer. Where can I find it, how should it be set up? The CheckStyle Ant task uses files placed in to directories, lib and xml. These directories have not been committed to CVS, so I cannot see them... Anyway, I wanted to comment on the directory structure. I think the "xml" directory name is not so intuitive, and for understandability reasons I think we should be careful with introducing new directories on the root level. What do you think about creating a tools directory, something like this: tools CheckStyle lib checkstyle-all-3.4.jar (perhaps not part of our CVS tree) config sun_checks.xml checkstyle-simple.xsl Other future tools go here... What do you think about this? If you send me the CheckStyle files, including the xml files (or perhaps they can be downloaded from somewhere), I will fix the broken Ant script. Sorry for messing it up... Regards, Petter __________________________________________________ Do You Yahoo!? Tired of spam? Yahoo! Mail has the best spam protection around http://mail.yahoo.com |