From 716102fff38f4067d9b9fcf6bb0ff9e6dbd78105 Mon Sep 17 00:00:00 2001 From: KOBAYASHI Shuji Date: Thu, 26 Dec 2019 17:18:41 +0900 Subject: Enable `MRB_GC_STRESS` test on CI The `debug` build target (`MRB_GC_STRESS` is enabled) on CI have been compiled but not tested so far. However, I think testing with `MRB_GC_STRESS` is effective (in fact, I found #4907 bug). Therefore, I integrated `debug` and `full-debug` build targets to enable `MRB_GC_STRESS` testing. Testing with `MRB_GC_STRESS` takes a little time, but compiling takes more time, so CI execution time does not increase due to decrease of build target. --- appveyor_config.rb | 15 +-------------- travis_config.rb | 4 ++-- 2 files changed, 3 insertions(+), 16 deletions(-) diff --git a/appveyor_config.rb b/appveyor_config.rb index b50a9e4ef..fd14f4a4c 100644 --- a/appveyor_config.rb +++ b/appveyor_config.rb @@ -1,23 +1,10 @@ -MRuby::Build.new('debug') do |conf| - toolchain :visualcpp - enable_debug - - # include all core GEMs - conf.gembox 'full-core' - conf.compilers.each do |c| - c.defines += %w(MRB_GC_STRESS MRB_GC_FIXED_ARENA MRB_METHOD_CACHE) - end - - build_mrbc_exec -end - MRuby::Build.new('full-debug') do |conf| toolchain :visualcpp enable_debug # include all core GEMs conf.gembox 'full-core' - conf.cc.defines += %w(MRB_ENABLE_DEBUG_HOOK) + conf.cc.defines += %w(MRB_GC_STRESS MRB_METHOD_CACHE MRB_ENABLE_DEBUG_HOOK) conf.enable_test end diff --git a/travis_config.rb b/travis_config.rb index 35a086e1f..a8189dee6 100644 --- a/travis_config.rb +++ b/travis_config.rb @@ -19,9 +19,9 @@ MRuby::Build.new('full-debug') do |conf| # include all core GEMs conf.gembox 'full-core' - conf.cc.defines += %w(MRB_ENABLE_DEBUG_HOOK) - conf.cc.flags += %w(-fsanitize=address,undefined) + conf.cc.flags += %w(-Werror=declaration-after-statement -fsanitize=address,undefined) conf.linker.flags += %W(-fsanitize=address,undefined) + conf.cc.defines += %w(MRB_GC_STRESS MRB_METHOD_CACHE MRB_ENABLE_DEBUG_HOOK) conf.enable_test end -- cgit v1.2.3