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

[AIX] Build fix for using system installed protobuf/onnx #22272

Merged

Conversation

ranjitshs
Copy link
Contributor

Description

To fix the build issues for AIX OS while using system installed protobuf/onnx.

Motivation and Context

Code changes in this PR contains:

  1. Fix for below compilation issue.
collect2: fatal error: library liblibprotobuf-lite not found
compilation terminated.
  1. Adding onnx library into dependency list for test applicaitons.

@ranjitshs
Copy link
Contributor Author

Hi @tianleiwu @snnn
Couple of fixes under AIX. Could you please review and trigger build pipeline

@snnn
Copy link
Member

snnn commented Sep 30, 2024

/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 Sep 30, 2024

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

@snnn
Copy link
Member

snnn commented Sep 30, 2024

/azp run Windows CPU CI Pipeline, 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, orttraining-linux-ci-pipeline, orttraining-linux-gpu-ci-pipeline

Copy link

Azure Pipelines successfully started running 6 pipeline(s).

Copy link

Azure Pipelines successfully started running 5 pipeline(s).

Copy link

Azure Pipelines successfully started running 9 pipeline(s).

@snnn snnn merged commit 8120757 into microsoft:main Sep 30, 2024
70 checks passed
@ranjitshs
Copy link
Contributor Author

Thanks @snnn for merging this.

@ranjitshs ranjitshs deleted the build-fix-using-installed-protobuf/onnx branch September 30, 2024 20:52
@ranjitshs
Copy link
Contributor Author

Hi @snnn
#22046 is not properly taken the changes from this PR.
So, this PR changes is somewhat reverted.
I will raise another PR to fix AIX

tianleiwu pushed a commit that referenced this pull request Oct 4, 2024
### Description
Fixing merge issue occurred in
#22272

### Motivation and Context
To build onnxruntime using system installed protobuf/onnx.
ishwar-raut1 pushed a commit to ishwar-raut1/onnxruntime that referenced this pull request Nov 19, 2024
…2272)

### Description
To fix the build issues for AIX OS while using system installed
protobuf/onnx.

### Motivation and Context
Code changes in this PR contains:

1. Fix for below compilation issue.
```
collect2: fatal error: library liblibprotobuf-lite not found
compilation terminated.
```
2.  Adding onnx library into dependency list for test applicaitons.
ishwar-raut1 pushed a commit to ishwar-raut1/onnxruntime that referenced this pull request Nov 19, 2024
### Description
Fixing merge issue occurred in
microsoft#22272

### Motivation and Context
To build onnxruntime using system installed protobuf/onnx.
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