Skip to content

[build] Fix the file copy in get_docker_image.py - #25548

Merged
jywu-mysoft merged 2 commits into
mainfrom
fs-eire/fix-path-get-docker-image
Jul 26, 2025
Merged

[build] Fix the file copy in get_docker_image.py#25548
jywu-mysoft merged 2 commits into
mainfrom
fs-eire/fix-path-get-docker-image

Conversation

@fs-eire

@fs-eire fs-eire commented Jul 26, 2025

Copy link
Copy Markdown
Contributor

Description

Fixes the packaging pipeline.

@fs-eire
fs-eire requested review from Copilot and snnn July 26, 2025 19:27

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.

Pull Request Overview

This PR fixes a file copy issue in the Docker image build pipeline by ensuring the target directory exists before copying files. The fix prevents potential failures when the scripts directory doesn't exist in the Docker context.

  • Extracts the destination scripts directory path into a variable for better readability
  • Adds a directory creation step to ensure the target directory exists before file copying

Comment thread tools/ci_build/get_docker_image.py Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@jywu-mysoft
jywu-mysoft merged commit 7c0c29d into main Jul 26, 2025
92 checks passed
@jywu-mysoft
jywu-mysoft deleted the fs-eire/fix-path-get-docker-image branch July 26, 2025 23:39
snnn pushed a commit that referenced this pull request Jul 27, 2025
@snnn snnn mentioned this pull request Jul 28, 2025
sanketkaleoss pushed a commit to sanketkaleoss/onnxruntime that referenced this pull request Aug 11, 2025
### Description

Fixes the packaging pipeline.

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@snnn snnn mentioned this pull request Sep 16, 2025
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