Return-Path: X-Original-To: apmail-commons-issues-archive@minotaur.apache.org Delivered-To: apmail-commons-issues-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 1DD929850 for ; Tue, 8 Nov 2011 20:32:13 +0000 (UTC) Received: (qmail 92571 invoked by uid 500); 8 Nov 2011 20:32:13 -0000 Delivered-To: apmail-commons-issues-archive@commons.apache.org Received: (qmail 92346 invoked by uid 500); 8 Nov 2011 20:32:12 -0000 Mailing-List: contact issues-help@commons.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: issues@commons.apache.org Delivered-To: mailing list issues@commons.apache.org Received: (qmail 92338 invoked by uid 99); 8 Nov 2011 20:32:12 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 08 Nov 2011 20:32:12 +0000 X-ASF-Spam-Status: No, hits=-2001.2 required=5.0 tests=ALL_TRUSTED,RP_MATCHES_RCVD X-Spam-Check-By: apache.org Received: from [140.211.11.116] (HELO hel.zones.apache.org) (140.211.11.116) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 08 Nov 2011 20:32:11 +0000 Received: from hel.zones.apache.org (hel.zones.apache.org [140.211.11.116]) by hel.zones.apache.org (Postfix) with ESMTP id A11E63EF79 for ; Tue, 8 Nov 2011 20:31:51 +0000 (UTC) Date: Tue, 8 Nov 2011 20:31:51 +0000 (UTC) From: "Oliver Heger (Commented) (JIRA)" To: issues@commons.apache.org Message-ID: <1454733640.11940.1320784311661.JavaMail.tomcat@hel.zones.apache.org> In-Reply-To: <1220442066.57411.1320354813878.JavaMail.tomcat@hel.zones.apache.org> Subject: [jira] [Commented] (CONFIGURATION-470) Extract classes generated by JavaCC MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-JIRA-FingerPrint: 30527f35849b9dde25b450d4833f0394 [ https://issues.apache.org/jira/browse/CONFIGURATION-470?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13146537#comment-13146537 ] Oliver Heger commented on CONFIGURATION-470: -------------------------------------------- I am sorry, but is this really a problem? On my machine (an almost 4 years old laptop) the additional JavaCC step takes about a second; this hardly counts compared to some of the tests which take much longer. Eclipse does not have any problems with finding the generated sources. Benefits are that redundant code has been removed and the danger of editing generated classes has been minimized. > Extract classes generated by JavaCC > ----------------------------------- > > Key: CONFIGURATION-470 > URL: https://issues.apache.org/jira/browse/CONFIGURATION-470 > Project: Commons Configuration > Issue Type: Improvement > Components: Build > Affects Versions: 1.7 > Reporter: Oliver Heger > Assignee: Oliver Heger > Priority: Minor > Fix For: 1.8 > > > In the plist package classes generated by JavaCC are used to parse specific configuration files. Currently these files are stored in SVN like regular source files. This does not conform to best practices, is confusing, and requires a special treatment in the configuration of tools like checkstyle or findbugs. > It would be better to let these files be generated during the build process by the maven JavaCC plug-in. A working configuration can kindly be borrowed from the OGNL component. The additional generation step should not slow down the build significantly. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira