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

Fix typo #2067

Merged
merged 1 commit into from
Nov 28, 2021
Merged

Fix typo #2067

merged 1 commit into from
Nov 28, 2021

Conversation

rayw000
Copy link
Contributor

@rayw000 rayw000 commented Nov 28, 2021

Fix typo

Copy link
Member

@alex-w alex-w left a comment

Choose a reason for hiding this comment

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

Thanks!

@alex-w alex-w merged commit dd006bc into Stellarium:master Nov 28, 2021
@gzotti
Copy link
Member

gzotti commented Nov 28, 2021

Oops, thanks!

@rayw000
Copy link
Contributor Author

rayw000 commented Nov 28, 2021

Thanks!

BTW, seems WITH_QTWEBENGINE didn't work well for me.

I built with brew installed Qt 5.15.2. Seemed that it's shipped along with QtWebEngine. Line 30 evaluated true in my case.

FIND_PACKAGE(Qt5WebEngine QUIET)
FIND_PACKAGE(Qt5WebEngineWidgets QUIET)
IF(Qt5WebEngine_FOUND AND Qt5WebEngineWidgets_FOUND)
ADD_DEFINITIONS(-DWITH_QTWEBENGINE)
SET(QTWEBENGINE_LIBS Qt5::WebEngine Qt5::WebEngineWidgets)
ELSE(Qt5WebEngine_FOUND AND Qt5WebEngineWidgets_FOUND)
MESSAGE(WARNING "Qt5WebEngine and/or Qt5WebEngineWidgets not found. \
If these packages exist for your platform, install and rerun. \
Else the program will call the system web browser where required.")
ENDIF(Qt5WebEngine_FOUND AND Qt5WebEngineWidgets_FOUND)

I've read #2006 and #2049. -DWITH_QTWEBENGINE=0 didn't work for me, neither -DWITH_QTWEBENGINE=off. There was a cmake warning saying

CMake Warning:
  Manually-specified variables were not used by the project:

    WITH_QTWEBENGINE

and issue #1982 was still.

Only fully disabling online queries plugin with option -DUSE_PLUGIN_ONLINEQUERIES=false worked for me.

I built on Monterey 12.0.1 with Apple M1 Max chip. I'm trying to find out why.

@alex-w
Copy link
Member

alex-w commented Nov 28, 2021

Issue #1982 happened with Qt 5.15 only and probably this is bug in Qt/macOS.

@rayw000
Copy link
Contributor Author

rayw000 commented Nov 28, 2021

Oh, I thought we could walk around it by -DWITH_QTWEBENGINE=off after reading your reply #2006 (comment).

@gzotti
Copy link
Member

gzotti commented Nov 28, 2021

I have made another round of corrections, seeing that my Odroid C1 SBC with Ubuntu 18LTS has QWebEngine, shows a webengine panel, but fails to work, emitting error messages to stderr.

Now

  • We detect QWebEngine availablilty at compile time, emit a warning if not found, but compile anyhow.
  • On systems where QWebEngine is present but buggy, compile with it, but manually disable the built-in QWebEngineView and open the URL in the system webbrowser.

Now finally the external system webbrowser also works on that (minimal, 5fps) platform.
The User Guide (sources) includes instructions for users.

@gzotti
Copy link
Member

gzotti commented Nov 28, 2021

The point seems to be that at least on ARM platform, QWebEngine does not work properly before Qt6.2, and also not on X11 platforms with some GLX1.3 problem.

@github-actions
Copy link

Hello @rayw000! Please check the fresh version (development snapshot) of Stellarium:
https://github.com/Stellarium/stellarium-data/releases/tag/weekly-snapshot

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants