Skip to content

Migrate from qt4 to qt5 [WIP]#33239

Closed
peterhoeg wants to merge 8 commits intoNixOS:masterfrom
peterhoeg:f/qt4to5
Closed

Migrate from qt4 to qt5 [WIP]#33239
peterhoeg wants to merge 8 commits intoNixOS:masterfrom
peterhoeg:f/qt4to5

Conversation

@peterhoeg
Copy link
Copy Markdown
Member

@peterhoeg peterhoeg commented Dec 31, 2017

Motivation for this change

Qt4 is deprecated.

Mixxx can simply be recompiled for Qt5.
OpenMW is also updated from 0.42.0 to 0.43.0 with a slightly newer openscenegraph.
Vanilla openscenegraph is updated from 3.4.0 -> 3.4.1

If you guys think they should be split up into multiple different PRs for the sake of not changing too many things at the same time, let me know.

Cc: @abbradar @aszlig @cillianderoiste @7c6f434c

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-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 nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

@GrahamcOfBorg GrahamcOfBorg added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. labels Dec 31, 2017
@lukateras
Copy link
Copy Markdown
Member

lukateras commented Dec 31, 2017 •
edited
Loading

I would like to take on some Qt4 apps too. List of files mentioning Qt4: [truncated, moved to #33248]

@lukateras
Copy link
Copy Markdown
Member

lukateras commented Dec 31, 2017

223 total.

I think we already have Qt5 version of VLC in libsForQt5.vlc attribute, but we might want to drop Qt4 version.

pkgs/tools/inputmethods/* will all be dropped when Qt4 is removed entirely, and will also remove Qt4 from closure of everyone who uses non-alphabetic writing systems.

LibreOffice will probably land Qt5 in February: https://www.phoronix.com/scan.php?page=news_item&px=LibreOffice-VCL-Qt5

@orivej
Copy link
Copy Markdown
Contributor

orivej commented Dec 31, 2017

It is fine to build applications that support Qt 5 with Qt 5, but is there any reason to stop building applications with Qt 4 that don't (e.g. i7z)? Qt 4 works well and is not going to be removed in any near future.

@FRidh FRidh added the 6.topic: qt/kde Object-oriented framework for GUI creation label Dec 31, 2017
@adisbladis
Copy link
Copy Markdown
Member

adisbladis commented Dec 31, 2017

@yegortimoshenko Wouldnt it be better to create a tracking issue with that list? I doubt that all of those packages will be handled by this PR.

@orivej I agree with you. I think it may be worth revisiting that question once there are only a few packages depending on Qt4 but for now I don't see any reason to drop Qt4 from packages not supporting Qt5.

@orivej
Copy link
Copy Markdown
Contributor

orivej commented Dec 31, 2017

@peterhoeg The fork of qjoypad that you have found is pretty good, but its UI is a bit deficient compared to its qt4 ancestor. I'd like these pull requests to be merged before we switch: panzi/qjoypad#17 and panzi/qjoypad#18.

@7c6f434c
Copy link
Copy Markdown
Member

7c6f434c commented Dec 31, 2017

Sounds reasonable for xxdiff, for openscenegraph I am not sure whether flightgear or speed_dreams care about Qt support in the library. Hopefully they use as much as openmw, though.

@lukateras lukateras mentioned this pull request Dec 31, 2017
Closed
@lukateras
Copy link
Copy Markdown
Member

lukateras commented Dec 31, 2017

@orivej: i7z-gui segfaults for me, so I too think we should drop it, if it doesn't work for anyone:

$ i7z-gui
i7z DEBUG: i7z version: svn-r77-(20-Nov-2011)
i7z DEBUG: Found Intel Processor
i7z DEBUG:    Stepping 7
i7z DEBUG:    Model a
i7z DEBUG:    Family 6
i7z DEBUG:    Processor Type 0
i7z DEBUG:    Extended Model 2
i7z DEBUG: msr = Model Specific Register
segmentation fault

But in other cases, patching application to support Qt5 is preferable.

@adisbladis
Copy link
Copy Markdown
Member

adisbladis commented Dec 31, 2017 •
edited
Loading

@yegortimoshenko Works fine for me.

@orivej
Copy link
Copy Markdown
Contributor

orivej commented Dec 31, 2017

Works for me too. It must be run as root, but when it isn't it doesn't segfault, it prints:

i7z DEBUG: i7z version: svn-r77-(20-Nov-2011)
i7z DEBUG: Found Intel Processor
i7z DEBUG:    Stepping 3
i7z DEBUG:    Model e
i7z DEBUG:    Family 6
i7z DEBUG:    Processor Type 0
i7z DEBUG:    Extended Model 4
i7z DEBUG: msr = Model Specific Register
i7z DEBUG: Unknown processor, not exactly based on Nehalem, Sandy bridge or Ivy Bridge
i7z DEBUG: msr device files exist /dev/cpu/*/msr
i7z DEBUG: You DONOT have write permissions to msr device files
i7z DEBUG: A solution is to run this program as root

@lukateras
Copy link
Copy Markdown
Member

lukateras commented Dec 31, 2017

Interesting, i7z doesn't segfault for me (output identical to #33239 (comment)) but i7z-gui does.

@peterhoeg peterhoeg mentioned this pull request Apr 19, 2018
Closed
8 tasks
@nixos-discourse
Copy link
Copy Markdown

nixos-discourse commented Feb 13, 2019

This pull request has been mentioned on Nix community. There might be relevant details there:

https://discourse.nixos.org/t/bump-openmw-0-44-0-and-ported-to-qt5/2121/1

@geekware geekware mentioned this pull request Feb 14, 2019
Closed
10 tasks
@geekware geekware mentioned this pull request Mar 30, 2019
Closed
10 tasks
@mmahut
Copy link
Copy Markdown
Member

mmahut commented Aug 20, 2019

Are there any updates on this pull request, please?

@peterhoeg
Copy link
Copy Markdown
Member Author

peterhoeg commented Aug 26, 2019

No longer relevant.

@peterhoeg peterhoeg closed this Aug 26, 2019
@peterhoeg peterhoeg deleted the f/qt4to5 branch August 26, 2019 12:52
@woffs woffs mentioned this pull request Sep 9, 2020
Merged
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

6.topic: qt/kde Object-oriented framework for GUI creation 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

10 participants

@peterhoeg @lukateras @orivej @adisbladis @7c6f434c @nixos-discourse @mmahut @matthewbauer @FRidh @GrahamcOfBorg