Skip to content

Commit

Permalink
i#7025: Restore msgbox_mask package defaults
Browse files Browse the repository at this point in the history
Replaces the AUTOMATED_TESTING set in package builds by PR #5769 with
a new DISABLE_ZLIB CMake option to accomplish the goal of disabling
the zlib found on these VMs while avoiding turning off msgbox_mask.
The disabling by default of msgbox_mask in packages has caused many
users to fail to obtain error information and has led to confusion
with silent errors.

Fixes #7025
  • Loading branch information
derekbruening committed Oct 8, 2024
1 parent e3fbf73 commit 149ef73
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 4 deletions.
5 changes: 3 additions & 2 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
# **********************************************************
# Copyright (c) 2010-2023 Google, Inc. All rights reserved.
# Copyright (c) 2010-2024 Google, Inc. All rights reserved.
# Copyright (c) 2009-2010 VMware, Inc. All rights reserved.
# Copyright (c) 2018 Arm Limited All rights reserved.
# **********************************************************
Expand Down Expand Up @@ -1840,7 +1840,8 @@ endfunction ()
# TODO i#5767: Install an explicit zlib package on our Windows GA CI images
# (this find_package finds a strawberry perl zlib which causes 32-bit build
# and 64-bit private loader issues).
if (WIN32 AND AUTOMATED_TESTING)
option(DISABLE_ZLIB "Disable looking for and using zlib" OFF)
if (WIN32 AND NOT DISABLE_ZLIB)
set(ZLIB_FOUND OFF)
else ()
find_package(ZLIB)
Expand Down
9 changes: 7 additions & 2 deletions make/package.cmake
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
# **********************************************************
# Copyright (c) 2011-2022 Google, Inc. All rights reserved.
# Copyright (c) 2011-2024 Google, Inc. All rights reserved.
# Copyright (c) 2009-2010 VMware, Inc. All rights reserved.
# **********************************************************

Expand Down Expand Up @@ -164,10 +164,15 @@ set(base_cache "
BUILD_NUMBER:STRING=${arg_build}
UNIQUE_BUILD_NUMBER:STRING=${arg_ubuild}
BUILD_PACKAGE:BOOL=ON
AUTOMATED_TESTING:BOOL=ON
${arg_cacheappend}
")

if (WIN32)
# TODO i#5767: Install a working zlib package on our Windows GA CI images.
set(base_cache "${base_cache}
DISABLE_ZLIB:BOOL=ON")
endif()

# version is optional
if (arg_version)
set(base_cache "${base_cache}
Expand Down

0 comments on commit 149ef73

Please sign in to comment.