Skip to content

Use preinstalled vcpkg on Azure#2928

Merged
taketwo merged 1 commit intoPointCloudLibrary:masterfrom
taketwo:use-preinstalled-vcpkg
Mar 16, 2019
Merged

Use preinstalled vcpkg on Azure#2928
taketwo merged 1 commit intoPointCloudLibrary:masterfrom
taketwo:use-preinstalled-vcpkg

Conversation

@taketwo
Copy link
Copy Markdown
Member

@taketwo taketwo commented Mar 16, 2019

Our Windows CI jobs have been failing for a day or so on the "Free Up Space" step. I did not investigate what exactly was the problem, but this commit fixes it.

Closes #2832.

While working on this I dropped installation of Boost through vcpkg to accelerate debugging of the job. To my surprise, at some point the overall job has passed. Apparently, there is Boost 1.69 already installed in the images! Thus I propose to switch to using this preinstalled Boost and not install it through vcpkg. This will save us ~20 minutes of CI time. Does anyone of our vcpkg gurus (@UnaNancyOwen, @jasjuang, @claudiofantacci) have an opinion on this? Can there be any downsides?

@taketwo taketwo merged commit 9bb32e5 into PointCloudLibrary:master Mar 16, 2019
@taketwo taketwo deleted the use-preinstalled-vcpkg branch March 16, 2019 14:06
@UnaNancyOwen
Copy link
Copy Markdown
Member

Apparently, there is Boost 1.69 already installed in the images! Thus I propose to switch to using this preinstalled Boost and not install it through vcpkg. This will save us ~20 minutes of CI time.

That would be nice! 👍

@claudiofantacci
Copy link
Copy Markdown
Contributor

Apparently, there is Boost 1.69 already installed in the images!

I was unaware of this as well. Good to know!

The pre-installed boost version is the same as the vcpkg's one and hence it should be just fine 👍

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Use preinstalled vcpkg in Azure Pipelines

4 participants