Skip to content

Install uuid-dev in the Linux minimal build_full_ort job - #32491

Merged
Tianlei Wu (tianleiwu) merged 2 commits into
mainfrom
copilot/fix-linux-minimal-build-dependency
Sep 9, 2026
Merged

Tianlei Wu (tianleiwu) merged 2 commits into
mainfrom
copilot/fix-linux-minimal-build-dependency

Conversation

Copilot AI commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Description

build_full_ort in .github/workflows/linux_minimal_build.yml now installs uuid-dev before building:

      # This job builds with --use_coreml. The coremltools modelpackage sources include <uuid/uuid.h>,
      # which is provided by uuid-dev on Ubuntu.
      - name: Install libuuid development files
        run: sudo apt-get update -y && sudo apt-get install -y uuid-dev
  • Scoped to build_full_ort only — it is the sole job in this workflow that builds with --use_coreml directly on the runner; the rest build inside the CI docker image or don't enable CoreML.
  • Placed before Build Full ORT and Prepare Test Files, so the header/library are present both when CMake configures and when ModelPackage.cpp compiles.
  • No CMake change: cmake/onnxruntime_providers_coreml.cmake already does find_path/find_library for libuuid on Linux (with an actionable FATAL_ERROR) and links it into onnxruntime_providers_coreml.

Motivation and Context

Linux CPU Minimal Build E2E / build_full_ort fails in run 34290674202, job 102276280773:

_deps/coremltools-src/modelpackage/src/ModelPackage.cpp:35:10: fatal error: uuid/uuid.h: No such file or directory

The job enables the CoreML EP, and the fetched coremltools modelpackage sources include <uuid/uuid.h>, supplied on Ubuntu by uuid-dev. Neither the workflow nor the reusable setup-build-tools action (which only provisions cmake/ccache/vcpkg) installs apt packages, so the header is only available when vcpkg's coreml-ep feature happens to provide it; without it Ninja stops while compiling onnxruntime_providers_coreml.

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

…package sources

Co-authored-by: tianleiwu <30328909+tianleiwu@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix missing Linux dependency for CoreML provider build Install uuid-dev in the Linux minimal build_full_ort job Sep 9, 2026
@tianleiwu
Tianlei Wu (tianleiwu) marked this pull request as ready for review September 9, 2026 03:56
Copilot AI balanced review requested due to automatic review settings September 9, 2026 03:56
@tianleiwu
Tianlei Wu (tianleiwu) enabled auto-merge (squash) September 9, 2026 03:56

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

The scoped workflow dependency fix has no unresolved issues.

Pull request overview

Installs the Ubuntu libuuid development dependency required by the CoreML-enabled Linux build.

Changes:

  • Installs uuid-dev before the full ORT build.
  • Documents why CoreML requires the dependency.
File summaries
File Description
.github/workflows/linux_minimal_build.yml Installs libuuid headers for the CoreML-enabled job.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Balanced

💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@tianleiwu
Tianlei Wu (tianleiwu) merged commit f26e546 into main Sep 9, 2026
90 of 91 checks passed
@tianleiwu
Tianlei Wu (tianleiwu) deleted the copilot/fix-linux-minimal-build-dependency branch September 9, 2026 06:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants