Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix cli in dockerfiles/README.md #23493

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Fix cli in dockerfiles/README.md #23493

wants to merge 2 commits into from

Conversation

genarks
Copy link

@genarks genarks commented Jan 26, 2025

Added context to command line example when specifying platform.

Description

Docker image build fails due to missing context in the command line example when specifying the platform.

The dockerfiles directory is assumed as the current directory in the command example, so the parent directory must be specified as the context of the docker build command

Added context to command line example when specifying platform.
Fix cli in dockerfiles/README.md
@snnn
Copy link
Member

snnn commented Jan 27, 2025

/azp run Big Models, Linux Android Emulator QNN CI Pipeline, Linux CPU CI Pipeline, Linux CPU Minimal Build E2E CI Pipeline, Linux GPU CI Pipeline, Linux GPU TensorRT CI Pipeline

@snnn
Copy link
Member

snnn commented Jan 27, 2025

/azp run Linux OpenVINO CI Pipeline, Linux QNN CI Pipeline, MacOS CI Pipeline, ONNX Runtime Web CI Pipeline, Win_TRT_Minimal_CUDA_Test_CI, Windows ARM64 QNN CI Pipeline, Windows CPU CI Pipeline

@snnn
Copy link
Member

snnn commented Jan 27, 2025

/azp run Windows GPU CUDA CI Pipeline, Windows GPU DML CI Pipeline, Windows GPU Doc Gen CI Pipeline, Windows GPU TensorRT CI Pipeline, Windows x64 QNN CI Pipeline, onnxruntime-binary-size-checks-ci-pipeline

Copy link

Azure Pipelines successfully started running 6 pipeline(s).

1 similar comment
Copy link

Azure Pipelines successfully started running 6 pipeline(s).

Copy link

Azure Pipelines successfully started running 7 pipeline(s).

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.

2 participants