Phil, First of all, note that this is a change in build only. This change will not affect the source code in question, nor how it is executed. The properties files will end up in the very same location as a result of the build, wether with or without this patch. Hence, there is no need to run any printing tests.
We can of course just as well integrate this patch using jdk9/client, if you prefer. Some background here might be useful, so you can understand what Erik is trying to achieve. Most files are stored in a suitable location: java files that should be compiled in src/*/classes. Files that should be copied to the lib directory in the resulting image are in src/share/lib. However, there are a few files that are stored elsewhere. This has been handled using hard-coded exception rules in the build system. Erik has spent some serious effort lately on cleaning up those idiosyncrasies. That normalization was something that we have been planning to do all along, but it has become urgent due to the jigsaw project. While we can handle the exceptions, even if its ugly, it has turned into a blocker to handle all those misplaced files when we need to reshuffle all source code. So, to summarize: * The code using those files will not be affected. * This is needed for the jigsaw project. With that information, I hope you can reconsider your position. /Magnus 26 feb 2014 kl. 19:48 skrev Phil Race <philip.r...@oracle.com>: > PS the same goes for other things like font property changes if you have such > in mind. > And unless you intend to run the printing test suite before you push, > then changes like this *must* go into jdk9/client. Not jdk9/dev. > Moot however, since I don't agree with the change. > > -phil. > > On 2/26/2014 10:46 AM, Phil Race wrote: >> Adding 2d-dev as 2d owns this file and might like to know where it went. >> My 2p : the files are however interpreted solely by the code with which it >> is co-located. >> So unless the build will come crashing down around us, I'd very much prefer >> that these >> files stay exactly where I put them ... >> >> >> -phil. >> >> On 2/26/2014 7:38 AM, Tim Bell wrote: >>> Hi Erik: >>> >>>> Here is another simple file move to bring more order to the sources. This >>>> time it's the psfont properties files that are being moved from >>>> src/share/classes to src/share/lib since in the build, they are just >>>> copied to the lib directory, so should not be confused with java resource >>>> files. >>>> >>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8035821 >>>> Webrev: http://cr.openjdk.java.net/~erikj/8035821/webrev.jdk.01/ >>> >>> Looks good to me. >>> >>> Tim >
