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.

Sign up for GitHub

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

Jump to bottom

Upgrade PhysX5 to 5.5.1 on windows/linux platform. #271

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Upgrade PhysX5 to 5.5.1 on windows/linux platform. #271

wants to merge 1 commit into from

Conversation

rossbridger
Copy link

Signed-off-by: Ross Charles Campbell [email protected]

@nick-l-o3de
Copy link
Contributor

Hmm, failed on ubuntu, I'm thinking

 PermissionError: [Errno 13] Permission denied: '/tmp/tmpzt2619uz/physx/buildtools/packman/packman'

Feels like it might need to have a chmod +x in there or something?

@nick-l-o3de
Copy link
Contributor

as for windows, failed

CMake Error at CMakeLists.txt:38 (project):
  Generator

    Visual Studio 16 2019

  could not find any instance of Visual Studio.

Are we trying to foce it to vs16? Why?

@rossbridger
Copy link
Author

Upgraded vs version to 2022, now need to figure out linux situation.

@rossbridger rossbridger force-pushed the main branch 2 times, most recently from 063a7b5 to fc12bd5 Compare February 5, 2025 08:20
@rossbridger
Copy link
Author

I have fixed Linux build. Note that Linux build now use gcc as its compiler since when I test it on GCS clang build fails despite it having clang installed.

@rossbridger
Copy link
Author

I don't have a aarch64 machine to test the build, may have to leave it be for now

@rossbridger rossbridger changed the title Upgrade PhysX5 to 5.5.0 on windows/linux platform. Upgrade PhysX5 to 5.5.1 on windows/linux platform. Feb 11, 2025
@rossbridger
Copy link
Author

Upgraded physx to 5.5.1, fixed linux-aarch64 build.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nick-l-o3de nick-l-o3de nick-l-o3de approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@rossbridger @nick-l-o3de