Flight Log / microsoft/onnxruntime-genai #1516
Fix ort_genai_c.h compilation in C mode by using conditional includes — a GitHub Copilot (GitHub) bot flight
The GitHub Copilot bot (GitHub) opened this pull request in microsoft/onnxruntime-genai on May 29, 2025; merged clean in 5.8 d, merged by @ajindal1: +11 −5, 2 files, 1 review.
- Lines
- +11 −5
- Files
- 2
- Commits
- 3
- Reviews
- 1
- To merge
- 5.8 d
- Task
- fix
Line and file counts are summed per commit from the AIDev dataset; GitHub's net diff can be smaller.
Copilot was asked to fix a compile failure in microsoft/onnxruntime-genai, where ort_genai_c.h broke in C builds by including the C++-only cstddef header. The change adds conditional includes across two files, 11 additions and five deletions, over three commits. One review round drew no change requests or review comments. It merged clean after 140.09 hours.
From the pull request description
The `ort_genai_c.h` header was failing to compile when used as a C file due to the inclusion of `<cstddef>`, which is a C++-only header. ## Problem ```c // This would fail when compiling as C #include "ort_genai_c.h" ``` Error: ``` onnxruntime-genai/src/ort_genai_c.h:7:10: fatal error: 'cstddef' f…
Excerpt (first 300 characters) — the full description is on GitHub.
View the pull request on GitHub →
This is your flight? Claim it — sign in with GitHub. Ownership is proven by the sign-in: the account that opened the PR is the only one that can claim it.
Flight timeline
- —committedcopilot-swe-agent[bot]
- —committedcopilot-swe-agent[bot]
- —committedBaiju Meswani
- —reviewed
- Jun 4, 2025mergedajindal1
- Jun 4, 2025closedajindal1
Read next
Cite this flight: AgentRiders Flight Log, microsoft/onnxruntime-genai#1516, record updated Nov 4, 2026, https://agentriders.com/logs/microsoft/onnxruntime-genai/1516/
Data from public GitHub activity; this page is not affiliated with GitHub or the repository.