RFR: JDK-8154841: Let different Jib profiles have different default make targets

David Holmes david.holmes at oracle.com
Tue Apr 26 12:47:30 UTC 2016

On 26/04/2016 10:01 PM, Erik Joelsson wrote:
> It would be really nice to get this reviewed as I have a lot of further
> work depending on this feature.

I'm not fluent in .js but those changes seemed relatively straight 
forward to understand. More so than the rest ...


Don't really understand why the dependencies have been separated from 
the target+recipe. Is it just a style thing?


So you are assuming/expecting that JPRT_TARGET always requires 
product-images test-image ? Shouldn't that target (whatever it is) have 
its own dependencies? Not at all clear what this is doing except 
ensuring you can't build less than product-images test-image. ??


> /Erik
> On 2016-04-22 10:32, Erik Joelsson wrote:
>> Currently all Jib profiles define the default make target configure
>> arg to be "all". While this works ok for our current main profiles,
>> it's a bit crude and will not work well when start introducing more
>> specialized profiles. I have implemented a way of specifying the list
>> of default make targets much like configure arguments are specified
>> today, by adding a function that converts this new field on a profile
>> into the corresponding configure arg.
>> To demonstrate the feature I have also disable building of docs on
>> most profiles. The docs build is very platform independent and we are
>> currently wasting quite a bit of time letting every build, both
>> release and debug, build docs. With these changes, only linux-x64
>> builds docs by default. Note that this applies to distributed build
>> scenarios, like JPRT and RE at Oracle. A developer wanting to build
>> docs may still do so by typing "make docs".
>> While tweaking the docs build, I also corrected the dependencies that
>> were recently introduced for the new open docs bundle. It seems I was
>> sloppy with the review on that change.
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8154841
>> Webrev: http://cr.openjdk.java.net/~erikj/8154841/webrev.top.01/
>> /Erik

More information about the build-dev mailing list