Review Request : 7143162 : Allow disable building of jdk demos and samples (original) (raw)

Weijun Wang weijun.wang at oracle.com
Tue Mar 6 03:34:31 UTC 2012

If NO_DEMOS are specified, there is no demo dir and "make images" fails. Yes, maybe it should fail, but I must work with images build now (on deploy workspace) and I really like the NO_DEMOS option...

How about this?

diff --git a/make/common/Release.gmk b/make/common/Release.gmk --- a/make/common/Release.gmk +++ b/make/common/Release.gmk @@ -947,6 +947,7 @@ done (CP)(CP) (CP)(ABS_DB_PATH)/README-JDK.html $(JDK_IMAGE_DIR)/db (RM)−rf(RM) -rf (RM)−rf(DEMODIR)/db

Thanks Max

On 02/28/2012 09:53 AM, Kelly O'Hair wrote:

Looks fine to me.

-kto On Feb 27, 2012, at 4:14 PM, Mike Duigou wrote:

Hello all;

WEBREV @ http://cr.openjdk.java.net/~mduigou/7143162/1/webrev/ An updated version of my previous webrev. I'm reposting it because this version includes a previously missing bit from the top level sanity makefile that produce warning notices when demos and samples are turned off similar to the NODOCS javadoc warning. Also includes a correction in Release.gmk. Overall this is fairly simple patch for review. It allows developers to shut off building of the jdk demos and samples. Disabling these builds has a very positive impact upon build turnaround for incremental builds. Export NODEMOS and/or NOSAMPLES in your shell before running make and demos and/or samples will be skipped. The default behaviour remains unchanged. Thanks, Mike

More information about the build-dev mailing list