Skip to content

[Ubuntu] Remove workaround for vcpkg#728

Merged
AlenaSviridenko merged 5 commits intoactions:masterfrom
miketimofeev:v-mitim/vcpkg_workaround
Apr 17, 2020
Merged

[Ubuntu] Remove workaround for vcpkg#728
AlenaSviridenko merged 5 commits intoactions:masterfrom
miketimofeev:v-mitim/vcpkg_workaround

Conversation

@miketimofeev
Copy link
Copy Markdown
Contributor

Don't need this workaround anymore – issue fixed in microsoft/vcpkg#10846

@maxim-lobanov
Copy link
Copy Markdown
Contributor

/azp run ubuntu1604, ubuntu1804

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 2 pipeline(s).

@miketimofeev
Copy link
Copy Markdown
Contributor Author

There is a new error with g++, need to invesitgate

==> azure-arm: CMake Error at cmake/utilities.cmake:6 (message):
==> azure-arm:   The g++ version picked up is too old; please install a newer compiler such
==> azure-arm:   as g++-7.
    azure-arm: -- Configuring incomplete, errors occurred!
    azure-arm: See also "/usr/local/share/vcpkg/toolsrc/build.rel/CMakeFiles/CMakeOutput.log".
==> azure-arm:
==> azure-arm:                       On Ubuntu try the following:
==> azure-arm:                           sudo add-apt-repository ppa:ubuntu-toolchain-r/test -y
==> azure-arm:                           sudo apt-get update -y
==> azure-arm:                           sudo apt-get install g++-7 -y
==> azure-arm:                       On CentOS try the following:
==> azure-arm:                           sudo yum install centos-release-scl
==> azure-arm:                           sudo yum install devtoolset-7
==> azure-arm:                           scl enable devtoolset-7 bash
==> azure-arm: Call Stack (most recent call first):
==> azure-arm:   CMakeLists.txt:29 (vcpkg_detect_compiler)
==> azure-arm:
==> azure-arm:

@AlenaSviridenko AlenaSviridenko merged commit b1cf25f into actions:master Apr 17, 2020
@miketimofeev miketimofeev deleted the v-mitim/vcpkg_workaround branch June 1, 2021 15:45
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.

5 participants