Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[POC|Invalid] Testing vcpkg without installed MSVC #42129

Draft
wants to merge 9 commits into
base: master
Choose a base branch
from
Draft

[POC|Invalid] Testing vcpkg without installed MSVC #42129

wants to merge 9 commits into from

Conversation

Neumann-A
Copy link
Contributor

@Neumann-A Neumann-A commented Nov 12, 2024 •
edited
Loading

Hijacking x64-windows CI for this.

wished somebody would make Visual Studio truly portable. MSBuild looking up all those reg keys is a bit annoying.

No NETFXSDK installed.

Also what are all those different versions about and why don't any of those correlate to the folder version names.

Build system check:

  • cmake
  • autotools
  • meson
  • bazel
  • qmake
  • msbuild (needs ENV{WindowsSDKVersion} fixed) fixed it here

@FrankXie05 FrankXie05 added the requires:testing Needs tests added before merging label Nov 13, 2024
Neumann-A
Neumann-A commented Nov 13, 2024
View reviewed changes
ports/msvc/portfile.cmake Outdated Show resolved Hide resolved
Neumann-A
Neumann-A commented Nov 13, 2024
View reviewed changes
ports/msvc/msvc-env.cmake Outdated Show resolved Hide resolved
@Neumann-A Neumann-A force-pushed the windows-triplet-without-installed-msvc branch from 7538913 to 13f4af9 Compare November 13, 2024 22:55
Neumann-A
Neumann-A commented Nov 14, 2024
View reviewed changes
ports/msvc/vcpkg.json Outdated Show resolved Hide resolved
Neumann-A
Neumann-A commented Nov 14, 2024
View reviewed changes
triplets/x64-windows.cmake Outdated Show resolved Hide resolved
@Neumann-A
Copy link
Contributor Author

Already pretty good:

SUMMARY FOR x64-windows
  SUCCEEDED: 2120
  BUILD_FAILED: 36
  POST_BUILD_CHECKS_FAILED: 10
  CASCADED_DUE_TO_MISSING_DEPENDENCIES: 225

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@FrankXie05 FrankXie05

Labels
requires:testing Needs tests added before merging
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Neumann-A @FrankXie05