-
Notifications
You must be signed in to change notification settings - Fork 6.6k
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
[cmake] upgrade to 3.18 #12612
[cmake] upgrade to 3.18 #12612
Conversation
depends #12569 |
It looks like |
It's not cmake 3.18, it's a bug in our script (a slipped required keyword which should not be there) |
In the meantime 3.18.1 was released |
Updated to 3.18.2 |
@cenit Don't worry, I can continue this PR. |
Wow, I can't believe our cmake update caused those too many regressions. We need to handle them for a long time. |
Regression list:
Disappeared.
Aleady solved in master branch.
Fixed in master branch.
Disappeared.
Disappeared.
Disappeared.
Disappeared.
Disappeared.
need disable pararrel configure #13162
Disappeared. |
The osx regression are mainly due to autopoint (gettext) not being available. |
@JackBoosY: Seems like most of the errors are baseline regression. libtool missing on linux, autopoint missing on osx I saw one vcpkg-cmake-wrapper.cmake regression which seems to use an incomplete list of library names. Did the CI machines get reprovisioned? |
on windows the wrong compiler seems to be used. (clang instead of MSVC) |
Okay, all regressions listed, we need to open a few PRs to fix them. |
@Neumann-A What do you think about ignition-msgs and qt5-tools regressions? |
missing link to uuid.lib ? |
just a guess: maybe clang gets on path due to vcpkg_add_to_path(${CURRENT_INSTALLED_DIR}/bin) (from ignition-msg5 portfile) and due to a bug in llvm portfile clang is left in the bin folder and not moved to tools? |
But it should not be the first compiler on the path.
seems like it: I also had an issue with it here #12936 (comment) . |
@Neumann-A Tools vcpkg/scripts/cmake/vcpkg_configure_make.cmake Lines 427 to 434 in 4a9b658
Maybe related to cmake update changes. Yes, I can confirm that's related. |
@JackBoosY: Add search paths |
Ah that is the same issue as the pkg-config not found issue before. The
|
One issue left. Almost done. |
# Conflicts: # ports/sentry-native/portfile.cmake
These regressions will be fixed in another PR. |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
The qca regressions has appeared again and will be fixed in another PR. This fix has not been merged into 3.18.4, we still need to submit a PR to fix this issue. |
The "freetds" failures are due to their upstream server being down and nothing depends on that port, so I'm going to merge this despite that "regression" (assuming osx comes back OK; it's not done at time of writing) |
[cmake] upgrade to 3.18 (microsoft#12612)
Describe the pull request
In order to get ready for CUDA 11 and OpenCV 4.4, this is a first important step.