Mailing-List: contact dev-help@openjpa.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@openjpa.apache.org
Message-ID: <2055859968.1238738712944.JavaMail.jira@brutus>
Date: Thu, 2 Apr 2009 23:05:12 -0700 (PDT)
From: "Albert Lee (JIRA)" <jira@apache.org>
To: dev@openjpa.apache.org
Subject: [jira] Commented: (OPENJPA-1015) Enforce 80-column line width for
 source code
In-Reply-To: <1870011503.1238527610652.JavaMail.jira@brutus>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


    [ https://issues.apache.org/jira/browse/OPENJPA-1015?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12695251#action_12695251 ] 

Albert Lee commented on OPENJPA-1015:
-------------------------------------

After applying/updating to the latest commits, I am getting the folllowing error when "mvn clean install -DskipTests=true":

[INFO] ------------------------------------------------------------------------
[INFO] Building OpenJPA Utilities
[INFO]    task-segment: [clean, install]
[INFO] ------------------------------------------------------------------------
[INFO] [clean:clean]
[INFO] Deleting file set: C:\a.tools.alleetp\wasx.serv1\openjpa.cur\openjpa-lib\target (included: [**], excluded: [])
[INFO] [checkstyle:checkstyle {execution: default}]
[INFO] ------------------------------------------------------------------------
[ERROR] BUILD ERROR
[INFO] ------------------------------------------------------------------------
[INFO] An error has occurred in Checkstyle report generation.

Embedded error: Unable to find configuration file at location checkstyle.xml
Could not find resource 'checkstyle.xml'.
[INFO] ------------------------------------------------------------------------
[INFO] For more information, run Maven with the -e switch
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 6 seconds

Where Is this checkstyle.xml file and is it supposedly part of the commit or I have to create it somehow?

Thanks,

> Enforce 80-column line width for source code
> --------------------------------------------
>
>                 Key: OPENJPA-1015
>                 URL: https://issues.apache.org/jira/browse/OPENJPA-1015
>             Project: OpenJPA
>          Issue Type: Test
>            Reporter: Pinaki Poddar
>            Assignee: B.J. Reed
>         Attachments: line80.txt, OPENJPA-1015-max80chars.patch, OPENJPA-1015-newlineendoffile.patch, TestLineWidth.java
>
>
> There used to be a rule of 80-column width for source code.
> That rule is slowly yielding. 
> Should we add a test case to catch such violation?
> Or should we not reignite the age-old battle about line width, placement of brackets etc?

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.