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