Skip to content

simplify export include dir cmake code#4575

Merged
nibanks merged 1 commit intomicrosoft:mainfrom
talregev:TalR/inc_target_include_dir
Oct 14, 2024
Merged

simplify export include dir cmake code#4575
nibanks merged 1 commit intomicrosoft:mainfrom
talregev:TalR/inc_target_include_dir

Conversation

@talregev
Copy link
Copy Markdown
Contributor

simplify export include dir cmake code.

@talregev talregev requested a review from a team as a code owner September 28, 2024 04:47
@nibanks nibanks added Area: Build external Proposed by non-MSFT labels Sep 28, 2024
@codecov
Copy link
Copy Markdown

codecov bot commented Sep 28, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 86.68%. Comparing base (56ef97b) to head (0e0cec2).
Report is 24 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4575      +/-   ##
==========================================
+ Coverage   86.00%   86.68%   +0.68%     
==========================================
  Files          56       56              
  Lines       17354    17354              
==========================================
+ Hits        14925    15044     +119     
+ Misses       2429     2310     -119     

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

@talregev talregev mentioned this pull request Sep 28, 2024
@talregev
Copy link
Copy Markdown
Contributor Author

talregev commented Oct 3, 2024

@nibanks Can you review my PR?
Thank you.

@nibanks nibanks merged commit 6e23861 into microsoft:main Oct 14, 2024
@talregev talregev deleted the TalR/inc_target_include_dir branch October 14, 2024 16:20
@talregev talregev mentioned this pull request Dec 4, 2024
talregev added a commit to talregev/msquic that referenced this pull request Dec 4, 2024
talregev added a commit to talregev/msquic that referenced this pull request Dec 4, 2024
nibanks pushed a commit that referenced this pull request Dec 5, 2024
* Change public headers to support msh3 (#4525)

* simplify export include dir cmake code (#4575)

* Delegate build environment configuration with QUIC_EXTERNAL_TOOLCHAIN (#4625)

Well managed CMake build environments make use of CMAKE_TOOLCHAIN_FILE
to configure build environment such as include/lib search paths,
system libraries, build flags, etc.

This change introduces QUIC_EXTERNAL_TOOLCHAIN build option, which
when enabled stops MSQuic CMake scripts from attempting to
do CMake toolchain job.

QUIC_EXTERNAL_TOOLCHAIN defaults to OFF to preserve original behaviour.

* export platform (#4574)

* Add wbemuuid (#4687)

---------

Co-authored-by: Maxim Ivanov <hi+github@yamlcoder.me>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: Build external Proposed by non-MSFT

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants