Review request: JDK-8012643 JDK8 b86 source with GPL header errors (jaxws & jdk)
Alan Bateman
Alan.Bateman at oracle.com
Sat Apr 20 00:18:36 PDT 2013
On 20/04/2013 00:04, David Katleman wrote:
> Below are corrections to GPL headers found in JDK8 b86:
>
> http://cr.openjdk.java.net/~katleman/8012643/webrev.jaxws.01
> http://cr.openjdk.java.net/~katleman/8012643/webrev.jdk.01
>
> Corresponding CR
>
> http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8012643
>
> The jdk headers are punctuation changes.
>
> jaxws integrated 51 files with a wrong GPL header, so that part is a
> bit more involved.
> Still no code changes, just headers, so anyone is qualified to review
> them.
Looks fine to me too.
I sponsored the JAX-WS update that went into b86. It was a monster patch
(>70000 lines of changes) and really hard to see the wood from the trees
when bringing in this volume of changes. In this case, the 51 files had
to have their package changed from upstream so I'll bet this is why
their headers were missed.
Going forward, are fixing up issues in master a problem? Which scripts
should be run to catch issues like this? I see in the top level repo
that we have lic_check.sh - is that the script you use? Do you want
these issues caught by integrators before they are pushed to master?
-Alan.
More information about the jdk8-dev
mailing list