commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Gary Gregory <garydgreg...@gmail.com>
Subject Re: [Codec] Next Commons-Codec release?
Date Mon, 27 Aug 2012 12:59:52 GMT
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


> bye,
>
> -christian-
>
>


-- 
E-Mail: garydgregory@gmail.com | ggregory@apache.org
JUnit in Action, 2nd Ed: <http://goog_1249600977>http://bit.ly/ECvg0
Spring Batch in Action: <http://s.apache.org/HOq>http://bit.ly/bqpbCK
Blog: http://garygregory.wordpress.com
Home: http://garygregory.com/
Tweet! http://twitter.com/GaryGregory

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message