http://cr.openjdk.java.net/~erikj/build-infra-m1/webrev-langtools-new.03/ Addressed the comments below. Leaving GenStubs functionality active, but with just one class, for demo purposes and to keep exercising the functionality for now. At a later stage we should make it optional. For our compare script to keep reporting success, I will leave the javax.tools.JavaCompilerTool in there, with the explaining comment. If this looks ok, we have had the reviews out for a week and no other major concerns or protests. Does that mean Kelly could start pushing this into the build forest later today? /Erik On 2012-04-04 16:56, Jonathan Gibbons wrote: In langtools/Makefile, |
- Review Request: Build-infra M1 Erik Joelsson
- Re: Review Request: Build-infra M1 Erik Joelsson
- Re: Review Request: Build-infra M1 Jonathan Gibbons
- Re: Review Request: Build-infra M1 Jonathan Gibbons
- Re: Review Request: Build-infra M1 Kelly O'Hair
- Re: Review Request: Build-infra M1 Erik Joelsson
- Re: Review Request: Build-infra M1 Erik Joelsson
- Re: Review Request: Build-infra M1 Jonathan Gibbons
- Re: Review Request: Build-infra M... Erik Joelsson
- Re: Review Request: Build-inf... Jonathan Gibbons
- Re: Review Request: Build-infra M... Erik Joelsson
- Re: Review Request: Build-inf... Jonathan Gibbons
- Re: Review Request: Build-infra M1 Magnus Ihse Bursie
- Re: Review Request: Build-infra M1 Fredrik Öhrström
- Re: Review Request: Build-infra M1 David Holmes
- Re: Review Request: Build-infra M1 Erik Joelsson
- Re: Review Request: Build-infra M1 Andrew Hughes
- Re: Review Request: Build-infra M1 Jonathan Gibbons
- Re: Review Request: Build-infra M1 Fredrik Öhrström
- Re: Review Request: Build-infra M1 David Holmes
- Re: Review Request: Build-infra M1 Erik Joelsson