author | coleenp |
Tue, 31 Oct 2017 11:55:09 -0400 | |
changeset 47765 | b7c7428eaab9 |
parent 47346 | ea082b202a23 |
child 47456 | 9c1e4b0a4761 |
permissions | -rw-r--r-- |
13133
701e61c32ece
7182051: Update of latest build-infra Makefiles (missing files)
erikj
parents:
diff
changeset
|
1 |
# |
43173 | 2 |
# Copyright (c) 2011, 2017, Oracle and/or its affiliates. All rights reserved. |
13133
701e61c32ece
7182051: Update of latest build-infra Makefiles (missing files)
erikj
parents:
diff
changeset
|
3 |
# DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
701e61c32ece
7182051: Update of latest build-infra Makefiles (missing files)
erikj
parents:
diff
changeset
|
4 |
# |
701e61c32ece
7182051: Update of latest build-infra Makefiles (missing files)
erikj
parents:
diff
changeset
|
5 |
# This code is free software; you can redistribute it and/or modify it |
701e61c32ece
7182051: Update of latest build-infra Makefiles (missing files)
erikj
parents:
diff
changeset
|
6 |
# under the terms of the GNU General Public License version 2 only, as |
701e61c32ece
7182051: Update of latest build-infra Makefiles (missing files)
erikj
parents:
diff
changeset
|
7 |
# published by the Free Software Foundation. Oracle designates this |
701e61c32ece
7182051: Update of latest build-infra Makefiles (missing files)
erikj
parents:
diff
changeset
|
8 |
# particular file as subject to the "Classpath" exception as provided |
701e61c32ece
7182051: Update of latest build-infra Makefiles (missing files)
erikj
parents:
diff
changeset
|
9 |
# by Oracle in the LICENSE file that accompanied this code. |
701e61c32ece
7182051: Update of latest build-infra Makefiles (missing files)
erikj
parents:
diff
changeset
|
10 |
# |
701e61c32ece
7182051: Update of latest build-infra Makefiles (missing files)
erikj
parents:
diff
changeset
|
11 |
# This code is distributed in the hope that it will be useful, but WITHOUT |
701e61c32ece
7182051: Update of latest build-infra Makefiles (missing files)
erikj
parents:
diff
changeset
|
12 |
# ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or |
701e61c32ece
7182051: Update of latest build-infra Makefiles (missing files)
erikj
parents:
diff
changeset
|
13 |
# FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License |
701e61c32ece
7182051: Update of latest build-infra Makefiles (missing files)
erikj
parents:
diff
changeset
|
14 |
# version 2 for more details (a copy is included in the LICENSE file that |
701e61c32ece
7182051: Update of latest build-infra Makefiles (missing files)
erikj
parents:
diff
changeset
|
15 |
# accompanied this code). |
701e61c32ece
7182051: Update of latest build-infra Makefiles (missing files)
erikj
parents:
diff
changeset
|
16 |
# |
701e61c32ece
7182051: Update of latest build-infra Makefiles (missing files)
erikj
parents:
diff
changeset
|
17 |
# You should have received a copy of the GNU General Public License version |
701e61c32ece
7182051: Update of latest build-infra Makefiles (missing files)
erikj
parents:
diff
changeset
|
18 |
# 2 along with this work; if not, write to the Free Software Foundation, |
701e61c32ece
7182051: Update of latest build-infra Makefiles (missing files)
erikj
parents:
diff
changeset
|
19 |
# Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. |
701e61c32ece
7182051: Update of latest build-infra Makefiles (missing files)
erikj
parents:
diff
changeset
|
20 |
# |
701e61c32ece
7182051: Update of latest build-infra Makefiles (missing files)
erikj
parents:
diff
changeset
|
21 |
# Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
701e61c32ece
7182051: Update of latest build-infra Makefiles (missing files)
erikj
parents:
diff
changeset
|
22 |
# or visit www.oracle.com if you need additional information or have any |
701e61c32ece
7182051: Update of latest build-infra Makefiles (missing files)
erikj
parents:
diff
changeset
|
23 |
# questions. |
701e61c32ece
7182051: Update of latest build-infra Makefiles (missing files)
erikj
parents:
diff
changeset
|
24 |
# |
701e61c32ece
7182051: Update of latest build-infra Makefiles (missing files)
erikj
parents:
diff
changeset
|
25 |
|
29662 | 26 |
################################################################################ |
27 |
# This file contains helper functions for Init.gmk. |
|
28 |
# It is divided in two parts, depending on if a SPEC is present or not |
|
29 |
# (HAS_SPEC is true or not). |
|
30 |
################################################################################ |
|
13133
701e61c32ece
7182051: Update of latest build-infra Makefiles (missing files)
erikj
parents:
diff
changeset
|
31 |
|
29662 | 32 |
ifndef _INITSUPPORT_GMK |
33 |
_INITSUPPORT_GMK := 1 |
|
13133
701e61c32ece
7182051: Update of latest build-infra Makefiles (missing files)
erikj
parents:
diff
changeset
|
34 |
|
29662 | 35 |
ifeq ($(HAS_SPEC),) |
44027 | 36 |
|
37 |
# Include the corresponding closed file, if present. |
|
47313
eb28be8f935d
8188768: Fix interaction between make and autoconf after consolidation
ihse
parents:
47296
diff
changeset
|
38 |
ifneq ($(CUSTOM_MAKE_DIR), ) |
eb28be8f935d
8188768: Fix interaction between make and autoconf after consolidation
ihse
parents:
47296
diff
changeset
|
39 |
-include $(CUSTOM_MAKE_DIR)/InitSupport.gmk |
eb28be8f935d
8188768: Fix interaction between make and autoconf after consolidation
ihse
parents:
47296
diff
changeset
|
40 |
endif |
44027 | 41 |
|
29662 | 42 |
############################################################################## |
43 |
# Helper functions for the initial part of Init.gmk, before the spec file is |
|
44 |
# loaded. Most of these functions provide parsing and setting up make options |
|
45 |
# from the command-line. |
|
46 |
############################################################################## |
|
14111 | 47 |
|
29662 | 48 |
# Make control variables, handled by Init.gmk |
44027 | 49 |
INIT_CONTROL_VARIABLES += LOG CONF CONF_NAME SPEC JOBS TEST_JOBS CONF_CHECK \ |
50 |
COMPARE_BUILD JTREG GTEST |
|
14111 | 51 |
|
29662 | 52 |
# All known make control variables |
53 |
MAKE_CONTROL_VARIABLES := $(INIT_CONTROL_VARIABLES) TEST JDK_FILTER |
|
14111 | 54 |
|
29662 | 55 |
# Define a simple reverse function. |
56 |
# Should maybe move to MakeBase.gmk, but we can't include that file now. |
|
57 |
reverse = \ |
|
58 |
$(if $(strip $(1)), $(call reverse, $(wordlist 2, $(words $(1)), $(1)))) \ |
|
59 |
$(firstword $(1)) |
|
14111 | 60 |
|
29662 | 61 |
# The variable MAKEOVERRIDES contains variable assignments from the command |
62 |
# line, but in reverse order to what the user entered. |
|
32715
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
63 |
# The '\#' <=> '\ 'dance is needed to keep values with space in them connected. |
29662 | 64 |
COMMAND_LINE_VARIABLES := $(subst \#,\ , $(call reverse, $(subst \ ,\#,$(MAKEOVERRIDES)))) |
65 |
||
29788 | 66 |
# A list like FOO="val1" BAR="val2" containing all user-supplied make |
67 |
# variables that we should propagate. |
|
32715
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
68 |
# The '\#' <=> '\ 'dance is needed to keep values with space in them connected. |
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
69 |
USER_MAKE_VARS := $(subst \#,\ , $(filter-out $(addsuffix =%, $(INIT_CONTROL_VARIABLES)), \ |
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
70 |
$(subst \ ,\#,$(MAKEOVERRIDES)))) |
14111 | 71 |
|
29788 | 72 |
# Setup information about available configurations, if any. |
47346 | 73 |
ifneq ($(CUSTOM_ROOT), ) |
74 |
build_dir=$(CUSTOM_ROOT)/build |
|
75 |
else |
|
47217 | 76 |
build_dir=$(topdir)/build |
77 |
endif |
|
29788 | 78 |
all_spec_files=$(wildcard $(build_dir)/*/spec.gmk) |
79 |
# Extract the configuration names from the path |
|
80 |
all_confs=$(patsubst %/spec.gmk, %, $(patsubst $(build_dir)/%, %, $(all_spec_files))) |
|
81 |
||
29662 | 82 |
# Check for unknown command-line variables |
83 |
define CheckControlVariables |
|
84 |
command_line_variables := $$(strip $$(foreach var, \ |
|
85 |
$$(subst \ ,_,$$(MAKEOVERRIDES)), \ |
|
86 |
$$(firstword $$(subst =, , $$(var))))) |
|
29788 | 87 |
unknown_command_line_variables := $$(strip \ |
88 |
$$(filter-out $$(MAKE_CONTROL_VARIABLES), $$(command_line_variables))) |
|
29662 | 89 |
ifneq ($$(unknown_command_line_variables), ) |
90 |
$$(info Note: Command line contains non-control variables:) |
|
91 |
$$(foreach var, $$(unknown_command_line_variables), $$(info * $$(var)=$$($$(var)))) |
|
92 |
$$(info Make sure it is not mistyped, and that you intend to override this variable.) |
|
93 |
$$(info 'make help' will list known control variables.) |
|
94 |
$$(info ) |
|
95 |
endif |
|
96 |
endef |
|
14111 | 97 |
|
29662 | 98 |
# Check for deprecated ALT_ variables |
99 |
define CheckDeprecatedEnvironment |
|
100 |
defined_alt_variables := $$(filter ALT_%, $$(.VARIABLES)) |
|
101 |
ifneq ($$(defined_alt_variables), ) |
|
102 |
$$(info Warning: You have the following ALT_ variables set:) |
|
103 |
$$(foreach var, $$(defined_alt_variables), $$(info * $$(var)=$$($$(var)))) |
|
104 |
$$(info ALT_ variables are deprecated, and may result in a failed build.) |
|
105 |
$$(info Please clean your environment.) |
|
106 |
$$(info ) |
|
107 |
endif |
|
108 |
endef |
|
14111 | 109 |
|
29662 | 110 |
# Check for invalid make flags like -j |
111 |
define CheckInvalidMakeFlags |
|
112 |
# This is a trick to get this rule to execute before any other rules |
|
113 |
# MAKEFLAGS only indicate -j if read in a recipe (!) |
|
114 |
$$(topdir)/make/Init.gmk: .FORCE |
|
115 |
$$(if $$(findstring --jobserver, $$(MAKEFLAGS)), \ |
|
116 |
$$(info Error: 'make -jN' is not supported, use 'make JOBS=N') \ |
|
117 |
$$(error Cannot continue) \ |
|
118 |
) |
|
119 |
.FORCE: |
|
120 |
.PHONY: .FORCE |
|
121 |
endef |
|
14111 | 122 |
|
29662 | 123 |
# Check that the CONF_CHECK option is valid and set up handling |
124 |
define ParseConfCheckOption |
|
125 |
ifeq ($$(CONF_CHECK), ) |
|
126 |
# Default behavior is fail |
|
127 |
CONF_CHECK := fail |
|
128 |
else ifneq ($$(filter-out auto fail ignore, $$(CONF_CHECK)),) |
|
129 |
$$(info Error: CONF_CHECK must be one of: auto, fail or ignore.) |
|
130 |
$$(error Cannot continue) |
|
131 |
endif |
|
132 |
endef |
|
14111 | 133 |
|
35444 | 134 |
# Look for a given option in the LOG variable, and if found, set a variable |
135 |
# and remove the option from the LOG variable |
|
136 |
# $1: The option to look for |
|
137 |
# $2: The option to set to "true" if the option is found |
|
138 |
define ParseLogOption |
|
139 |
ifneq ($$(findstring $1, $$(LOG)),) |
|
140 |
$2 := true |
|
141 |
# COMMA is defined in spec.gmk, but that is not included yet |
|
142 |
COMMA := , |
|
143 |
# First try to remove ",<option>" if it exists, otherwise just remove "<option>" |
|
144 |
LOG_STRIPPED := $$(subst $1,, $$(subst $$(COMMA)$(strip $1),, $$(LOG))) |
|
145 |
# We might have ended up with a leading comma. Remove it. Need override |
|
146 |
# since LOG is set from the command line. |
|
147 |
override LOG := $$(strip $$(patsubst $$(COMMA)%, %, $$(LOG_STRIPPED))) |
|
148 |
endif |
|
149 |
endef |
|
150 |
||
29662 | 151 |
define ParseLogLevel |
152 |
# Catch old-style VERBOSE= command lines. |
|
153 |
ifneq ($$(origin VERBOSE), undefined) |
|
154 |
$$(info Error: VERBOSE is deprecated. Use LOG=<warn|info|debug|trace> instead.) |
|
155 |
$$(error Cannot continue) |
|
156 |
endif |
|
14111 | 157 |
|
29662 | 158 |
# Setup logging according to LOG |
14379
b449aa34783e
8003274: build-infra: Makefile changes needed for sjavac
tbell
parents:
14111
diff
changeset
|
159 |
|
20363 | 160 |
# If the "nofile" argument is given, act on it and strip it away |
35444 | 161 |
$$(eval $$(call ParseLogOption, nofile, LOG_NOFILE)) |
162 |
||
163 |
# If the "cmdline" argument is given, act on it and strip it away |
|
164 |
$$(eval $$(call ParseLogOption, cmdlines, LOG_CMDLINES)) |
|
165 |
||
44465
26fabd8abee9
8177770: Need more precise control on build system logging
asemenyuk
parents:
44027
diff
changeset
|
166 |
# If the "profile-to-log" argument is given, write shell times in build log |
26fabd8abee9
8177770: Need more precise control on build system logging
asemenyuk
parents:
44027
diff
changeset
|
167 |
$$(eval $$(call ParseLogOption, profile-to-log, LOG_PROFILE_TIMES_LOG)) |
26fabd8abee9
8177770: Need more precise control on build system logging
asemenyuk
parents:
44027
diff
changeset
|
168 |
|
26fabd8abee9
8177770: Need more precise control on build system logging
asemenyuk
parents:
44027
diff
changeset
|
169 |
# If the "profile" argument is given, write shell times in separate log file |
26fabd8abee9
8177770: Need more precise control on build system logging
asemenyuk
parents:
44027
diff
changeset
|
170 |
# IMPORTANT: $(ParseLogOption profile-to-log) should go first. Otherwise |
26fabd8abee9
8177770: Need more precise control on build system logging
asemenyuk
parents:
44027
diff
changeset
|
171 |
# parsing of 'LOG=debug,profile-to-log,nofile' ends up in the following error: |
26fabd8abee9
8177770: Need more precise control on build system logging
asemenyuk
parents:
44027
diff
changeset
|
172 |
# Error: LOG contains unknown option or log level: debug-to-log. |
26fabd8abee9
8177770: Need more precise control on build system logging
asemenyuk
parents:
44027
diff
changeset
|
173 |
$$(eval $$(call ParseLogOption, profile, LOG_PROFILE_TIMES_FILE)) |
26fabd8abee9
8177770: Need more precise control on build system logging
asemenyuk
parents:
44027
diff
changeset
|
174 |
|
26fabd8abee9
8177770: Need more precise control on build system logging
asemenyuk
parents:
44027
diff
changeset
|
175 |
# Treat LOG=profile-to-log as if it were LOG=profile,profile-to-log |
26fabd8abee9
8177770: Need more precise control on build system logging
asemenyuk
parents:
44027
diff
changeset
|
176 |
LOG_PROFILE_TIMES_FILE := $$(firstword $$(LOG_PROFILE_TIMES_FILE) $$(LOG_PROFILE_TIMES_LOG)) |
26fabd8abee9
8177770: Need more precise control on build system logging
asemenyuk
parents:
44027
diff
changeset
|
177 |
|
35444 | 178 |
LOG_LEVEL := $$(LOG) |
13697 | 179 |
|
20363 | 180 |
ifeq ($$(LOG_LEVEL),) |
29662 | 181 |
# Set LOG to "warn" as default if not set |
182 |
LOG_LEVEL := warn |
|
20363 | 183 |
endif |
29662 | 184 |
|
185 |
ifeq ($$(LOG_LEVEL), warn) |
|
186 |
MAKE_LOG_FLAGS := -s |
|
187 |
else ifeq ($$(LOG_LEVEL), info) |
|
188 |
MAKE_LOG_FLAGS := -s |
|
189 |
else ifeq ($$(LOG_LEVEL), debug) |
|
190 |
MAKE_LOG_FLAGS := |
|
191 |
else ifeq ($$(LOG_LEVEL), trace) |
|
35008 | 192 |
MAKE_LOG_FLAGS := |
13133
701e61c32ece
7182051: Update of latest build-infra Makefiles (missing files)
erikj
parents:
diff
changeset
|
193 |
else |
35444 | 194 |
$$(info Error: LOG contains unknown option or log level: $$(LOG).) |
44465
26fabd8abee9
8177770: Need more precise control on build system logging
asemenyuk
parents:
44027
diff
changeset
|
195 |
$$(info LOG can be <level>[,<opt>[...]] where <opt> is nofile | cmdlines | profile | profile-to-log) |
35444 | 196 |
$$(info and <level> is warn | info | debug | trace) |
29662 | 197 |
$$(error Cannot continue) |
13133
701e61c32ece
7182051: Update of latest build-infra Makefiles (missing files)
erikj
parents:
diff
changeset
|
198 |
endif |
29662 | 199 |
endef |
13133
701e61c32ece
7182051: Update of latest build-infra Makefiles (missing files)
erikj
parents:
diff
changeset
|
200 |
|
29662 | 201 |
define ParseConfAndSpec |
202 |
ifneq ($$(origin SPEC), undefined) |
|
20363 | 203 |
# We have been given a SPEC, check that it works out properly |
29662 | 204 |
ifneq ($$(origin CONF), undefined) |
205 |
# We also have a CONF argument. We can't have both. |
|
206 |
$$(info Error: Cannot use CONF=$$(CONF) and SPEC=$$(SPEC) at the same time. Choose one.) |
|
207 |
$$(error Cannot continue) |
|
20363 | 208 |
endif |
34491 | 209 |
ifneq ($$(origin CONF_NAME), undefined) |
210 |
# We also have a CONF_NAME argument. We can't have both. |
|
211 |
$$(info Error: Cannot use CONF_NAME=$$(CONF_NAME) and SPEC=$$(SPEC) at the same time. Choose one.) |
|
212 |
$$(error Cannot continue) |
|
213 |
endif |
|
27595
cff167b3bfa2
8065914: Various improvements and cleanup of build system
ihse
parents:
27000
diff
changeset
|
214 |
ifeq ($$(wildcard $$(SPEC)),) |
cff167b3bfa2
8065914: Various improvements and cleanup of build system
ihse
parents:
27000
diff
changeset
|
215 |
$$(info Error: Cannot locate spec.gmk, given by SPEC=$$(SPEC).) |
29662 | 216 |
$$(error Cannot continue) |
27595
cff167b3bfa2
8065914: Various improvements and cleanup of build system
ihse
parents:
27000
diff
changeset
|
217 |
endif |
29662 | 218 |
ifeq ($$(filter /%, $$(SPEC)),) |
219 |
# If given with relative path, make it absolute |
|
220 |
SPECS := $$(CURDIR)/$$(strip $$(SPEC)) |
|
221 |
else |
|
222 |
SPECS := $$(SPEC) |
|
223 |
endif |
|
224 |
||
225 |
# For now, unset this SPEC variable. |
|
226 |
override SPEC := |
|
20363 | 227 |
else |
29662 | 228 |
# Use spec.gmk files in the build output directory |
29788 | 229 |
ifeq ($$(all_spec_files),) |
47346 | 230 |
ifneq ($(CUSTOM_ROOT), ) |
231 |
$$(info Error: No configurations found for $$(CUSTOM_ROOT).) |
|
232 |
else |
|
233 |
$$(info Error: No configurations found for $$(topdir).) |
|
234 |
endif |
|
27595
cff167b3bfa2
8065914: Various improvements and cleanup of build system
ihse
parents:
27000
diff
changeset
|
235 |
$$(info Please run 'bash configure' to create a configuration.) |
29662 | 236 |
$$(info ) |
237 |
$$(error Cannot continue) |
|
20363 | 238 |
endif |
13133
701e61c32ece
7182051: Update of latest build-infra Makefiles (missing files)
erikj
parents:
diff
changeset
|
239 |
|
34491 | 240 |
ifneq ($$(origin CONF_NAME), undefined) |
241 |
ifneq ($$(origin CONF), undefined) |
|
242 |
# We also have a CONF argument. We can't have both. |
|
243 |
$$(info Error: Cannot use CONF=$$(CONF) and CONF_NAME=$$(CONF_NAME) at the same time. Choose one.) |
|
244 |
$$(error Cannot continue) |
|
245 |
endif |
|
246 |
matching_conf := $$(strip $$(filter $$(CONF_NAME), $$(all_confs))) |
|
247 |
ifeq ($$(matching_conf),) |
|
248 |
$$(info Error: No configurations found matching CONF_NAME=$$(CONF_NAME).) |
|
249 |
$$(info Available configurations in $$(build_dir):) |
|
250 |
$$(foreach var, $$(all_confs), $$(info * $$(var))) |
|
251 |
$$(error Cannot continue) |
|
252 |
else ifneq ($$(words $$(matching_conf)), 1) |
|
253 |
$$(info Error: Matching more than one configuration CONF_NAME=$$(CONF_NAME).) |
|
254 |
$$(info Available configurations in $$(build_dir):) |
|
255 |
$$(foreach var, $$(all_confs), $$(info * $$(var))) |
|
256 |
$$(error Cannot continue) |
|
257 |
else |
|
258 |
$$(info Building configuration '$$(matching_conf)' (matching CONF_NAME=$$(CONF_NAME))) |
|
259 |
endif |
|
260 |
# Create a SPEC definition. This will contain the path to exactly one spec file. |
|
261 |
SPECS := $$(build_dir)/$$(matching_conf)/spec.gmk |
|
262 |
else ifneq ($$(origin CONF), undefined) |
|
20363 | 263 |
# User have given a CONF= argument. |
264 |
ifeq ($$(CONF),) |
|
265 |
# If given CONF=, match all configurations |
|
29662 | 266 |
matching_confs := $$(strip $$(all_confs)) |
20363 | 267 |
else |
268 |
# Otherwise select those that contain the given CONF string |
|
29788 | 269 |
matching_confs := $$(strip $$(foreach var, $$(all_confs), \ |
270 |
$$(if $$(findstring $$(CONF), $$(var)), $$(var)))) |
|
20363 | 271 |
endif |
272 |
ifeq ($$(matching_confs),) |
|
27595
cff167b3bfa2
8065914: Various improvements and cleanup of build system
ihse
parents:
27000
diff
changeset
|
273 |
$$(info Error: No configurations found matching CONF=$$(CONF).) |
29662 | 274 |
$$(info Available configurations in $$(build_dir):) |
275 |
$$(foreach var, $$(all_confs), $$(info * $$(var))) |
|
276 |
$$(error Cannot continue) |
|
20363 | 277 |
else |
35008 | 278 |
# Don't repeat this output on make restarts caused by including |
279 |
# generated files. |
|
280 |
ifeq ($$(MAKE_RESTARTS),) |
|
281 |
ifeq ($$(words $$(matching_confs)), 1) |
|
282 |
$$(info Building configuration '$$(matching_confs)' (matching CONF=$$(CONF))) |
|
283 |
else |
|
284 |
$$(info Building these configurations (matching CONF=$$(CONF)):) |
|
285 |
$$(foreach var, $$(matching_confs), $$(info * $$(var))) |
|
286 |
endif |
|
20363 | 287 |
endif |
288 |
endif |
|
14111 | 289 |
|
20363 | 290 |
# Create a SPEC definition. This will contain the path to one or more spec.gmk files. |
29662 | 291 |
SPECS := $$(addsuffix /spec.gmk, $$(addprefix $$(build_dir)/, $$(matching_confs))) |
20363 | 292 |
else |
293 |
# No CONF or SPEC given, check the available configurations |
|
29662 | 294 |
ifneq ($$(words $$(all_spec_files)), 1) |
27595
cff167b3bfa2
8065914: Various improvements and cleanup of build system
ihse
parents:
27000
diff
changeset
|
295 |
$$(info Error: No CONF given, but more than one configuration found.) |
29662 | 296 |
$$(info Available configurations in $$(build_dir):) |
297 |
$$(foreach var, $$(all_confs), $$(info * $$(var))) |
|
298 |
$$(info Please retry building with CONF=<config pattern> (or SPEC=<spec file>).) |
|
299 |
$$(info ) |
|
300 |
$$(error Cannot continue) |
|
20363 | 301 |
endif |
14111 | 302 |
|
20363 | 303 |
# We found exactly one configuration, use it |
29662 | 304 |
SPECS := $$(strip $$(all_spec_files)) |
20363 | 305 |
endif |
13133
701e61c32ece
7182051: Update of latest build-infra Makefiles (missing files)
erikj
parents:
diff
changeset
|
306 |
endif |
29662 | 307 |
endef |
29156
bd932374081c
8072842: Add support for building native JTReg tests
ihse
parents:
29060
diff
changeset
|
308 |
|
29788 | 309 |
# Extract main targets from Main.gmk using the spec provided in $2. |
310 |
# |
|
311 |
# Param 1: FORCE = force generation of main-targets.gmk or LAZY = do not force. |
|
312 |
# Param 2: The SPEC file to use. |
|
313 |
define DefineMainTargets |
|
314 |
||
315 |
# We will start by making sure the main-targets.gmk file is removed, if |
|
316 |
# make has not been restarted. By the -include, we will trigger the |
|
317 |
# rule for generating the file (which is never there since we removed it), |
|
318 |
# thus generating it fresh, and make will restart, incrementing the restart |
|
319 |
# count. |
|
320 |
main_targets_file := $$(dir $(strip $2))make-support/main-targets.gmk |
|
321 |
||
322 |
ifeq ($$(MAKE_RESTARTS),) |
|
323 |
# Only do this if make has not been restarted, and if we do not force it. |
|
324 |
ifeq ($(strip $1), FORCE) |
|
325 |
$$(shell rm -f $$(main_targets_file)) |
|
326 |
endif |
|
327 |
endif |
|
328 |
||
329 |
$$(main_targets_file): |
|
330 |
@( cd $$(topdir) && \ |
|
35008 | 331 |
$$(MAKE) $$(MAKE_LOG_FLAGS) -r -R -f $$(topdir)/make/Main.gmk \ |
29788 | 332 |
-I $$(topdir)/make/common SPEC=$(strip $2) NO_RECIPES=true \ |
44465
26fabd8abee9
8177770: Need more precise control on build system logging
asemenyuk
parents:
44027
diff
changeset
|
333 |
$$(MAKE_LOG_VARS) \ |
29788 | 334 |
create-main-targets-include ) |
335 |
||
336 |
# Now include main-targets.gmk. This will define ALL_MAIN_TARGETS. |
|
337 |
-include $$(main_targets_file) |
|
338 |
endef |
|
339 |
||
29662 | 340 |
define PrintConfCheckFailed |
341 |
@echo ' ' |
|
342 |
@echo "Please rerun configure! Easiest way to do this is by running" |
|
343 |
@echo "'make reconfigure'." |
|
344 |
@echo "This behavior may also be changed using CONF_CHECK=<ignore|auto>." |
|
345 |
@echo ' ' |
|
346 |
endef |
|
13697 | 347 |
|
29662 | 348 |
else # $(HAS_SPEC)=true |
349 |
############################################################################## |
|
350 |
# Helper functions for the 'main' target. These functions assume a single, |
|
29788 | 351 |
# proper and existing SPEC is included. |
29662 | 352 |
############################################################################## |
29156
bd932374081c
8072842: Add support for building native JTReg tests
ihse
parents:
29060
diff
changeset
|
353 |
|
47217 | 354 |
include $(TOPDIR)/make/common/MakeBase.gmk |
27560 | 355 |
|
29662 | 356 |
# Define basic logging setup |
47253
92fd0e04e0e1
8187544: Replace BUILD_OUTPUT and OUTPUT_ROOT with OUTPUTDIR
ihse
parents:
47217
diff
changeset
|
357 |
BUILD_LOG := $(OUTPUTDIR)/build.log |
92fd0e04e0e1
8187544: Replace BUILD_OUTPUT and OUTPUT_ROOT with OUTPUTDIR
ihse
parents:
47217
diff
changeset
|
358 |
BUILD_PROFILE_LOG := $(OUTPUTDIR)/build-profile.log |
29662 | 359 |
|
37410
fbab762803ef
8154326: bash >(...) construct causes race conditions
erikj
parents:
35744
diff
changeset
|
360 |
BUILD_LOG_PIPE := > >($(TEE) -a $(BUILD_LOG)) 2> >($(TEE) -a $(BUILD_LOG) >&2) && wait |
27560 | 361 |
|
29662 | 362 |
# Sanity check the spec file, so it matches this source code |
363 |
define CheckSpecSanity |
|
29788 | 364 |
ifneq ($$(ACTUAL_TOPDIR), $$(TOPDIR)) |
365 |
ifneq ($$(ACTUAL_TOPDIR), $$(ORIGINAL_TOPDIR)) |
|
366 |
ifneq ($$(ACTUAL_TOPDIR), $$(CANONICAL_TOPDIR)) |
|
29663
74ff65003536
8076123: 9-dev build fail: make/Init.gmk:142: *** multiple target patterns. Stop.
erikj
parents:
29662
diff
changeset
|
367 |
$$(info Error: SPEC mismatch! Current working directory) |
29788 | 368 |
$$(info $$(ACTUAL_TOPDIR)) |
29663
74ff65003536
8076123: 9-dev build fail: make/Init.gmk:142: *** multiple target patterns. Stop.
erikj
parents:
29662
diff
changeset
|
369 |
$$(info does not match either TOPDIR, ORIGINAL_TOPDIR or CANONICAL_TOPDIR) |
74ff65003536
8076123: 9-dev build fail: make/Init.gmk:142: *** multiple target patterns. Stop.
erikj
parents:
29662
diff
changeset
|
370 |
$$(info $$(TOPDIR)) |
74ff65003536
8076123: 9-dev build fail: make/Init.gmk:142: *** multiple target patterns. Stop.
erikj
parents:
29662
diff
changeset
|
371 |
$$(info $$(ORIGINAL_TOPDIR)) |
74ff65003536
8076123: 9-dev build fail: make/Init.gmk:142: *** multiple target patterns. Stop.
erikj
parents:
29662
diff
changeset
|
372 |
$$(info $$(CANONICAL_TOPDIR)) |
74ff65003536
8076123: 9-dev build fail: make/Init.gmk:142: *** multiple target patterns. Stop.
erikj
parents:
29662
diff
changeset
|
373 |
$$(error Cannot continue) |
74ff65003536
8076123: 9-dev build fail: make/Init.gmk:142: *** multiple target patterns. Stop.
erikj
parents:
29662
diff
changeset
|
374 |
endif |
74ff65003536
8076123: 9-dev build fail: make/Init.gmk:142: *** multiple target patterns. Stop.
erikj
parents:
29662
diff
changeset
|
375 |
endif |
29662 | 376 |
endif |
377 |
endef |
|
27560 | 378 |
|
32715
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
379 |
# Parse COMPARE_BUILD into COMPARE_BUILD_* |
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
380 |
# Syntax: COMPARE_BUILD=CONF=<configure options>:PATCH=<patch file>: |
32720
7e0e586a6817
8137014: Various improvements in build infrastructure
ihse
parents:
32715
diff
changeset
|
381 |
# MAKE=<make targets>:COMP_OPTS=<compare script options>: |
35744
4f5e0998b6e9
8149479: Fix compare.sh to have a clean baseline with COMPARE_BUILD
erikj
parents:
35444
diff
changeset
|
382 |
# COMP_DIR=<compare script base dir>|<default>: |
4f5e0998b6e9
8149479: Fix compare.sh to have a clean baseline with COMPARE_BUILD
erikj
parents:
35444
diff
changeset
|
383 |
# FAIL=<bool> |
32715
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
384 |
# If neither CONF or PATCH is given, assume <default> means CONF if it |
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
385 |
# begins with "--", otherwise assume it means PATCH. |
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
386 |
# MAKE and COMP_OPTS can only be used with CONF and/or PATCH specified. |
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
387 |
# If any value contains "+", it will be replaced by space. |
35744
4f5e0998b6e9
8149479: Fix compare.sh to have a clean baseline with COMPARE_BUILD
erikj
parents:
35444
diff
changeset
|
388 |
# FAIL can be set to false to have the return value of compare be ignored. |
32715
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
389 |
define ParseCompareBuild |
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
390 |
ifneq ($$(COMPARE_BUILD), ) |
47253
92fd0e04e0e1
8187544: Replace BUILD_OUTPUT and OUTPUT_ROOT with OUTPUTDIR
ihse
parents:
47217
diff
changeset
|
391 |
COMPARE_BUILD_OUTPUTDIR := $(TOPDIR)/build/compare-build/$(CONF_NAME) |
35744
4f5e0998b6e9
8149479: Fix compare.sh to have a clean baseline with COMPARE_BUILD
erikj
parents:
35444
diff
changeset
|
392 |
COMPARE_BUILD_FAIL := true |
33572 | 393 |
|
32715
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
394 |
ifneq ($$(findstring :, $$(COMPARE_BUILD)), ) |
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
395 |
$$(foreach part, $$(subst :, , $$(COMPARE_BUILD)), \ |
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
396 |
$$(if $$(filter PATCH=%, $$(part)), \ |
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
397 |
$$(eval COMPARE_BUILD_PATCH=$$(strip $$(patsubst PATCH=%, %, $$(part)))) \ |
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
398 |
) \ |
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
399 |
$$(if $$(filter CONF=%, $$(part)), \ |
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
400 |
$$(eval COMPARE_BUILD_CONF=$$(strip $$(subst +, , $$(patsubst CONF=%, %, $$(part))))) \ |
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
401 |
) \ |
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
402 |
$$(if $$(filter MAKE=%, $$(part)), \ |
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
403 |
$$(eval COMPARE_BUILD_MAKE=$$(strip $$(subst +, , $$(patsubst MAKE=%, %, $$(part))))) \ |
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
404 |
) \ |
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
405 |
$$(if $$(filter COMP_OPTS=%, $$(part)), \ |
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
406 |
$$(eval COMPARE_BUILD_COMP_OPTS=$$(strip $$(subst +, , $$(patsubst COMP_OPTS=%, %, $$(part))))) \ |
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
407 |
) \ |
32720
7e0e586a6817
8137014: Various improvements in build infrastructure
ihse
parents:
32715
diff
changeset
|
408 |
$$(if $$(filter COMP_DIR=%, $$(part)), \ |
7e0e586a6817
8137014: Various improvements in build infrastructure
ihse
parents:
32715
diff
changeset
|
409 |
$$(eval COMPARE_BUILD_COMP_DIR=$$(strip $$(subst +, , $$(patsubst COMP_DIR=%, %, $$(part))))) \ |
7e0e586a6817
8137014: Various improvements in build infrastructure
ihse
parents:
32715
diff
changeset
|
410 |
) \ |
35744
4f5e0998b6e9
8149479: Fix compare.sh to have a clean baseline with COMPARE_BUILD
erikj
parents:
35444
diff
changeset
|
411 |
$$(if $$(filter FAIL=%, $$(part)), \ |
4f5e0998b6e9
8149479: Fix compare.sh to have a clean baseline with COMPARE_BUILD
erikj
parents:
35444
diff
changeset
|
412 |
$$(eval COMPARE_BUILD_FAIL=$$(strip $$(subst +, , $$(patsubst FAIL=%, %, $$(part))))) \ |
4f5e0998b6e9
8149479: Fix compare.sh to have a clean baseline with COMPARE_BUILD
erikj
parents:
35444
diff
changeset
|
413 |
) \ |
32715
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
414 |
) |
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
415 |
else |
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
416 |
# Separate handling for single field case, to allow for spaces in values. |
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
417 |
ifneq ($$(filter PATCH=%, $$(COMPARE_BUILD)), ) |
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
418 |
COMPARE_BUILD_PATCH=$$(strip $$(patsubst PATCH=%, %, $$(COMPARE_BUILD))) |
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
419 |
else ifneq ($$(filter CONF=%, $$(COMPARE_BUILD)), ) |
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
420 |
COMPARE_BUILD_CONF=$$(strip $$(subst +, , $$(patsubst CONF=%, %, $$(COMPARE_BUILD)))) |
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
421 |
else ifneq ($$(filter --%, $$(COMPARE_BUILD)), ) |
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
422 |
# Assume CONF if value begins with -- |
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
423 |
COMPARE_BUILD_CONF=$$(strip $$(subst +, , $$(COMPARE_BUILD))) |
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
424 |
else |
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
425 |
# Otherwise assume patch file |
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
426 |
COMPARE_BUILD_PATCH=$$(strip $$(COMPARE_BUILD)) |
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
427 |
endif |
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
428 |
endif |
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
429 |
ifneq ($$(COMPARE_BUILD_PATCH), ) |
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
430 |
ifneq ($$(wildcard $$(TOPDIR)/$$(COMPARE_BUILD_PATCH)), ) |
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
431 |
# Assume relative path, if file exists |
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
432 |
COMPARE_BUILD_PATCH := $$(wildcard $$(TOPDIR)/$$(COMPARE_BUILD_PATCH)) |
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
433 |
else ifeq ($$(wildcard $$(COMPARE_BUILD_PATCH)), ) |
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
434 |
$$(error Patch file $$(COMPARE_BUILD_PATCH) does not exist) |
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
435 |
endif |
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
436 |
endif |
35744
4f5e0998b6e9
8149479: Fix compare.sh to have a clean baseline with COMPARE_BUILD
erikj
parents:
35444
diff
changeset
|
437 |
ifneq ($$(COMPARE_BUILD_FAIL), true) |
4f5e0998b6e9
8149479: Fix compare.sh to have a clean baseline with COMPARE_BUILD
erikj
parents:
35444
diff
changeset
|
438 |
COMPARE_BUILD_IGNORE_RESULT := || true |
4f5e0998b6e9
8149479: Fix compare.sh to have a clean baseline with COMPARE_BUILD
erikj
parents:
35444
diff
changeset
|
439 |
endif |
32715
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
440 |
endif |
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
441 |
endef |
4d558a41a1ec
8136695: Automatic build comparison with COMPARE_BUILD
ihse
parents:
32344
diff
changeset
|
442 |
|
33572 | 443 |
# Prepare for a comparison rebuild |
444 |
define PrepareCompareBuild |
|
445 |
$(ECHO) "Preparing for comparison rebuild" |
|
446 |
# Apply patch, if any |
|
447 |
$(if $(COMPARE_BUILD_PATCH), $(PATCH) -p1 < $(COMPARE_BUILD_PATCH)) |
|
448 |
# Move the first build away temporarily |
|
449 |
$(RM) -r $(TOPDIR)/build/.compare-build-temp |
|
450 |
$(MKDIR) -p $(TOPDIR)/build/.compare-build-temp |
|
47253
92fd0e04e0e1
8187544: Replace BUILD_OUTPUT and OUTPUT_ROOT with OUTPUTDIR
ihse
parents:
47217
diff
changeset
|
451 |
$(MV) $(OUTPUTDIR) $(TOPDIR)/build/.compare-build-temp |
33572 | 452 |
# Restore an old compare-build, or create a new compare-build directory. |
47253
92fd0e04e0e1
8187544: Replace BUILD_OUTPUT and OUTPUT_ROOT with OUTPUTDIR
ihse
parents:
47217
diff
changeset
|
453 |
if test -d $(COMPARE_BUILD_OUTPUTDIR); then \ |
92fd0e04e0e1
8187544: Replace BUILD_OUTPUT and OUTPUT_ROOT with OUTPUTDIR
ihse
parents:
47217
diff
changeset
|
454 |
$(MV) $(COMPARE_BUILD_OUTPUTDIR) $(OUTPUTDIR); \ |
33572 | 455 |
else \ |
47253
92fd0e04e0e1
8187544: Replace BUILD_OUTPUT and OUTPUT_ROOT with OUTPUTDIR
ihse
parents:
47217
diff
changeset
|
456 |
$(MKDIR) -p $(OUTPUTDIR); \ |
33572 | 457 |
fi |
458 |
# Re-run configure with the same arguments (and possibly some additional), |
|
459 |
# must be done after patching. |
|
47253
92fd0e04e0e1
8187544: Replace BUILD_OUTPUT and OUTPUT_ROOT with OUTPUTDIR
ihse
parents:
47217
diff
changeset
|
460 |
( cd $(OUTPUTDIR) && PATH="$(ORIGINAL_PATH)" \ |
33572 | 461 |
$(BASH) $(TOPDIR)/configure $(CONFIGURE_COMMAND_LINE) $(COMPARE_BUILD_CONF)) |
462 |
endef |
|
463 |
||
464 |
# Cleanup after a compare build |
|
465 |
define CleanupCompareBuild |
|
466 |
# If running with a COMPARE_BUILD patch, reverse-apply it |
|
467 |
$(if $(COMPARE_BUILD_PATCH), $(PATCH) -R -p1 < $(COMPARE_BUILD_PATCH)) |
|
468 |
# Move this build away and restore the original build |
|
469 |
$(MKDIR) -p $(TOPDIR)/build/compare-build |
|
47253
92fd0e04e0e1
8187544: Replace BUILD_OUTPUT and OUTPUT_ROOT with OUTPUTDIR
ihse
parents:
47217
diff
changeset
|
470 |
$(MV) $(OUTPUTDIR) $(COMPARE_BUILD_OUTPUTDIR) |
92fd0e04e0e1
8187544: Replace BUILD_OUTPUT and OUTPUT_ROOT with OUTPUTDIR
ihse
parents:
47217
diff
changeset
|
471 |
$(MV) $(TOPDIR)/build/.compare-build-temp/$(CONF_NAME) $(OUTPUTDIR) |
33572 | 472 |
$(RM) -r $(TOPDIR)/build/.compare-build-temp |
473 |
endef |
|
474 |
||
475 |
# Do the actual comparison of two builds |
|
476 |
define CompareBuildDoComparison |
|
477 |
# Compare first and second build. Ignore any error code from compare.sh. |
|
478 |
$(ECHO) "Comparing between comparison rebuild (this/new) and baseline (other/old)" |
|
479 |
$(if $(COMPARE_BUILD_COMP_DIR), \ |
|
47253
92fd0e04e0e1
8187544: Replace BUILD_OUTPUT and OUTPUT_ROOT with OUTPUTDIR
ihse
parents:
47217
diff
changeset
|
480 |
+(cd $(COMPARE_BUILD_OUTPUTDIR) && ./compare.sh $(COMPARE_BUILD_COMP_OPTS) \ |
92fd0e04e0e1
8187544: Replace BUILD_OUTPUT and OUTPUT_ROOT with OUTPUTDIR
ihse
parents:
47217
diff
changeset
|
481 |
-2dirs $(COMPARE_BUILD_OUTPUTDIR)/$(COMPARE_BUILD_COMP_DIR) \ |
92fd0e04e0e1
8187544: Replace BUILD_OUTPUT and OUTPUT_ROOT with OUTPUTDIR
ihse
parents:
47217
diff
changeset
|
482 |
$(OUTPUTDIR)/$(COMPARE_BUILD_COMP_DIR) $(COMPARE_BUILD_IGNORE_RESULT)), \ |
92fd0e04e0e1
8187544: Replace BUILD_OUTPUT and OUTPUT_ROOT with OUTPUTDIR
ihse
parents:
47217
diff
changeset
|
483 |
+(cd $(COMPARE_BUILD_OUTPUTDIR) && ./compare.sh $(COMPARE_BUILD_COMP_OPTS) \ |
92fd0e04e0e1
8187544: Replace BUILD_OUTPUT and OUTPUT_ROOT with OUTPUTDIR
ihse
parents:
47217
diff
changeset
|
484 |
-o $(OUTPUTDIR) $(COMPARE_BUILD_IGNORE_RESULT)) \ |
33572 | 485 |
) |
486 |
endef |
|
487 |
||
488 |
define PrintFailureReports |
|
43173 | 489 |
$(if $(wildcard $(MAKESUPPORT_OUTPUTDIR)/failure-logs/*.log), \ |
490 |
$(PRINTF) "\n=== Output from failing command(s) repeated here ===\n" $(NEWLINE) \ |
|
491 |
$(foreach logfile, $(sort $(wildcard $(MAKESUPPORT_OUTPUTDIR)/failure-logs/*.log)), \ |
|
33572 | 492 |
$(PRINTF) "* For target $(notdir $(basename $(logfile))):\n" $(NEWLINE) \ |
35370
f9b430645a18
8148120: Incremental update from build-infra project
ihse
parents:
35008
diff
changeset
|
493 |
($(GREP) -v -e "^Note: including file:" < $(logfile) || true) | $(HEAD) -n 12 $(NEWLINE) \ |
f9b430645a18
8148120: Incremental update from build-infra project
ihse
parents:
35008
diff
changeset
|
494 |
if test `$(WC) -l < $(logfile)` -gt 12; then \ |
f9b430645a18
8148120: Incremental update from build-infra project
ihse
parents:
35008
diff
changeset
|
495 |
$(ECHO) " ... (rest of output omitted)" ; \ |
f9b430645a18
8148120: Incremental update from build-infra project
ihse
parents:
35008
diff
changeset
|
496 |
fi $(NEWLINE) \ |
33572 | 497 |
) \ |
43173 | 498 |
$(PRINTF) "\n* All command lines available in $(MAKESUPPORT_OUTPUTDIR)/failure-logs.\n" $(NEWLINE) \ |
33572 | 499 |
$(PRINTF) "=== End of repeated output ===\n" \ |
500 |
) |
|
501 |
endef |
|
502 |
||
503 |
define PrintBuildLogFailures |
|
504 |
if $(GREP) -q "recipe for target .* failed" $(BUILD_LOG) 2> /dev/null; then \ |
|
43173 | 505 |
$(PRINTF) "\n=== Make failed targets repeated here ===\n" ; \ |
33572 | 506 |
$(GREP) "recipe for target .* failed" $(BUILD_LOG) ; \ |
507 |
$(PRINTF) "=== End of repeated output ===\n" ; \ |
|
43173 | 508 |
$(PRINTF) "\nHint: Try searching the build log for the name of the first failed target.\n" ; \ |
33572 | 509 |
else \ |
43173 | 510 |
$(PRINTF) "\nNo indication of failed target found.\n" ; \ |
33572 | 511 |
$(PRINTF) "Hint: Try searching the build log for '] Error'.\n" ; \ |
512 |
fi |
|
513 |
endef |
|
514 |
||
29662 | 515 |
define RotateLogFiles |
35008 | 516 |
$(RM) $(BUILD_LOG).old 2> /dev/null && \ |
29662 | 517 |
$(MV) $(BUILD_LOG) $(BUILD_LOG).old 2> /dev/null || true |
44465
26fabd8abee9
8177770: Need more precise control on build system logging
asemenyuk
parents:
44027
diff
changeset
|
518 |
$(if $(findstring true, $(LOG_PROFILE_TIMES_FILE)), \ |
26fabd8abee9
8177770: Need more precise control on build system logging
asemenyuk
parents:
44027
diff
changeset
|
519 |
$(RM) $(BUILD_PROFILE_LOG).old 2> /dev/null && \ |
26fabd8abee9
8177770: Need more precise control on build system logging
asemenyuk
parents:
44027
diff
changeset
|
520 |
$(MV) $(BUILD_PROFILE_LOG) $(BUILD_PROFILE_LOG).old 2> /dev/null || true \ |
29662 | 521 |
) |
522 |
endef |
|
26116
f051bd253364
8055095: Improve "do nothing" incremental build performance after modularized source code integration
erikj
parents:
25854
diff
changeset
|
523 |
|
32344
0b288e0efcfa
8062618: Create a build failure summary at end of build log
ihse
parents:
29788
diff
changeset
|
524 |
define PrepareFailureLogs |
35008 | 525 |
$(RM) -r $(MAKESUPPORT_OUTPUTDIR)/failure-logs 2> /dev/null && \ |
32344
0b288e0efcfa
8062618: Create a build failure summary at end of build log
ihse
parents:
29788
diff
changeset
|
526 |
$(MKDIR) -p $(MAKESUPPORT_OUTPUTDIR)/failure-logs |
44027 | 527 |
$(RM) $(MAKESUPPORT_OUTPUTDIR)/exit-with-error 2> /dev/null |
32344
0b288e0efcfa
8062618: Create a build failure summary at end of build log
ihse
parents:
29788
diff
changeset
|
528 |
endef |
0b288e0efcfa
8062618: Create a build failure summary at end of build log
ihse
parents:
29788
diff
changeset
|
529 |
|
29662 | 530 |
# Remove any javac server logs and port files. This |
531 |
# prevents a new make run to reuse the previous servers. |
|
532 |
define PrepareSmartJavac |
|
533 |
$(if $(SJAVAC_SERVER_DIR), \ |
|
534 |
$(RM) -r $(SJAVAC_SERVER_DIR) 2> /dev/null && \ |
|
535 |
$(MKDIR) -p $(SJAVAC_SERVER_DIR) \ |
|
536 |
) |
|
537 |
endef |
|
27560 | 538 |
|
29662 | 539 |
define CleanupSmartJavac |
540 |
[ -f $(SJAVAC_SERVER_DIR)/server.port ] && $(ECHO) Stopping sjavac server && \ |
|
541 |
$(TOUCH) $(SJAVAC_SERVER_DIR)/server.port.stop; true |
|
542 |
endef |
|
26116
f051bd253364
8055095: Improve "do nothing" incremental build performance after modularized source code integration
erikj
parents:
25854
diff
changeset
|
543 |
|
35370
f9b430645a18
8148120: Incremental update from build-infra project
ihse
parents:
35008
diff
changeset
|
544 |
ifeq ($(OPENJDK_BUILD_OS), windows) |
f9b430645a18
8148120: Incremental update from build-infra project
ihse
parents:
35008
diff
changeset
|
545 |
# On windows we need to synchronize with the javac server to be able to |
f9b430645a18
8148120: Incremental update from build-infra project
ihse
parents:
35008
diff
changeset
|
546 |
# move or remove the build output directory. Since we have no proper |
f9b430645a18
8148120: Incremental update from build-infra project
ihse
parents:
35008
diff
changeset
|
547 |
# synchronization process, wait for a while and hope it helps. This is only |
f9b430645a18
8148120: Incremental update from build-infra project
ihse
parents:
35008
diff
changeset
|
548 |
# used by build comparisons. |
f9b430645a18
8148120: Incremental update from build-infra project
ihse
parents:
35008
diff
changeset
|
549 |
define WaitForSmartJavacFinish |
f9b430645a18
8148120: Incremental update from build-infra project
ihse
parents:
35008
diff
changeset
|
550 |
$(if $(SJAVAC_SERVER_DIR), \ |
f9b430645a18
8148120: Incremental update from build-infra project
ihse
parents:
35008
diff
changeset
|
551 |
sleep 5\ |
f9b430645a18
8148120: Incremental update from build-infra project
ihse
parents:
35008
diff
changeset
|
552 |
) |
f9b430645a18
8148120: Incremental update from build-infra project
ihse
parents:
35008
diff
changeset
|
553 |
endef |
f9b430645a18
8148120: Incremental update from build-infra project
ihse
parents:
35008
diff
changeset
|
554 |
else |
f9b430645a18
8148120: Incremental update from build-infra project
ihse
parents:
35008
diff
changeset
|
555 |
define WaitForSmartJavacFinish |
f9b430645a18
8148120: Incremental update from build-infra project
ihse
parents:
35008
diff
changeset
|
556 |
endef |
f9b430645a18
8148120: Incremental update from build-infra project
ihse
parents:
35008
diff
changeset
|
557 |
endif |
f9b430645a18
8148120: Incremental update from build-infra project
ihse
parents:
35008
diff
changeset
|
558 |
|
29662 | 559 |
define StartGlobalTimer |
35008 | 560 |
$(RM) -r $(BUILDTIMESDIR) 2> /dev/null && \ |
561 |
$(MKDIR) -p $(BUILDTIMESDIR) && \ |
|
29662 | 562 |
$(call RecordStartTime,TOTAL) |
563 |
endef |
|
27560 | 564 |
|
29662 | 565 |
define StopGlobalTimer |
566 |
$(call RecordEndTime,TOTAL) |
|
567 |
endef |
|
27560 | 568 |
|
29662 | 569 |
# Find all build_time_* files and print their contents in a list sorted |
570 |
# on the name of the sub repository. |
|
571 |
define ReportBuildTimes |
|
35008 | 572 |
$(PRINTF) $(LOG_INFO) -- \ |
29662 | 573 |
"----- Build times -------\nStart %s\nEnd %s\n%s\n%s\n-------------------------\n" \ |
574 |
"`$(CAT) $(BUILDTIMESDIR)/build_time_start_TOTAL_human_readable`" \ |
|
575 |
"`$(CAT) $(BUILDTIMESDIR)/build_time_end_TOTAL_human_readable`" \ |
|
576 |
"`$(LS) $(BUILDTIMESDIR)/build_time_diff_* | $(GREP) -v _TOTAL | \ |
|
577 |
$(XARGS) $(CAT) | $(SORT) -k 2`" \ |
|
35008 | 578 |
"`$(CAT) $(BUILDTIMESDIR)/build_time_diff_TOTAL`" \ |
579 |
$(BUILD_LOG_PIPE) |
|
29662 | 580 |
endef |
27560 | 581 |
|
44465
26fabd8abee9
8177770: Need more precise control on build system logging
asemenyuk
parents:
44027
diff
changeset
|
582 |
define ReportProfileTimes |
26fabd8abee9
8177770: Need more precise control on build system logging
asemenyuk
parents:
44027
diff
changeset
|
583 |
$(if $(findstring true, $(LOG_PROFILE_TIMES_LOG)), \ |
26fabd8abee9
8177770: Need more precise control on build system logging
asemenyuk
parents:
44027
diff
changeset
|
584 |
[ ! -f $(BUILD_PROFILE_LOG) ] || \ |
26fabd8abee9
8177770: Need more precise control on build system logging
asemenyuk
parents:
44027
diff
changeset
|
585 |
{ $(ECHO) Begin $(notdir $(BUILD_PROFILE_LOG)) && \ |
26fabd8abee9
8177770: Need more precise control on build system logging
asemenyuk
parents:
44027
diff
changeset
|
586 |
$(CAT) $(BUILD_PROFILE_LOG) && \ |
26fabd8abee9
8177770: Need more precise control on build system logging
asemenyuk
parents:
44027
diff
changeset
|
587 |
$(ECHO) End $(notdir $(BUILD_PROFILE_LOG)); \ |
26fabd8abee9
8177770: Need more precise control on build system logging
asemenyuk
parents:
44027
diff
changeset
|
588 |
} \ |
26fabd8abee9
8177770: Need more precise control on build system logging
asemenyuk
parents:
44027
diff
changeset
|
589 |
$(BUILD_LOG_PIPE) |
26fabd8abee9
8177770: Need more precise control on build system logging
asemenyuk
parents:
44027
diff
changeset
|
590 |
) |
26fabd8abee9
8177770: Need more precise control on build system logging
asemenyuk
parents:
44027
diff
changeset
|
591 |
endef |
26fabd8abee9
8177770: Need more precise control on build system logging
asemenyuk
parents:
44027
diff
changeset
|
592 |
|
29662 | 593 |
endif # HAS_SPEC |
27560 | 594 |
|
44465
26fabd8abee9
8177770: Need more precise control on build system logging
asemenyuk
parents:
44027
diff
changeset
|
595 |
MAKE_LOG_VARS = $(foreach v, \ |
26fabd8abee9
8177770: Need more precise control on build system logging
asemenyuk
parents:
44027
diff
changeset
|
596 |
LOG_LEVEL LOG_NOFILE LOG_CMDLINES LOG_PROFILE_TIMES_LOG LOG_PROFILE_TIMES_FILE, \ |
26fabd8abee9
8177770: Need more precise control on build system logging
asemenyuk
parents:
44027
diff
changeset
|
597 |
$v=$($v) \ |
26fabd8abee9
8177770: Need more precise control on build system logging
asemenyuk
parents:
44027
diff
changeset
|
598 |
) |
26fabd8abee9
8177770: Need more precise control on build system logging
asemenyuk
parents:
44027
diff
changeset
|
599 |
|
29662 | 600 |
endif # _INITSUPPORT_GMK |