Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@liqunfu
Copy link
Collaborator

@liqunfu liqunfu commented Feb 22, 2024

Description

https://gitlab.kitware.com/cmake/cmake/-/issues/25704

Motivation and Context

Signed-off-by: liqunfu <[email protected]>
@liqunfu liqunfu requested a review from a team as a code owner February 22, 2024 01:26
@codecov
Copy link

codecov bot commented Feb 22, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 56.68%. Comparing base (66b7fb6) to head (aafa40d).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #5951   +/-   ##
=======================================
  Coverage   56.68%   56.68%           
=======================================
  Files         506      506           
  Lines       30227    30227           
  Branches     4565     4565           
=======================================
  Hits        17133    17133           
  Misses      12268    12268           
  Partials      826      826           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Signed-off-by: liqunfu <[email protected]>
@liqunfu liqunfu added this to the 1.16 milestone Feb 22, 2024
@liqunfu liqunfu added the run release CIs Use this label to trigger release tests in CI label Feb 22, 2024
Signed-off-by: liqunfu <[email protected]>
@liqunfu liqunfu changed the title use cmake 3.27.9 The latest Cmake 3.28.3 is failing with "Could NOT find Protobuf (missing: Protobuf_LIBRARIES). Use Cmake 3.27.9 Feb 22, 2024
@liqunfu liqunfu changed the title The latest Cmake 3.28.3 is failing with "Could NOT find Protobuf (missing: Protobuf_LIBRARIES). Use Cmake 3.27.9 The latest Cmake 3.28.3 is failing with "Could NOT find Protobuf (missing: Protobuf_LIBRARIES)". Use Cmake 3.27.9 Feb 22, 2024
Copy link

@thiagocrepaldi thiagocrepaldi left a comment

Choose a reason for hiding this comment

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

Is choco available on the CI?

@liqunfu
Copy link
Collaborator Author

liqunfu commented Feb 22, 2024

Is choco available on the CI?

yes it is - cmake of desired version is installed with the fix.

@liqunfu liqunfu enabled auto-merge February 22, 2024 18:37
@liqunfu liqunfu closed this Feb 22, 2024
auto-merge was automatically disabled February 22, 2024 18:38

Pull request was closed

@liqunfu liqunfu reopened this Feb 22, 2024
@liqunfu liqunfu removed the run release CIs Use this label to trigger release tests in CI label Feb 22, 2024
@liqunfu liqunfu enabled auto-merge February 22, 2024 22:51
@liqunfu liqunfu closed this Feb 22, 2024
auto-merge was automatically disabled February 22, 2024 22:51

Pull request was closed

@liqunfu liqunfu reopened this Feb 22, 2024
Signed-off-by: liqunfu <[email protected]>
Signed-off-by: liqunfu <[email protected]>
@liqunfu liqunfu added this pull request to the merge queue Feb 22, 2024
Merged via the queue into main with commit 9140ca7 Feb 23, 2024
@liqunfu liqunfu deleted the liqun/ci-with-external-protobuf-failure branch February 23, 2024 00:40
linshokaku pushed a commit to linshokaku/onnx that referenced this pull request Oct 2, 2024
…sing: Protobuf_LIBRARIES)". Use Cmake 3.27.9 (onnx#5951)

### Description
https://gitlab.kitware.com/cmake/cmake/-/issues/25704

### Motivation and Context
<!-- - Why is this change required? What problem does it solve? -->
<!-- - If it fixes an open issue, please link to the issue here. -->

---------

Signed-off-by: liqunfu <[email protected]>
Signed-off-by: Linsho Kaku <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants