Mailing-List: contact dev-help@commons.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Commons Developers List" <dev@commons.apache.org>
Received-SPF: pass (athena.apache.org: local policy)
Date: Mon, 27 Aug 2012 23:50:25 +0200
From: Christian Hammers <ch@lathspell.de>
To: "Commons Developers List" <dev@commons.apache.org>
Cc: garydgregory@gmail.com
Subject: Re: [Codec] Next Commons-Codec release?
Message-ID: <20120827235025.172dd882@james.intern>
In-Reply-To: 
 <CACZkXPw28U+vjAigXuVNNSWdgwhzYeJn-99Cbv5VUa7JWGJx4w@mail.gmail.com>
References: <20120524110000.39d65ba2@sys-251.netcologne.de>
	<20120815210110.0b6cf796@james.intern>
	<CACZkXPydhsTJrVQc-7CFD=nTXzjk_d-eynW2vLMQgVbpWJr1CA@mail.gmail.com>
	<CACZkXPwrCxah06hvSVj-_yzi85ZFGtPmjXzd+y6bBU79bvZ4Aw@mail.gmail.com>
	<502E9225.8050800@gmail.com>
	<CACZkXPwmKW5PVqSTZTGipuW8=CFA-xG8SBDbvOrh7yTAJObqYA@mail.gmail.com>
	<5032A074.5020907@gmail.com>
	<CACZkXPxpWkO6RR11_uJcR8oqxeJ6fvDZGMg4yyKTehRFfBZ5+g@mail.gmail.com>
	<5032ADCD.6050002@gmail.com>
	<CACZkXPzbzNyqGKUBL=qF1h2EY6f+GEzH=ye4_ZW-5rt3GrYNsw@mail.gmail.com>
	<5033F495.8090706@gmail.com>
	<CACZkXPzff8Mxh8UBAtOHtyGRsQSNtha-q=U+QhzCC9z3RRYC+g@mail.gmail.com>
	<50347AD9.5040909@gmail.com>
	<-5658534165078364893@unknownmsgid>
	<50368CFD.4050409@gmail.com>
	<CACZkXPzz116LWaiwaEPTmXu_NjkTTHLerkK1gYT83QfUJwDFug@mail.gmail.com>
	<20120826010927.1fdd0e96@james.intern>
	<CACZkXPw28U+vjAigXuVNNSWdgwhzYeJn-99Cbv5VUa7JWGJx4w@mail.gmail.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit


Am Mon, 27 Aug 2012 08:59:52 -0400
schrieb Gary Gregory <garydgregory@gmail.com>:

> On Sat, Aug 25, 2012 at 7:09 PM, Christian Hammers <ch@lathspell.de>
> wrote:
> 
> > Hello
> >
> > Am Fri, 24 Aug 2012 09:28:55 -0400
> > schrieb Gary Gregory <garydgregory@gmail.com>:
> >
> > > On Thu, Aug 23, 2012 at 4:05 PM, Thomas Neidhart
> > > <thomas.neidhart@gmail.com>wrote:
> > >
> > > > On 08/22/2012 01:23 PM, Gary Gregory wrote:
> > > >
> > > > > I did the SVN update from E and the build from a Win7 command
> > > > > line.
> > > >
> > > > Does it still not work for you?
> > >
> > >
> > > Yes, it still happens, very weird. Are you building on Win? Is
> > > this an OS-specific EOL thing? Unix vs. DOS? Arg.
> >
> > I found the cause of the "Missing a header - not enough
> > lines in file." checkstyle errors: the checkstyle.xml says:
> >
> >   <property name="headerFile" value="${checkstyle.header.file}" />
> >
> > but you define the header as property "headerFile" to the checkstyle
> > module and not as POM property. So either remove the above line
> > (untested) or apply the following patch (tested):
> >
> > Index: pom.xml
> > ===================================================================
> > --- pom.xml     (Revision 1377365)
> > +++ pom.xml     (Arbeitskopie)
> > @@ -208,6 +208,7 @@
> >      <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
> >
> >  <project.reporting.outputEncoding>UTF-8</project.reporting.outputEncoding>
> >      <commons.encoding>UTF-8</commons.encoding>
> > +
> >
> > <checkstyle.header.file>${basedir}/LICENSE-header.txt</checkstyle.header.file>
> > </properties> <build>
> >      <pluginManagement>
> > @@ -272,7 +273,6 @@
> >          <configuration>
> >            <configLocation>${basedir}/checkstyle.xml</configLocation>
> >            <enableRulesSummary>false</enableRulesSummary>
> > -          <headerFile>${basedir}/LICENSE-header.txt</headerFile>
> >          </configuration>
> >        </plugin>
> >        <plugin>
> >
> >
> Weird, the trunk code is already like that (expect the extra blank
> line).
> 
> Gary

You meant "except" instead of "expect", right? The format of the diff
is messed up, the first block does not add a blank line but the
following "<checkstyle.header.file>" line.
At least in the current trunk r1377882 the header file location is
still in the <plugins> and not in the <properties> block.

bye,

-christian-

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org