Re: RFR: 8056216 : Remove sun directory layer from libawt and common

2014-09-19 Thread Erik Joelsson
Looks good to me. Thanks for fixing this! /Erik On 2014-09-18 22:42, Phil Race wrote: https://bugs.openjdk.java.net/browse/JDK-8056216 http://cr.openjdk.java.net/~prr/8056216/ This is all just removing the sequence sun/ from various pathnames. Aside from the make file changes there are over

Re: RFR: 8056216 : Remove sun directory layer from libawt and common

2014-09-19 Thread Magnus Ihse Bursie
On 2014-09-18 22:42, Phil Race wrote: https://bugs.openjdk.java.net/browse/JDK-8056216 http://cr.openjdk.java.net/~prr/8056216/ Looks good to me. Thanks! /Magnus This is all just removing the sequence sun/ from various pathnames. Aside from the make file changes there are over 600 file

Re: AWT Dev RFR: 8056216 : Remove sun directory layer from libawt and common

2014-09-19 Thread Seán Coffey
Hi Phil, you'll need to update the unshuffle script[1] also given your path changes. A find/replace operation should work. It probably makes sense to push all changes together. Regards, Sean. [1] http://cr.openjdk.java.net/~chegar/docs/portingScript.html On 19/09/14 09:28, Magnus Ihse

Re: AWT Dev RFR: 8056216 : Remove sun directory layer from libawt and common

2014-09-19 Thread Phil Race
Gosh that's going to be a pain to maintain .. here's an update to the 334 affected lines in that file ! Look ok ? http://cr.openjdk.java.net/~prr/8056216.1 -phil On 09/19/2014 03:37 AM, Seán Coffey wrote: Hi Phil, you'll need to update the unshuffle script[1] also given your path changes.

Re: AWT Dev RFR: 8056216 : Remove sun directory layer from libawt and common

2014-09-19 Thread Alan Bateman
On 19/09/2014 17:22, Phil Race wrote: Gosh that's going to be a pain to maintain .. here's an update to the 334 affected lines in that file ! Look ok ? http://cr.openjdk.java.net/~prr/8056216.1 -phil Ideally there should be just one line per directory, it should only list individual files

Re: AWT Dev RFR: 8056216 : Remove sun directory layer from libawt and common

2014-09-19 Thread Seán Coffey
On 19/09/14 17:33, Alan Bateman wrote: On 19/09/2014 17:22, Phil Race wrote: Gosh that's going to be a pain to maintain .. here's an update to the 334 affected lines in that file ! Look ok ? http://cr.openjdk.java.net/~prr/8056216.1 -phil Ideally there should be just one line per directory,

RFR: 8056216 : Remove sun directory layer from libawt and common

2014-09-18 Thread Phil Race
https://bugs.openjdk.java.net/browse/JDK-8056216 http://cr.openjdk.java.net/~prr/8056216/ This is all just removing the sequence sun/ from various pathnames. Aside from the make file changes there are over 600 file moves which I don't think its worth including in the webrev but they are the