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.

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

The latest vcpkg is broken, please don't take it #1332

Closed
2 tasks
snnn opened this issue Jul 31, 2020 · 7 comments
Closed
2 tasks

The latest vcpkg is broken, please don't take it #1332

snnn opened this issue Jul 31, 2020 · 7 comments
Assignees
Labels
Area: C/C++ investigate Collect additional information, like space on disk, other tool incompatibilities etc. OS: Windows

Comments

@snnn
Copy link

snnn commented Jul 31, 2020

Describe the bug
see: microsoft/vcpkg#12565

I know your team usually generates a new set of images every week. But, the build script always take vcpkg from master, it is kind of dangerous. Please consider to use tags. The issue 12565 is an example, if you rebuild the image now, you'll have the bug, and would break all the vcpkg Windows users.

Area for Triage:
C/C++

Question, Bug, or Feature?:
Bug

Virtual environments affected

  • Windows Server 2016 R2
  • Windows Server 2019

Expected behavior
Rebuild the image
Try to use vcpkg to install protobuf/libffi or some other open source libraries
The should work fine

Actual behavior
It fails, even through it's not your fault. Here just kindly remind you there is a problem. Please feel free to close this issue without taking extra actions.

@maxim-lobanov
Copy link
Contributor

Hello @snnn , Thank you for reaching us.
Am I right that your suggestion to use latest release instead of master branch during installation?
https://github.com/microsoft/vcpkg/releases/latest

@snnn
Copy link
Author

snnn commented Jul 31, 2020

Yes!

@dibir-magomedsaygitov dibir-magomedsaygitov added OS: Windows investigate Collect additional information, like space on disk, other tool incompatibilities etc. Area: C/C++ and removed needs triage labels Jul 31, 2020
@larshg
Copy link

larshg commented Jul 31, 2020

It seems that CMake 18 is the culprint and it already hit the pipelines like here:

https://dev.azure.com/PointCloudLibrary/pcl/_build/results?buildId=17096&view=logs&j=d944f354-b9eb-5c76-5ade-3c70deb27a9e&t=fb7e0546-b0fe-5e7c-2a44-5710f4ba7243

So please make a hotfix when VCPKG has solved its bug.

@kcgen
Copy link

kcgen commented Jul 31, 2020

This has tanked our Windows CI workflows since Monday.

@maxim-lobanov
Copy link
Contributor

Just to merge discussions in #1340 and this issue:
Looks like rolling back vcpkg to the latest release will cause additional impact because of cmake incompatibility.
Installing master branch looks like the better solution for now.

cc: @strega-nil

@kcgen
Copy link

kcgen commented Aug 3, 2020

@maxim-lobanov , drop an update when the new Windows GitHub images are regenerated w/ the upated vcpkg. 👍

Going on day 8 now with Windows CI broken, so can't wait for the fix 🙌

2020-08-03_08-32

@maxim-lobanov
Copy link
Contributor

Hello,
Based on conversation below and in #1340 we have decided to keep the current approach for now, install vcpkg from master branch during image generation. Per discussion, installing vcpkg from latest tag instead of master will bring more problems because of incompatibility with cmake.

I am closing this issue but if you have any other suggestions how we can improve installation process from our side, we are ready to continue discussion

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: C/C++ investigate Collect additional information, like space on disk, other tool incompatibilities etc. OS: Windows
Projects
None yet
Development

No branches or pull requests

6 participants