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.

Sign up for GitHub

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

Jump to bottom

qutebrowser: downgrade Qt to 5.14 #99456

Merged
merged 1 commit into from Oct 4, 2020
Merged

qutebrowser: downgrade Qt to 5.14 #99456

merged 1 commit into from Oct 4, 2020

Conversation

rnhmjoj
Copy link
Contributor

@rnhmjoj rnhmjoj commented Oct 3, 2020

Motivation for this change

There are a number of unresolved issues with Qt 5.15, the most
important one being the inability to properly restore sessions1.
For now it's better to stay at the earlier version.

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using (qutebrowser)
  • Tested execution of all binary files (usually in ./result/bin/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Ensured that relevant documentation is up to date
  • Fits CONTRIBUTING.md.

@rnhmjoj
qutebrowser: downgrade Qt to 5.14
3fafb02 
There are a number of unresolved issues with Qt 5.15, the most
important one being the inability to properly restore sessions[1].
For now it's better to stay at the earlier version.

[1]: qutebrowser/qutebrowser#5359
@rnhmjoj rnhmjoj added the 8.has: port to stable A PR already has a backport to the stable release. label Oct 3, 2020
@woffs
Copy link
Contributor

woffs commented Oct 3, 2020

This is related to #97586 2daf570 #98885 and others. I'm fine with 5.15, but as long as qutebrowser has issues with 5.15, it may be better to run under 5.14. This patch works.

@rnhmjoj rnhmjoj merged commit 0dbce97 into NixOS:master Oct 4, 2020
@rnhmjoj
Copy link
Contributor Author

rnhmjoj commented Oct 4, 2020

Backport: 3bef864

@The-Compiler
Copy link
Contributor

FWIW other than the session support, I think all of those issues are either fixed in Qt 5.15.1 (or .2), or just not relevant outside of qutebrowser development. The only other remaining issue with Qt 5.15.1 are renderer process crashes which can be fixed by backporting this patch: [Backport] Fix for bug 1071327 (Icf418990) · Gerrit Code Review

As for the session support, the choice is between being able to fully restore sessions or having security patches:

  • Qt 5.14.2: Based on Chromium 85.0.4183.83 (~2019-10-10) with security patches up to 80.0.3987.132 (2020-03-03)
  • Qt 5.15.1: Based on Chromium 80.0.3987.163 (2020-04-02) with security patches up to 85.0.4183.83 (2020-08-25)

IMHO, for a web browser, lagging behind more than absolutely necessary with security patches is a bigger problem than not getting back-/forward history after restoring sessions.

@rnhmjoj
Copy link
Contributor Author

rnhmjoj commented Oct 5, 2020

Hi, thank you for chiming in. I didn't most of the issue were already resolved, I'll take a look at the patch and consider porting it.

@gebner gebner mentioned this pull request Oct 8, 2020
Merged
FRidh added a commit to FRidh/nixpkgs that referenced this pull request Oct 8, 2020
@FRidh
qutebrowser: use current qt (5.15) again instead of 5.14
902c567 
For security reasons [1] we upgrade to the latest Qt, despite this
breaking a feature, which is why 5.14 was forced earlier.

[1] NixOS#99456 (comment)
@FRidh
Copy link
Member

FRidh commented Oct 8, 2020

Intention to "revert" to 5.15 in 902c567 as part of #99956.

@doronbehar
Copy link
Contributor

I'm working on a PR to add that patch to our qt5.15

@doronbehar doronbehar mentioned this pull request Oct 8, 2020
Closed
10 tasks
@doronbehar
Copy link
Contributor

qtwebengine PR: #100041

doronbehar added a commit to doronbehar/nixpkgs that referenced this pull request Oct 13, 2020
@doronbehar
qt5.qtwebengine: Sessions crashes fix
b206142 
Per NixOS#99456 (comment)
FRidh added a commit to FRidh/nixpkgs that referenced this pull request Oct 13, 2020
@FRidh
qutebrowser: use current qt (5.15) again instead of 5.14
a0b965a 
For security reasons [1] we upgrade to the latest Qt, despite this
breaking a feature, which is why 5.14 was forced earlier.

[1] NixOS#99456 (comment)

(cherry picked from commit 902c567)
@doronbehar doronbehar mentioned this pull request Nov 5, 2020
Closed
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@woffs woffs woffs approved these changes

@ebzzry ebzzry Awaiting requested review from ebzzry

@jagajaga jagajaga Awaiting requested review from jagajaga

Assignees
No one assigned
Labels
8.has: port to stable A PR already has a backport to the stable release. 10.rebuild-darwin: 1-10 10.rebuild-darwin: 1 10.rebuild-linux: 1-10 10.rebuild-linux: 1
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@rnhmjoj @woffs @The-Compiler @FRidh @doronbehar