RFR: JDK-8153969: Clean up module src dir logic

Erik Joelsson erik.joelsson at oracle.com
Tue Apr 12 13:44:32 UTC 2016


This failed on Windows in the workaround for jdk.vm.ci modulesourcepath. 
Here is a new version where I made the PathList macro safe for multiple 
calls. I also added a test to verify this behavior.

http://cr.openjdk.java.net/~erikj/8153969/webrev.top.03/

/Erik

On 2016-04-12 14:37, Erik Joelsson wrote:
> New webrev with some more cleanup in CompileJavaModules.gmk. Fixed the 
> variable name and also realized that the call to MakeDir was no longer 
> necessary. Starting to look pretty neat.
>
> http://cr.openjdk.java.net/~erikj/8153969/webrev.top.02/
>
> /Erik
>
> On 2016-04-12 11:37, Erik Joelsson wrote:
>> That is a good point, I will clean up that naming too.
>>
>> /Erik
>>
>> On 2016-04-11 18:26, Jonathan Gibbons wrote:
>>> On 04/11/2016 05:28 AM, Erik Joelsson wrote:
>>>> We currently have 3 places in the makefiles where the full list of 
>>>> source dirs for each module is calculated. I have refactored this 
>>>> into macros in make/common/Modules.gmk so that it's handled 
>>>> uniformly. This will also make it simpler for other potential users 
>>>> of this data, like IDE project generation.
>>>>
>>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8153969
>>>> Webrev: http://cr.openjdk.java.net/~erikj/8153969/webrev.top.01/
>>>>
>>>> /Erik
>>>
>>> In a world where we are trying to differentiate betwen module paths 
>>> and class paths, the name MODULECLASSPATH seems somewhat schizophrenic.
>>>
>>> -- Jon
>>
>




More information about the build-dev mailing list