Skip to content

update cmake mac - #7443

Merged
michaelgsharp merged 8 commits into
mainfrom
mac-cmake-fix
Apr 23, 2025
Merged

update cmake mac#7443
michaelgsharp merged 8 commits into
mainfrom
mac-cmake-fix

Conversation

@michaelgsharp

Copy link
Copy Markdown
Contributor

No description provided.

@codecov

codecov Bot commented Apr 11, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 69.00%. Comparing base (500385b) to head (c3e89ea).
Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7443      +/-   ##
==========================================
- Coverage   69.00%   69.00%   -0.01%     
==========================================
  Files        1482     1482              
  Lines      273876   273876              
  Branches    28254    28254              
==========================================
- Hits       188979   188975       -4     
- Misses      77514    77516       +2     
- Partials     7383     7385       +2     
Flag Coverage Δ
Debug 69.00% <ø> (-0.01%) ⬇️
production 63.28% <ø> (-0.01%) ⬇️
test 89.46% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

see 3 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
Comment thread build/ci/job-template.yml Outdated

@ericstj ericstj left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

It seems like this resolves the error.

@ericstj
ericstj marked this pull request as ready for review April 21, 2025 19:29
@ericstj

ericstj commented Apr 23, 2025

Copy link
Copy Markdown
Member

Change fixed the install step, but build was failing in the same way. I updated cmake_minimum_required(VERSION 3.20) since that's what I see we use in runtime for our native libs in net8.0: https://github.com/dotnet/runtime/blob/4f67954b64888688b52ad62c4bce8836f8e6e103/src/native/libs/CMakeLists.txt#L1

@michaelgsharp

Copy link
Copy Markdown
Contributor Author

/ba-g test failure is a known issue.

@michaelgsharp
michaelgsharp merged commit d070dcb into main Apr 23, 2025
ericstj added a commit that referenced this pull request May 21, 2025
* update cmake mac

* cmake specific version

* remove specific version

* cmake policy version minimum

* Update build/ci/job-template.yml

* Try adding -DCMAKE_POLICY_VERSION_MINIMUM=3.5 to libomp.rb

* Update job-template.yml

* Update cmake_minimum_required

---------

Co-authored-by: Eric StJohn <ericstj@microsoft.com>
@github-actions github-actions Bot locked and limited conversation to collaborators May 24, 2025
@ericstj
ericstj deleted the mac-cmake-fix branch June 12, 2025 20:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

2 participants