Skip to content

Conversation

@ann0see
Copy link
Member

@ann0see ann0see commented Aug 3, 2025

Short description of changes

Moves the macOS Qt installation to ~/qt as it seems that the home directory is cachable. /opt/ which would be the more standard compliant option would result in a permission error. I have Qt installed in my home directory on my mac test machine and it worked so far.

CHANGELOG: SKIP
Context: Fixes an issue?
Fixes: #3373

Does this change need documentation? What needs to be documented and how?

No

Status of this Pull Request

Ready for review

What is missing until this pull request can be merged?

Review

Checklist

  • I've verified that this Pull Request follows the general code principles
  • I tested my code and it does what I want
  • My code follows the style guide
  • I waited some time after this Pull Request was opened and all GitHub checks completed without errors.
  • I've filled all the content above

@ann0see ann0see added bug Something isn't working macOS macOS runtime issue labels Aug 3, 2025
@github-project-automation github-project-automation bot moved this to Triage in Tracking Aug 3, 2025
@ann0see ann0see added this to the Release 3.11.0 milestone Aug 3, 2025
@ann0see ann0see requested a review from pljones August 3, 2025 19:46
@ann0see ann0see moved this from Triage to Waiting on Team in Tracking Aug 3, 2025
Copy link
Member

@softins softins left a comment

Choose a reason for hiding this comment

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

Looks ok to me.

But it's possible it might interact with #3511, due to them changing adjacent lines in autobuild.yml, so a rebase might be required after merging whichever is done first.

@pljones
Copy link
Collaborator

pljones commented Aug 6, 2025

In fact, given the size of the change here, maybe do it with #3511?

@ann0see
Copy link
Member Author

ann0see commented Aug 8, 2025

@pljones I would rather have as small commits as possible

Copy link
Collaborator

@pljones pljones left a comment

Choose a reason for hiding this comment

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

Builds - we'll see the results later.

@pljones pljones merged commit 823289b into jamulussoftware:main Aug 8, 2025
39 of 48 checks passed
@github-project-automation github-project-automation bot moved this from Waiting on Team to Done in Tracking Aug 8, 2025
@ann0see ann0see deleted the autobuild/FixGtarPermissionIssue branch August 9, 2025 09:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working macOS macOS runtime issue

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Build process error and warning for 3.11.0rc1

3 participants