Skip to content

[binaryparagraph] Fix VS 2015 compile error.#3

Closed
Hoikas wants to merge 1 commit intomicrosoft:mainfrom
Hoikas:fix-vs2015
Closed

[binaryparagraph] Fix VS 2015 compile error.#3
Hoikas wants to merge 1 commit intomicrosoft:mainfrom
Hoikas:fix-vs2015

Conversation

@Hoikas
Copy link
Contributor

@Hoikas Hoikas commented Feb 6, 2021

This fixes compilation on Visual Studio 2015. constexpr lambdas were added in Visual Studio 2017 update 15.3. Further, std::all_of did not become constexpr until C++20.

@strega-nil
Copy link
Contributor

Are we still guaranteeing that we compile with VS2015? Wasn't part of the point of pulling out the tool that we didn't have to compile with VS2015 anymore?

@Hoikas
Copy link
Contributor Author

Hoikas commented Feb 8, 2021

Good question. I looked but didn't see any explicit guidance on that in this repository. This change only exists so I could test #4.

@strega-nil
Copy link
Contributor

@Hoikas we've agreed that we are explicitly not supporting VS2015 from this point onwards. Thanks for the PR, and @ras0219 will look at #4, but closing as "no moar VS2015 muahahaha"

@strega-nil strega-nil closed this Feb 8, 2021
@Hoikas
Copy link
Contributor Author

Hoikas commented Feb 8, 2021

I share your sentiment -- 💥 💥 💥 VS 2015 😄

vicroms added a commit to vicroms/vcpkg-tool that referenced this pull request Aug 28, 2025
# This is the 1st commit message:

WIP

# The commit message #2 will be skipped:

# Fix build error

# The commit message microsoft#3 will be skipped:

# Implement bulk operation

# The commit message microsoft#4 will be skipped:

# Remove unused struct

# The commit message microsoft#5 will be skipped:

# Follow redirects

# The commit message microsoft#6 will be skipped:

# Get HTTP response code

# The commit message microsoft#7 will be skipped:

# Map URL response code to the correct index

# The commit message microsoft#8 will be skipped:

# Map URL response code to the correct index

# The commit message microsoft#9 will be skipped:

# Undo changes to workflows

# The commit message microsoft#10 will be skipped:

# Implement store to asset cache

# The commit message microsoft#11 will be skipped:

# handle file:// protocol

# The commit message microsoft#12 will be skipped:

# Implement try_download_file

# The commit message microsoft#13 will be skipped:

# Add missing calls to curl_easy_cleanup

# The commit message microsoft#14 will be skipped:

# Fix global init

# The commit message microsoft#15 will be skipped:

# Replace all calls in downloads.cpp and fix unit tests

# The commit message microsoft#16 will be skipped:

# use libcurl to submit metrics payload

# The commit message microsoft#17 will be skipped:

# Use Schannel on Windows

# The commit message microsoft#18 will be skipped:

# Apply suggestions from code review
#
# Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

# The commit message microsoft#19 will be skipped:

# Fix unit test

# The commit message microsoft#20 will be skipped:

# Change curl error messages

# The commit message microsoft#21 will be skipped:

# generate message map

# The commit message microsoft#22 will be skipped:

# Always call multi_remove_handle

# The commit message microsoft#23 will be skipped:

# Fix end-to-end tests

# The commit message microsoft#24 will be skipped:

# Cleanup header lists

# The commit message microsoft#25 will be skipped:

# curl global init order

# The commit message microsoft#26 will be skipped:

# Use external libcurl on non-Windows

# The commit message microsoft#27 will be skipped:

# Install libcurl4 dev package on Linux

# The commit message microsoft#28 will be skipped:

# Disable metrics by default in debug builds

# The commit message microsoft#29 will be skipped:

# Set user agent using CURLOPT_USERAGENT

# The commit message microsoft#30 will be skipped:

# curl initialization again
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants