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

Skip Vulkan SDK installation in macOS if it is already installed. #98510

Closed
wants to merge 1 commit into from

Conversation

0xafbf
Copy link
Contributor

@0xafbf 0xafbf commented Oct 25, 2024

Skip installation of Vulkan SDK if it is already installed.
This is useful when setting up user-hosted workers that run under the same environment every time.

@@ -3,6 +3,11 @@
set -euo pipefail
IFS=$'\n\t'

if [ -d "$HOME/VulkanSDK" ]; then
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If I recall correctly godot engine uses the builtin system. Might want to investigate reusing that. This is simpler though.

Copy link
Member

@bruvzg bruvzg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure about this.

I think most users would expect to get VulkanSDK updated if it's already installed.

For the workers that are running in the persistent environment, it would make more sense to run this script as a cron task to update to the new version once a week, instead of running in every time.

Ideally it should check for installed available version, it can be done by downloading https://sdk.lunarg.com/sdk/download/latest/mac/config.json first, and checking version value from it against the list of the subfolders in $HOME/VulkanSDK.

@0xafbf
Copy link
Contributor Author

0xafbf commented Dec 17, 2024

Closing as superseded by #98608

@0xafbf 0xafbf closed this Dec 17, 2024
@0xafbf 0xafbf deleted the skip-vulkan-macos branch December 17, 2024 01:35
@akien-mga akien-mga removed this from the 4.x milestone Dec 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants