Looks good to me.

/Erik

On 2015-04-17 14:20, Magnus Ihse Bursie wrote:
The MacBundles logic needs some love.

1) The names defined in spec.gmk.in just says "bundle" but this is a somewhat overloaded term, shared e.g. by jprt. This should be renamed to macosx_bundle.

2) We should use the modern SetupTextFileProcessing to process the plist template files.

3) We should just clean up the open/non-open mess wrt names and version strings.

I chose to keep the non-open versions, for several reasons. It's probably more likely that anyone is dependent on the more widely distributed Oracle JDK, so changing that would likely be more disruptive. And secondly, in one case the non-open version was more informative, which is likely to be a good thing. (In the other case, it was just a difference of using paranthesis or not.)

Bug: https://bugs.openjdk.java.net/browse/JDK-8078058
WebRev: http://cr.openjdk.java.net/~ihse/JDK-8078058-cleanup-macosx-bundles/webrev.01

/Magnus

Reply via email to