[9] RFR(XL) 8166417: Integrate Graal-core into JDK for AOT compiler

[Vladimir Kozlov]

On 12/9/16 8:40 AM, Mandy Chung wrote:
>
>> On Dec 8, 2016, at 11:07 PM, Vladimir Kozlov <vladimir.kozlov at oracle.com> wrote:
>>
>>>
>>> The new syntax for provides is to have one `provides` clause of the same SPI
>>> with comma-separated implementation classes (see [1] for reference)
>>
>> Previous loop in this makefile also produces multiple `provides` of the same SPI. I fixed it too. The output is next for example:
>>
>> provides org.graalvm.compiler.hotspot.CompilerConfigurationFactory with
>>    org.graalvm.compiler.hotspot.CoreCompilerConfigurationFactory,
>>    org.graalvm.compiler.hotspot.EconomyCompilerConfigurationFactory,
>>    ;
>>
>> Looks like build tools excepts ; after ,
>> If you are fine with it I will make this change.
>
> It’s a bug in the build tool and I will file a bug.
>
> Is it easy to drop last comma?  If not, we could fix this makefile when the build tool is fixed.

It is very hard to drop the last comma from makefile code. I don't know 
how to do that. Please, don't fix this in tool and allow such output.

Otherwise I will have to return back to produces multiple `provides` of 
the same SPI.

Vladimir

>
> Mandy
>