RFR: 8243059: Build fails when --with-vendor-name contains a comma

[Severin Gehwolf]

Hi,

Could I please get a review of this build fix? When --with-vendor-name
contains a comma, like 'foo, bar, Inc.' the build fails. As it turns
out SetupBuildLauncherBody calls SetupJdkExecutable with some
parameters. If $(VERSION_CFLAGS) contain a comma, the comma is being
treated as a prameter delimiter and the build fails.

I propose to substitute ',' with $(DOLLAR)COMMA in VERSION_CFLAGS so as
to avoid inadvertent expansion of the comma and, thus, it being treated
as a parameter to the SetupJdkExecutable macro.

Bug: https://bugs.openjdk.java.net/browse/JDK-8243059
webrev: http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-8243059/02/webrev

NOTE: Magnus pointed out that there were two pre-existing bugs too.
LAUNCHER_NAME and LAUNCHER_CFLAGS should have $$ in front of them, not
just a single one. Fixed this too.

Testing: Setting --with-vendor-name='foo, bar' and build passes post-
patch. Also manually inspecting VENDOR is properly passed on the
command line when java/main.o is being compiled. I'll run this through
jdk-submit before I push.

Thoughts?

Thanks,
Severin