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 ...
make/Main.gmk
Don't really understand why the dependencies have been separated from
the target+recipe. Is it just a style thing?
make/Jprt.gmk
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. ??
David
/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