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

Use Qt 5.12.3 to work around macOS issues. #1558

Draft
wants to merge 48 commits into
base: master
Choose a base branch
from

Conversation

JulianGro
Copy link
Contributor

@JulianGro JulianGro commented Dec 31, 2021

This PR works around the issues we are experiencing when mixing macOSXSDK10.12 with Qt 5.15.2.

Fixes #1516

Depends on PR #1556

Tested on macOS Big Sur

@JulianGro JulianGro added needs testing (QA) The PR is ready for testing needs CR (code review) macos This is related to OSX and Mac. Qt labels Dec 31, 2021
@Steve-Pruneau
Copy link

Oh wow! I am absolutely loving PR 1558 !!! This is the best macOS interface yet!

It is stable and movement smooth (not as much lag during loading).

This should go into production.

@digisomni digisomni added QA Approved The PR has been tested successfully. and removed needs testing (QA) The PR is ready for testing labels Jan 2, 2022
@digisomni digisomni added this to the 2022.1.1 Selene Release milestone Jan 2, 2022
@digisomni
Copy link
Member

Maybe before each if statement we should have a "TODO" noting that it's a hack and may not really last long term as we have to move up in Qt versions over time which includes reworking more code.

@JulianGro
Copy link
Contributor Author

That would make the code even messier than it already is, and all occurrences can be found by the same if statement.
Adding something like "TODO: Remove once there is a Metal rendering backend" to ~120 if statements just seems a bit much.
Maybe someone that actually works on Qt deprecation like @daleglass could chime in.

@JulianGro
Copy link
Contributor Author

We actually also have if statements like this in the code already.
So if we decide on adding TODOs, then we should add them to those as well.

@digisomni digisomni changed the title Use Qt 5.12.3 to work around macOS issues Use Qt 5.12.3 to work around macOS issues. Jan 4, 2022
@ArcadeFever
Copy link
Contributor

ArcadeFever commented Jan 16, 2022

Sorry for the late comment on this, but is the goal to downgrade all macOS builds or split out "Big Sur" from the mac builds to resolve Steve's issue? If this PR is doing the former, then it will break web entity support for zoom/google docs/ etc, which currently work fine with newer macOS versions and Selene.

@JulianGro
Copy link
Contributor Author

I don't see how downgrading all builds would not break web entity support while a secondary build works.

Whatever the case, this works around all the OpenGL issues. If the current client works on your mac that is fine but that is an exception. Without this PR there is crashes on startup, crashes on resizing window, and freezes on startup on most Apple machines running macOS Big Sur or newer.

@ArcadeFever
Copy link
Contributor

So turns out I was running the release candidate for Selene on my MacBook, thus crossed out last part above. Testing with the full 2022.1.0 release it is also failing to launch for me. In fact, 2021.1.3 and 2021.1.2 also fail to launch, but 2021.1.1 works (and use the "interface" client before it was renamed to Vircadia).

The concern with downgrading QT, is it will break web entity support for certain applications like Zoom and Google Docs, so would be a step backward, but I can understand not a lot of mac users probably need this support right now.

I would think we should be investigating why the Selene RC build and 2021.1.1 (and prior) worked fine with QT 5.15.2 on mac. So I hope this PR fix to rollback QT is only temporary.

@JulianGro
Copy link
Contributor Author

The concern with downgrading QT, is it will break web entity support for certain applications like Zoom and Google Docs, so would be a step backward, but I can understand not a lot of mac users probably need this support right now.

Will it break support or have you actually tested the PR?

I would think we should be investigating why the Selene RC build and 2021.1.1 (and prior) worked fine with QT 5.15.2 on mac. So I hope this PR fix to rollback QT is only temporary.

I know why some builds worked and some didn't. Try fullscreen mode on that RC build you have been using.
That is a build that was made with a different macOSXSDK.
The macOSXSDK that we need to use to work around Apples OpenGL issues doesn't work properly together with newer Qt and macOS Big Sur or higher. And with properly I mean that it doesn't work at all for me and Steve.

This PR is to gain some time. If we don't get Metal support somehow, we will eventually not be able to run on macOS anymore.

ksuprynowicz and others added 20 commits January 28, 2022 19:40
set the new MS url for Overte: https://overte.org/mvs
update federation.json with the new MS url for Overte.
Ressources rebranding for Overte
(mostly images, icon, model updated with Overte brand)
some file has been edited to replace some files (they had vircadia in the name)
Most of this has not been tested. (cause I don't build)
Replaced by brand-logo.svg
Replaced by:
logo_overte.fbx
logo_overte_white-color-emissive.fst
Replaced by brand.png
Replaced by : brand-logo-512.png
This rebrands the resource used for the installer. 

Note 1: I have not upgrade the file under "launchers" directory, these are visibly not used (still High Fidelity brand)

Note 2: There is one file I can't modify: "install-folder.rsrc". There is nothing to edit that on the web... this would need a mac.

Note 3: there was a slogan on 2 pictures under the logo: "virtual realities for one, for all". I left this blank for now... We would need a slogan to replace tit.
New Overte logo in a better size and spacing
Adjustment for Logo ratio
Logo ratio adjustment
@digisomni
Copy link
Member

This PR needs to be updated from Vircadia master.

@JulianGro JulianGro marked this pull request as draft February 19, 2022 22:28
@digisomni digisomni added do not merge do not merge due to issues or pending updates and removed QA Approved The PR has been tested successfully. needs CR (code review) labels Feb 20, 2022
@stale
Copy link

stale bot commented Aug 31, 2022

Hello! Is this still an issue?

@stale stale bot added the stale Issue / PR has not had activity label Aug 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do not merge do not merge due to issues or pending updates macos This is related to OSX and Mac. Qt stale Issue / PR has not had activity
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2022.1.0-Selene will not launch on macOS 11.6 "Big Sur"
9 participants