DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG�
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=35009>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND�
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=35009

           Summary: Add checkstyle and jalopy targets to build.xml's
           Product: Log4j
           Version: 1.3alpha
          Platform: Other
        OS/Version: other
            Status: NEW
          Severity: enhancement
          Priority: P2
         Component: Other
        AssignedTo: [email protected]
        ReportedBy: [EMAIL PROTECTED]


Checkstyle (http://checkstyle.sourceforge.net) checks source code against 
coding standards.  Jalopy 
(http://jalopy.sourceforge.net) reformats code to adhere to a specified code 
layout.  Both have been 
used manually in earlier log4j development and had configuration files in the 
CVS.  This bug adds 
targets to the build.xml files to support invoking the tools.  In addition, the 
Jalopy and Checkstyle 
configuration files were inconsistent in the treatment of whitespace within 
array initializers which is 
also addressed.

Both Checkstyle and Jalopy require entries in build.properties to locate their 
implementations, see 
build.properties.sample for examples.

To invoke checkstyle (all files, specified files):

ant checkstyle 
ant checkstyle -Dcheckstyle.files=org/apache/log4j/pattern/\*.java

To invoke Jalopy

ant jalopy -Djalopy.files=org/apache/log4j/pattern/\*.java

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to