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

Build fix with MinGW GCC 9.2.0 #12084

Merged
merged 4 commits into from
Jul 18, 2020
Merged

Conversation

cristianadam
Copy link
Contributor

@cristianadam cristianadam commented Jun 24, 2020

Describe the pull request

This PR will allow building vcpkg itself with MinGW GCC 9.2.0. It fixes some warnings and because warnings are treated as errors, the build would fail.

It also fixes the usage of MinGW community tripplets.

Fixes: #12065

@NancyLi1013
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@JackBoosY
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@JackBoosY JackBoosY added category:vcpkg-bug The issue is with the vcpkg system (including helper scripts in `scripts/cmake/`) info:reviewed Pull Request changes follow basic guidelines labels Jun 28, 2020
Copy link
Contributor

@strega-nil strega-nil left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you merge to latest master? This seems like something we definitely want.

toolsrc/src/vcpkg/build.cpp Outdated Show resolved Hide resolved
@cristianadam
Copy link
Contributor Author

Can you merge to latest master? This seems like something we definitely want.

I've rebased on master.

cristianadam and others added 3 commits July 16, 2020 20:45
read_symlink_implementation was used only in copy_symlink_implementation
when #if defined(_WIN32) && !VCPKG_USE_STD_FILESYSTEM  was true.

Removed the warning otherwise.
@strega-nil strega-nil merged commit f4bd642 into microsoft:master Jul 18, 2020
Jimmy-Hu added a commit to Jimmy-Hu/vcpkg that referenced this pull request Jul 19, 2020
remz1337 pushed a commit to remz1337/vcpkg that referenced this pull request Aug 11, 2020
* Build fix with MinGW GCC 9.2.0

* Fix MinGW tripplets

Fixes microsoft#12065

* clang-format

* Fix unused read_symlink_implementation warning marked as error

read_symlink_implementation was used only in copy_symlink_implementation
when #if defined(_WIN32) && !VCPKG_USE_STD_FILESYSTEM  was true.

Removed the warning otherwise.

Co-authored-by: JackBoosY <yuzaiyang@beyondsoft.com>
strega-nil pushed a commit to strega-nil/vcpkg that referenced this pull request May 5, 2021
* Build fix with MinGW GCC 9.2.0

* Fix MinGW tripplets

Fixes microsoft#12065

* clang-format

* Fix unused read_symlink_implementation warning marked as error

read_symlink_implementation was used only in copy_symlink_implementation
when #if defined(_WIN32) && !VCPKG_USE_STD_FILESYSTEM  was true.

Removed the warning otherwise.

Co-authored-by: JackBoosY <yuzaiyang@beyondsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:vcpkg-bug The issue is with the vcpkg system (including helper scripts in `scripts/cmake/`) info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Community triplets not working because of toolchain detection mechanism
5 participants