RFR: JDK-8034769: Move logutil in corba to make/tools

Alan Bateman Alan.Bateman at oracle.com
Wed Feb 12 11:58:17 UTC 2014


On 12/02/2014 10:42, Erik Joelsson wrote:
> Here is another source location cleanup, this time in the corba repo. 
> The 6 classes in com/sun/tools/corba/se/logutil are only used as a 
> build tool during compilation of the corba repo. These files should be 
> moved to the corba/make/tools/src directory so that it's not confused 
> with product code.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8034769
> Webrev: http://cr.openjdk.java.net/~erikj/8034769/webrev.corba.01/
This looks okay to me. Just one comment (it's really a question) is 
whether this is consistent with the jdk repository. In the jdk 
repository then the location is make/src/classes.

In passing I see the DELETES value in GensrcCorba.gmk is huge and I 
wonder why it wasn't split over multiple lines. It makes side-by-side 
diffs difficult.

-Alan.



More information about the build-dev mailing list