Skip to content

Commit

Permalink
[chromium-base] Don't use ambiguous variables (microsoft#39343)
Browse files Browse the repository at this point in the history
For warnings in microsoft#39337.
  • Loading branch information
dg0yt authored Jun 17, 2024
1 parent d36776f commit 40616a5
Show file tree
Hide file tree
Showing 4 changed files with 13 additions and 8 deletions.
12 changes: 6 additions & 6 deletions ports/chromium-base/portfile.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -68,15 +68,15 @@ set(OPTIONS "\
treat_warnings_as_errors=false")
set(DEFINITIONS "")

if(WIN32)
if(VCPKG_TARGET_IS_WINDOWS)
# Windows 10 SDK >= (10.0.19041.0) is required
# https://developer.microsoft.com/en-us/windows/downloads/windows-10-sdk
SET(VCPKG_POLICY_SKIP_ARCHITECTURE_CHECK enabled)
set(ENV{DEPOT_TOOLS_WIN_TOOLCHAIN} 0)
set(OPTIONS "${OPTIONS} use_lld=false")
endif()

if(VCPKG_CMAKE_SYSTEM_NAME STREQUAL Linux)
if(VCPKG_TARGET_IS_LINUX)
set(OPTIONS "${OPTIONS} use_allocator=\"none\" use_sysroot=false use_glib=false")
endif()

Expand All @@ -88,8 +88,8 @@ if(CLANG MATCHES "-NOTFOUND")
endif()
get_filename_component(CLANG "${CLANG}" DIRECTORY)
get_filename_component(CLANG "${CLANG}" DIRECTORY)
if((WIN32 AND NOT EXISTS "${CLANG}/bin/clang-cl.exe") OR
(APPLE AND NOT EXISTS "${CLANG}/bin/clang"))
if((VCPKG_TARGET_IS_WINDOWS AND NOT EXISTS "${CLANG}/bin/clang-cl.exe") OR
(VCPKG_TARGET_IS_OSX AND NOT EXISTS "${CLANG}/bin/clang"))
message(FATAL_ERROR "Clang needs to be inside a bin directory.")
endif()
set(OPTIONS "${OPTIONS} clang_base_path=\"${CLANG}\"")
Expand All @@ -101,7 +101,7 @@ else()
set(OPTIONS "${OPTIONS} is_component_build=false")
endif()

if(APPLE)
if(VCPKG_TARGET_IS_OSX)
set(OPTIONS "${OPTIONS} enable_dsyms=true")
endif()

Expand All @@ -126,7 +126,7 @@ set(TARGETS
base/third_party/dynamic_annotations
base/third_party/double_conversion)

if(WIN32)
if(VCPKG_TARGET_IS_WINDOWS)
list(APPEND TARGETS base/win:pe_image)
endif()

Expand Down
2 changes: 1 addition & 1 deletion ports/chromium-base/vcpkg.json
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
{
"name": "chromium-base",
"version": "86.0.4199.1",
"port-version": 5,
"port-version": 6,
"description": "Chromium is an open-source browser project that aims to build a safer, faster, and more stable way for all users to experience the web.",
"homepage": "https://chromium.googlesource.com/chromium/src",
"license": "BSD-3-Clause",
Expand Down
2 changes: 1 addition & 1 deletion versions/baseline.json
Original file line number Diff line number Diff line change
Expand Up @@ -1614,7 +1614,7 @@
},
"chromium-base": {
"baseline": "86.0.4199.1",
"port-version": 5
"port-version": 6
},
"chronoengine": {
"baseline": "8.0.0",
Expand Down
5 changes: 5 additions & 0 deletions versions/c-/chromium-base.json
Original file line number Diff line number Diff line change
@@ -1,5 +1,10 @@
{
"versions": [
{
"git-tree": "7f3a0b54287d0d86a331bc8fdc004086b992ab05",
"version": "86.0.4199.1",
"port-version": 6
},
{
"git-tree": "08b399ae573db73407ae4095c79628553416a992",
"version": "86.0.4199.1",
Expand Down

0 comments on commit 40616a5

Please sign in to comment.