Skip to content

Commit

Permalink
Restore msgbox_mask package defaults (#2527)
Browse files Browse the repository at this point in the history
Updates DR to
[a15656a0c](36a6d23).
Replaces the AUTOMATED_TESTING set in package builds by PR #2474 with
the new DISABLE_ZLIB CMake option added by
DynamoRIO/dynamorio#7030. This fixes a regression where msgbox_mask was
set to 0 by default in packages, which caused many users to fail to
obtain error information and has led to confusion with silent errors.

Tested:
Built
https://github.com/DynamoRIO/drmemory/releases/tag/cronbuild-2.6.20005,
unzipped it, and confirmed it pops up a message box by default on the
machine where DynamoRIO/dynamorio#7024 is hit.

Fixes DynamoRIO/dynamorio#7025
  • Loading branch information
derekbruening authored Oct 9, 2024
1 parent 384f6aa commit f62daf1
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions package.cmake
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
# **********************************************************
# Copyright (c) 2010-2022 Google, Inc. All rights reserved.
# Copyright (c) 2010-2024 Google, Inc. All rights reserved.
# Copyright (c) 2009-2010 VMware, Inc. All rights reserved.
# **********************************************************

Expand Down Expand Up @@ -136,7 +136,7 @@ set(base_cache "
UNIQUE_BUILD_NUMBER:STRING=${arg_ubuild}
BUILD_TOOL_TESTS:BOOL=OFF
BUILDING_PACKAGE:BOOL=ON
AUTOMATED_TESTING:BOOL=ON
DISABLE_ZLIB:BOOL=ON
${sub_entry}
${arg_cacheappend}
${base_cache}
Expand Down

0 comments on commit f62daf1

Please sign in to comment.