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

Updates and fixes for the pyside 3P package on Linux #251

Merged
merged 15 commits into from
Apr 12, 2024

Conversation

spham-amzn
Copy link
Contributor

  • Update / Fix the git URL
  • Update the dependent packages for linux
  • Update the patchelf to use --force-rpath to set the rpath properly
  • Add a symlink to the python package in the docker build script to correct the build

* Update the dependent packages for linux
* Add a symlink to the python package in the docker build script to correct the build

Signed-off-by: Steve Pham <[email protected]>
* Remove temp symlink of python before packaging to fix packaging error outside of the container

Signed-off-by: Steve Pham <[email protected]>
Signed-off-by: Steve Pham <[email protected]>
@spham-amzn spham-amzn closed this Apr 4, 2024
@spham-amzn spham-amzn reopened this Apr 4, 2024
@spham-amzn spham-amzn closed this Apr 4, 2024
@spham-amzn spham-amzn reopened this Apr 4, 2024
@spham-amzn spham-amzn requested a review from a team April 8, 2024 17:24
@amzn-changml
Copy link
Contributor

Let me know if this is ready to promote to dev for AR use

@spham-amzn
Copy link
Contributor Author

Let me know if this is ready to promote to dev for AR use

What is the package URL to use to test these artifacts?

@amzn-changml
Copy link
Contributor

Let me know if this is ready to promote to dev for AR use

What is the package URL to use to test these artifacts?

https://d1gg6ket0m44ly.cloudfront.net would be the dev 3p URL

@spham-amzn spham-amzn merged commit caf8100 into o3de:main Apr 12, 2024
8 checks passed
@spham-amzn spham-amzn deleted the fix_pyside2_shiboken_rpath branch September 30, 2024 15:21
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.

3 participants