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