Skip to content

Commit

Permalink
JBR-8219 revert "8292944: Noisy output when running make help the fir…
Browse files Browse the repository at this point in the history
…st time"

This reverts commit fcd5ebc.
  • Loading branch information
vprovodin committed Feb 4, 2025
1 parent 8482b7e commit c69e189
Show file tree
Hide file tree
Showing 5 changed files with 7 additions and 19 deletions.
6 changes: 0 additions & 6 deletions make/Init.gmk
Original file line number Diff line number Diff line change
Expand Up @@ -47,12 +47,6 @@ include $(TOPDIR)/make/common/MakeBase.gmk
include $(TOPDIR)/make/InitSupport.gmk
include $(TOPDIR)/make/common/LogUtils.gmk

# Force early generation of module-deps.gmk and find-tests.gmk
GENERATE_MODULE_DEPS_FILE := true
include $(TOPDIR)/make/common/Modules.gmk
GENERATE_FIND_TESTS_FILE := true
include $(TOPDIR)/make/common/FindTests.gmk

# Parse COMPARE_BUILD (for makefile development)
$(eval $(call ParseCompareBuild))

Expand Down
1 change: 0 additions & 1 deletion make/Main.gmk
Original file line number Diff line number Diff line change
Expand Up @@ -1483,7 +1483,6 @@ print-tests:

create-main-targets-include:
$(call LogInfo, Generating main target list)
$(call MakeDir, $(MAKESUPPORT_OUTPUTDIR))
@$(ECHO) ALL_MAIN_TARGETS := $(sort $(ALL_TARGETS)) > \
$(MAKESUPPORT_OUTPUTDIR)/main-targets.gmk

Expand Down
3 changes: 1 addition & 2 deletions make/RunTestsPrebuilt.gmk
Original file line number Diff line number Diff line change
Expand Up @@ -301,8 +301,7 @@ test-prebuilt:
# are always found and parsed, call FindTests.gmk stand alone once
# before calling RunTests.gmk.
@cd $(TOPDIR) && $(MAKE) $(MAKE_ARGS) -f make/common/FindTests.gmk \
SPEC=$(SPEC) FINDTESTS_STAND_ALONE=true \
GENERATE_FIND_TESTS_FILE=true
SPEC=$(SPEC) FINDTESTS_STAND_ALONE=true
@cd $(TOPDIR) && $(MAKE) $(MAKE_ARGS) -f make/RunTests.gmk run-test \
TEST="$(TEST)"

Expand Down
10 changes: 4 additions & 6 deletions make/common/FindTests.gmk
Original file line number Diff line number Diff line change
Expand Up @@ -62,11 +62,10 @@ $(foreach root, $(JTREG_TESTROOTS), \
# Cache the expensive to calculate test names in a generated makefile.
FIND_TESTS_CACHE_FILE := $(MAKESUPPORT_OUTPUTDIR)/find-tests.gmk

ifeq ($(GENERATE_FIND_TESTS_FILE), true)
# If this file is deemed outdated, it will automatically get regenerated
# by this rule before being included below.
$(FIND_TESTS_CACHE_FILE): $(JTREG_ROOT_FILES) $(JTREG_GROUP_FILES) \
$(TOPDIR)/test/make/TestMake.gmk
# If this file is deemed outdated, it will automatically get regenerated
# by this rule before being included below.
$(FIND_TESTS_CACHE_FILE): $(JTREG_ROOT_FILES) $(JTREG_GROUP_FILES) \
$(TOPDIR)/test/make/TestMake.gmk
$(call MakeTargetDir)
( $(foreach root, $(JTREG_TESTROOTS), \
$(PRINTF) "\n$(root)_JTREG_TEST_GROUPS := " ; \
Expand All @@ -79,7 +78,6 @@ ifeq ($(GENERATE_FIND_TESTS_FILE), true)
$(MAKE) -s --no-print-directory $(MAKE_ARGS) \
SPEC=$(SPEC) -f $(TOPDIR)/test/make/TestMake.gmk print-targets \
TARGETS_FILE=$@
endif

-include $(FIND_TESTS_CACHE_FILE)

Expand Down
6 changes: 2 additions & 4 deletions make/common/Modules.gmk
Original file line number Diff line number Diff line change
Expand Up @@ -172,9 +172,8 @@ MODULE_DEPS_MAKEFILE := $(MAKESUPPORT_OUTPUTDIR)/module-deps.gmk

MODULE_INFOS := $(call FindAllModuleInfos, *)

ifeq ($(GENERATE_MODULE_DEPS_FILE), true)
$(MODULE_DEPS_MAKEFILE): $(MODULE_INFOS) \
$(call DependOnVariable, MODULE_INFOS, $(MAKESUPPORT_OUTPUTDIR)/MODULE_INFOS.vardeps)
$(MODULE_DEPS_MAKEFILE): $(MODULE_INFOS) \
$(call DependOnVariable, MODULE_INFOS, $(MAKESUPPORT_OUTPUTDIR)/MODULE_INFOS.vardeps)
$(call MakeTargetDir)
$(RM) $@
$(foreach m, $(MODULE_INFOS), \
Expand Down Expand Up @@ -207,7 +206,6 @@ ifeq ($(GENERATE_MODULE_DEPS_FILE), true)
printf(" %s", $$0) } \
END { printf("\n") }' $m \
) >> $@ $(NEWLINE))
endif

-include $(MODULE_DEPS_MAKEFILE)

Expand Down

0 comments on commit c69e189

Please sign in to comment.