[前][次][番号順一覧][スレッド一覧]

ruby-changes:70693

From: Nobuyoshi <ko1@a...>
Date: Sun, 2 Jan 2022 02:17:47 +0900 (JST)
Subject: [ruby-changes:70693] 071653ecd7 (master): Run the prerequisites of test-bundled-gems in order

https://git.ruby-lang.org/ruby.git/commit/?id=071653ecd7

From 071653ecd79287661c795400797a5402c4322129 Mon Sep 17 00:00:00 2001
From: Nobuyoshi Nakada <nobu@r...>
Date: Sat, 1 Jan 2022 22:17:14 +0900
Subject: Run the prerequisites of test-bundled-gems in order

---
 common.mk     | 6 +++++-
 defs/gmake.mk | 7 ++++++-
 2 files changed, 11 insertions(+), 2 deletions(-)

diff --git a/common.mk b/common.mk
index 664f750a16b..98d3993b232 100644
--- a/common.mk
+++ b/common.mk
@@ -1376,8 +1376,12 @@ test-bundled-gems-precheck: $(TEST_RUNNABLE)-test-bundled-gems-precheck https://github.com/ruby/ruby/blob/trunk/common.mk#L1376
 yes-test-bundled-gems-precheck: main
 no-test-bundled-gems-precheck:
 
-test-bundled-gems-fetch: $(PREP)
+test-bundled-gems-fetch: yes-test-bundled-gems-fetch
+yes-test-bundled-gems-fetch: $(PREP)
+	$(ACTIONS_GROUP)
 	$(Q) $(BASERUBY) -C $(srcdir)/gems ../tool/fetch-bundled_gems.rb src bundled_gems
+	$(ACTIONS_ENDGROUP)
+no-test-bundled-gems-fetch:
 
 test-bundled-gems-prepare: test-bundled-gems-precheck test-bundled-gems-fetch
 test-bundled-gems-prepare: $(TEST_RUNNABLE)-test-bundled-gems-prepare
diff --git a/defs/gmake.mk b/defs/gmake.mk
index dd7da5f7707..da1bdb4b7c2 100644
--- a/defs/gmake.mk
+++ b/defs/gmake.mk
@@ -7,7 +7,7 @@ MSPECOPT += $(if $(filter -j%,$(MFLAGS)),-j) https://github.com/ruby/ruby/blob/trunk/defs/gmake.mk#L7
 nproc = $(subst -j,,$(filter -j%,$(MFLAGS)))
 
 ifeq ($(GITHUB_ACTIONS),true)
-override ACTIONS_GROUP = @echo "\#\#[group]$(@:yes-=)"
+override ACTIONS_GROUP = @echo "\#\#[group]$(patsubst yes-%,%,$@)"
 override ACTIONS_ENDGROUP = @echo "\#\#[endgroup]"
 endif
 
@@ -33,6 +33,9 @@ TEST_DEPENDS := $(filter-out test-all $(TEST_TARGETS),$(TEST_DEPENDS)) https://github.com/ruby/ruby/blob/trunk/defs/gmake.mk#L33
 TEST_TARGETS := $(patsubst test,test-short,$(TEST_TARGETS))
 TEST_DEPENDS := $(filter-out test $(TEST_TARGETS),$(TEST_DEPENDS))
 TEST_TARGETS := $(patsubst test-short,btest-ruby test-knownbug test-basic,$(TEST_TARGETS))
+TEST_TARGETS := $(patsubst test-bundled-gems,test-bundled-gems-run,$(TEST_TARGETS))
+TEST_TARGETS := $(patsubst test-bundled-gems-run,test-bundled-gems-run $(PREPARE_BUNDLED_GEMS),$(TEST_TARGETS))
+TEST_TARGETS := $(patsubst test-bundled-gems-prepare,test-bundled-gems-prepare test-bundled-gems-precheck test-bundled-gems-fetch,$(TEST_TARGETS))
 TEST_DEPENDS := $(filter-out test-short $(TEST_TARGETS),$(TEST_DEPENDS))
 TEST_DEPENDS += $(if $(filter great exam love check,$(MAKECMDGOALS)),all exts)
 
@@ -79,6 +82,8 @@ ORDERED_TEST_TARGETS := $(filter $(TEST_TARGETS), \ https://github.com/ruby/ruby/blob/trunk/defs/gmake.mk#L82
 	btest-ruby test-knownbug test-basic \
 	test-testframework test-tool test-ruby test-all \
 	test-spec test-bundler-prepare test-bundler test-bundler-parallel \
+	test-bundled-gems-precheck test-bundled-gems-fetch \
+	test-bundled-gems-prepare test-bundled-gems-run \
 	)
 prev_test := $(if $(filter test-spec,$(ORDERED_TEST_TARGETS)),test-spec-precheck)
 $(foreach test,$(ORDERED_TEST_TARGETS), \
-- 
cgit v1.2.1


--
ML: ruby-changes@q...
Info: http://www.atdot.net/~ko1/quickml/

[前][次][番号順一覧][スレッド一覧]