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

Snap builds broken on master #380

Closed
maxiberta opened this issue May 7, 2020 · 4 comments
Closed

Snap builds broken on master #380

maxiberta opened this issue May 7, 2020 · 4 comments

Comments

@maxiberta
Copy link
Contributor

maxiberta commented May 7, 2020

Hi! Snap builds are currently broken on master:

gui/sdlgamepadkeynavigation.cpp: In member function ‘void SdlGamepadKeyNavigation::sendWheel(QPoint&)’:
gui/sdlgamepadkeynavigation.cpp:253:145: error: no matching function for call to ‘QWheelEvent::QWheelEvent(QPoint&, QPoint&, QPoint, QPoint&, int, int, Qt::ScrollPhase, bool, Qt::MouseEventSource)’
         QWheelEvent wheelEvent(mousePos, globalPos, QPoint(), angleDelta, 0, 0, Qt::NoScrollPhase, false, Qt::MouseEventSynthesizedByApplication);

I think this is the relevant commit. I'm no expert but it looks like a newer Qt is expected?

The snap is currently based on core18 + Qt 5.9. It could hopefully be based on core20 + Qt 5.12 soon, but the qmake plugin does not yet support core20. Alternatively, the snap could use a custom build of Qt - feasible, but not trivial.

Cheers! o/

@auto-comment
Copy link

auto-comment bot commented May 7, 2020

If this is a question about Moonlight or you need help troubleshooting a streaming problem, please use the help channels on our Discord server instead of GitHub issues. There are many more people available on Discord to help you and answer your questions.

This issue tracker should only be used for specific bugs or feature requests.

Thank you, and happy streaming!

@cgutman
Copy link
Member

cgutman commented May 8, 2020

Thanks, I pushed a fix. I saw that the Steam Link's Qt 5.9 build was failing too.

On the subject of a newer Qt, I am hoping to be able to drop Qt 5.9 support soon. It will also be nice to have the dependencies and other libraries updated too.

Maybe we could make it work with core20 on Snapcraft 4.0 by using the make plugin and using override-build to run qmake first?

@maxiberta
Copy link
Contributor Author

maxiberta commented May 11, 2020

Thanks!
Quick update: already published a core20 based snap to the edge channel (the make hack was a great idea, btw!).
Bad news: core20 is amd64 only. But the i386 snap has ~100 active users and I'd rather not leave them unsupported. So, considering/trying alternatives:

  • Go back to core18 and use the kde-neon PPA.
  • Go back to core18 and depend on the KF5 content snap.

@maxiberta
Copy link
Contributor Author

Update: the snap on edge is now based on core18 + kde-neon (Qt 5.14.2) - both amd64 and i386.

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

No branches or pull requests

2 participants