Hi Tony,

Thanks for your reply!


Could you fill a JIRA Bug report at
> http://jira.codehaus.org/browse/MLICENSE.
>
>  Once this done, I will have a look at your patch and apply it if it is
> ok to me :)
>

Done: https://jira.codehaus.org/browse/MLICENSE-34


For your build problem this is another issue, you can have a look at
> target/its/download-licenses/configured-alt-location/build.log to sse the
> problem.
>

Thanks, I should have read the output more carefully. I updated the gist
with the contents of the relevant log. The problem is that the post-build
script fails due to a missing license file. See:
    https://gist.github.com/2220389#L406

Not a big deal to me though—passing "-Dmaven.test.skip=true" avoids the
issue.

Regards,
Curtis


On Tue, Mar 27, 2012 at 5:25 PM, Tony Chemit <[email protected]> wrote:

> On Tue, 27 Mar 2012 16:30:17 -0500
> Curtis Rueden <[email protected]> wrote:
>
> > Hi everyone,
>
> Hi Curtis,
>
> Could you fill a JIRA Bug report at
> http://jira.codehaus.org/browse/MLICENSE.
>
> Once this done, I will have a look at your patch and apply it if it is
> ok to me :)
>
> For your build problem this is another issue, you can have a look at
> target/its/download-licenses/configured-alt-location/build.log to sse
> the problem.
>
> Thanks for your report.
>
> Tony.
>
> >
> > Recently I started using license-maven-plugin. It is very useful, so
> > thanks to the authors!
> >
> > I encountered a problem with license:update-file-header when the
> > copyright holder string spans multiple lines. E.g.:
> >     https://github.com/imagej/imagej/blob/master/pom.xml#L31
> >
> > I have included the line breaks for organizationName to keep the
> > license block under 80 characters per line.
> >
> > This works fine for adding the initial license blocks, but causes an
> > exception to be thrown when processing files that already have a
> > proper header:
> >     https://gist.github.com/2220296
> >
> > I debugged the issue, and it comes down to the copyright regex using
> > dot (.) to match any character. But dot does not match newlines by
> > default. The fix is easy: pass the Pattern.DOTALL flag when compiling
> > the pattern.
> >
> > I fixed the problem on a branch "fix-multi-line-copyright" on my Git
> > mirror of the code:
> >
> >
> https://github.com/ctrueden/license-maven-plugin/tree/fix-multi-line-copyright
> >
> > The relevant commit is:
> >
> >
> https://github.com/ctrueden/license-maven-plugin/commit/01a76e7b15b750ec0c396e391a544dc803f86003
> >
> > I would greatly appreciate if the maintainer(s) could include this
> > patch upstream!
> >
> > Thanks,
> > Curtis
> >
> > P.S. Running "mvn install" takes a long time and eventually fails:
> >    https://gist.github.com/2220389
> >
> > (For testing, I had to use "mvn install:install-file" to manually
> > install the new JAR into my local repo.) What is the best way to
> > avoid this issue? I tried with "-DskipTests" but the same failure
> > still occurs.
>
>
>
> --
> Tony Chemit
> --------------------
> tél: +33 (0) 2 40 50 29 28
> email: [email protected]
> http://www.codelutin.com
>
> ---------------------------------------------------------------------
> To unsubscribe from this list, please visit:
>
>    http://xircles.codehaus.org/manage_email
>
>
>

Reply via email to