Skip to content

Commit

Permalink
Remove vmsafe build from pre-commit suite
Browse files Browse the repository at this point in the history
With all of the cross-compilation builds and now dynamorio_static, the
pre-commit suite is getting too long.  We remove the vmsafe build as it's
not really supported anymore.

Review-URL: https://codereview.appspot.com/310140043
  • Loading branch information
derekbruening committed Aug 31, 2016
1 parent d494a0e commit 73348c8
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions suite/runsuite.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -235,12 +235,12 @@ if (NOT cross_only)
${install_path_cache}
")
endif (DO_ALL_BUILDS)
# non-official-API builds but not all are in pre-commit suite on Windows
# Non-official-API builds but not all are in pre-commit suite, esp on Windows
# where building is slow: we'll rely on bots to catch breakage in most of these
# builds on Windows
# builds.
if (ARCH_IS_X86 AND NOT APPLE)
# we do not bother to support these on ARM
if (UNIX OR DO_ALL_BUILDS)
if (DO_ALL_BUILDS)
testbuild("vmsafe-debug-internal-32" OFF "
VMAP:BOOL=OFF
VMSAFE:BOOL=ON
Expand Down

0 comments on commit 73348c8

Please sign in to comment.