RFR: 7181175 Enable hotspot builds on Windows with MinGW/MSYS

David Holmes david.holmes at oracle.com
Wed Jul 18 00:16:07 UTC 2012


On 18/07/2012 7:29 AM, Kelly O'Hair wrote:
>
> On Jul 2, 2012, at 10:07 PM, David Holmes wrote:
>
>> Second this seems wrong:
>>
>> "$(COMPILE_RMIC)" -classpath ...
>>
>> If COMPILE_RMIC is re-defined as "rmic <some-arg>" then the above will
>> fail to execute. Plus didn't you already handle the spaces-in-paths
>> problem in rules.make when you defined COMPILE_RMIC?
>
> I think COMPILE_RMIC is already defined to have quotes in it's definition,
>
> http://cr.openjdk.java.net/~tbell/7181175/webrev.00/make/windows/makefiles/rules.make.sdiff.html
>
> I don't think you need quotes here too.

I'm not sure if you are agreeing with me :) so to be clear. We have:

COMPILE_JAVAC="$(BootStrapDir)\bin\javac" $(BOOTSTRAP_JAVAC_FLAGS)
COMPILE_RMIC="$(BootStrapDir)\bin\rmic"

at present. But if we were to change that to

COMPILE_RMIC="$(BootStrapDir)\bin\rmic" $(BOOTSTRAP_RMIC_FLAGS)

for example, then

"$(COMPILE_RMIC)" -classpath ...

would not work.

David

> -kto
>



More information about the build-dev mailing list